From: Tilman Sauerbeck <tilman@freedesktop.org>
Date: Tue, 11 Jul 2006 18:35:24 +0000 (+0000)
Subject: _mesa_destroy_context() frees radeon->glCtx so we must not access it afterwards
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=aead688e7fc605fada2624626c7543445412791a;p=mesa.git

_mesa_destroy_context() frees radeon->glCtx so we must not access it afterwards
---

diff --git a/src/mesa/drivers/dri/r300/radeon_context.c b/src/mesa/drivers/dri/r300/radeon_context.c
index 5de16104f0a..d886788b341 100644
--- a/src/mesa/drivers/dri/r300/radeon_context.c
+++ b/src/mesa/drivers/dri/r300/radeon_context.c
@@ -201,14 +201,15 @@ GLboolean radeonInitContext(radeonContextPtr radeon,
  */
 void radeonCleanupContext(radeonContextPtr radeon)
 {
+	/* _mesa_destroy_context() might result in calls to functions that
+	 * depend on the DriverCtx, so don't set it to NULL before.
+	 *
+	 * radeon->glCtx->DriverCtx = NULL;
+	 */
+
 	/* free the Mesa context */
 	_mesa_destroy_context(radeon->glCtx);
 
-	/* the above call might result in calls to functions that depend on
-	 * the DriverCtx.
-	 */
-	radeon->glCtx->DriverCtx = NULL;
-
 	if (radeon->state.scissor.pClipRects) {
 		FREE(radeon->state.scissor.pClipRects);
 		radeon->state.scissor.pClipRects = 0;