On 08/31/2012 03:03 PM, Ian Romanick wrote: > From: Ian Romanick <ian.d.roman...@intel.com> > > Otherwise intelDestroyContext would try to dereference the pointer to > freed memory. > > NOTE: This is a candidate for the 9.0 branch. > > Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=54301 > --- > src/mesa/drivers/dri/i965/brw_context.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/mesa/drivers/dri/i965/brw_context.c > b/src/mesa/drivers/dri/i965/brw_context.c > index 4e5aed6..ab485c3 100644 > --- a/src/mesa/drivers/dri/i965/brw_context.c > +++ b/src/mesa/drivers/dri/i965/brw_context.c > @@ -149,6 +149,7 @@ brwCreateContext(int api, > sharedContextPrivate, &functions )) { > printf("%s: failed to init intel context\n", __FUNCTION__); > FREE(brw); > + driContextPriv->driverPrivate = NULL; > *error = __DRI_CTX_ERROR_NO_MEMORY; > return false; > }
I had a little bit of a hard time figuring out where the NULL check is. It looks like glXDestroyContext() checks. I doubt it can hurt, anyway. Reviewed-by: Kenneth Graunke <kenn...@whitecape.org> _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
