From aead688e7fc605fada2624626c7543445412791a Mon Sep 17 00:00:00 2001 From: Tilman Sauerbeck Date: Tue, 11 Jul 2006 18:35:24 +0000 Subject: [PATCH] _mesa_destroy_context() frees radeon->glCtx so we must not access it afterwards --- src/mesa/drivers/dri/r300/radeon_context.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) 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; -- 2.30.2