On 02/11/2016 04:03 AM, Tapani Pälli wrote: > From: Daniel Czarnowski <[email protected]> > > Without the check, unsuccessful xcb_dri2_get_buffers_reply(...) causes > segmentation fault in dri2_get_buffers. > > Signed-off-by: Matt Roper <[email protected]> > Cc: "11.0 11.1" <[email protected] > --- > src/egl/drivers/dri2/platform_x11.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/egl/drivers/dri2/platform_x11.c > b/src/egl/drivers/dri2/platform_x11.c > index 08cbf2d..931ee51 100644 > --- a/src/egl/drivers/dri2/platform_x11.c > +++ b/src/egl/drivers/dri2/platform_x11.c > @@ -472,6 +472,8 @@ dri2_x11_get_buffers(__DRIdrawable * driDrawable, > dri2_surf->drawable, > count, count, attachments); > reply = xcb_dri2_get_buffers_reply (dri2_dpy->conn, cookie, NULL); > + if (reply == NULL) > + return NULL;
There should be a blank line here. With that fixed, this patch is Reviewed-by: Ian Romanick <[email protected]> > buffers = xcb_dri2_get_buffers_buffers (reply); > if (buffers == NULL) > return NULL; > _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
