intel: Don't call intelDestroyContext if there is no context to destroy
authorIan Romanick <ian.d.romanick@intel.com>
Fri, 28 Sep 2012 15:29:12 +0000 (08:29 -0700)
committerIan Romanick <ian.d.romanick@intel.com>
Fri, 28 Sep 2012 22:05:24 +0000 (15:05 -0700)
Some error paths in the device-specific context creation functions can exit
before the deintel_context structure is allocated.

NOTE: This is a candidate for the 9.0 branch.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Eric Anholt <eric@anholt.net>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=53618
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=54301

src/mesa/drivers/dri/intel/intel_screen.c

index 5cb7e5d7d228651137695ed2552766ad42590eaa..d6e6408afb2541759fe8d9b9c3d80a9333a44953 100644 (file)
@@ -821,7 +821,9 @@ intelCreateContext(gl_api api,
    if (success)
       return true;
 
-   intelDestroyContext(driContextPriv);
+   if (driContextPriv->driverPrivate != NULL)
+      intelDestroyContext(driContextPriv);
+
    return false;
 }