[Mesa-dev] [PATCH] i965: Don't leave dangling pointer to brw context on failure

Ian Romanick idr at freedesktop.org
Fri Aug 31 15:03:27 PDT 2012


From: Ian Romanick <ian.d.romanick at 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.romanick at 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;
    }
-- 
1.7.11.4



More information about the mesa-dev mailing list