[Mesa-dev] [PATCH] egl/dri: Don't invoke dri2_dpy->flush if it's NULL.

Jose Fonseca jfonseca at vmware.com
Tue Feb 5 11:44:21 PST 2013



----- Original Message -----
> On 02/05/2013 06:43 AM, jfonseca at vmware.com wrote:
> > From: José Fonseca<jfonseca at vmware.com>
> >
> > I'd like to test Mesa OpenGL ES along side with NVIDIA libGL
> > drivers. But
> > without this change, I get a NULL pointer dereference.
> > ---
> >   src/egl/drivers/dri2/egl_dri2.c |    3 ++-
> >   1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/egl/drivers/dri2/egl_dri2.c
> > b/src/egl/drivers/dri2/egl_dri2.c
> > index 351fbf4..01e6302 100644
> > --- a/src/egl/drivers/dri2/egl_dri2.c
> > +++ b/src/egl/drivers/dri2/egl_dri2.c
> > @@ -970,7 +970,8 @@ dri2_wait_client(_EGLDriver *drv, _EGLDisplay
> > *disp, _EGLContext *ctx)
> >      /* FIXME: If EGL allows frontbuffer rendering for window
> >      surfaces,
> >       * we need to copy fake to real here.*/
> >
> > -   (*dri2_dpy->flush->flush)(dri2_surf->dri_drawable);
> > +   if (dri2_dpy->flush != NULL)
> > +      (*dri2_dpy->flush->flush)(dri2_surf->dri_drawable);
> 
> I'd simplify that line to read:
> 
>           dri2_dpy->flush->flush(dri2_surf->dri_drawable);

Sure. Thanks for the review.

Jose


More information about the mesa-dev mailing list