On 02/11/2016 04:03 AM, Tapani Pälli wrote: > From: Daniel Czarnowski <daniel.czarnow...@intel.com> > > Without the check, unsuccessful xcb_dri2_get_buffers_reply(...) causes > segmentation fault in dri2_get_buffers. > > Signed-off-by: Matt Roper <matthew.d.ro...@intel.com> > Cc: "11.0 11.1" <mesa-sta...@lists.freedesktop.org > --- > 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 <ian.d.roman...@intel.com> > buffers = xcb_dri2_get_buffers_buffers (reply); > if (buffers == NULL) > return NULL; > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
