[PATCH libdrm] intel: avoid null pointer dereference

Thomas H.P. Andersen phomes at gmail.com
Tue Feb 7 14:59:32 UTC 2017


On Fri, Feb 3, 2017 at 11:57 AM, Eric Engestrom
<eric.engestrom at imgtec.com> wrote:
>
> On Thursday, 2017-02-02 23:57:29 +0100, Thomas Hindoe Paaboel Andersen wrote:
> > Introduced in 028715ee
> >
> > Move the dereference after the null check.
>
> Fixes: 028715ee707469189505 ("intel: Avoid the need for most overflow
>                               checks by using a scratch page.")
> Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>

Thanks. I do not have commit access. Can I ask you to push it?

>
> > ---
> >  intel/intel_decode.c | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/intel/intel_decode.c b/intel/intel_decode.c
> > index 803d202..2721ffd 100644
> > --- a/intel/intel_decode.c
> > +++ b/intel/intel_decode.c
> > @@ -3899,7 +3899,7 @@ drm_intel_decode(struct drm_intel_decode *ctx)
> >       int ret;
> >       unsigned int index = 0;
> >       uint32_t devid;
> > -     int size = ctx->base_count * 4;
> > +     int size;
> >       void *temp;
> >
> >       if (!ctx)
> > @@ -3909,6 +3909,7 @@ drm_intel_decode(struct drm_intel_decode *ctx)
> >        * the batchbuffer.  This lets us avoid a bunch of length
> >        * checking in statically sized packets.
> >        */
> > +     size = ctx->base_count * 4;
> >       temp = malloc(size + 4096);
> >       memcpy(temp, ctx->base_data, size);
> >       memset((char *)temp + size, 0xd0, 4096);
> > --
> > 2.9.3
> >


More information about the dri-devel mailing list