Mesa (master): mesa/st: Check for a NULL _LinkedShader[i] before using it.

Stephane Marchesin marcheu at kemper.freedesktop.org
Mon Oct 10 07:15:14 UTC 2011


Module: Mesa
Branch: master
Commit: ddba509c16c4cb0630a4f4841b31953f02be6b3f
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ddba509c16c4cb0630a4f4841b31953f02be6b3f

Author: Stéphane Marchesin <marcheu at chromium.org>
Date:   Sun Oct  9 16:49:19 2011 -0700

mesa/st: Check for a NULL _LinkedShader[i] before using it.

The rest of the linker/glsl translation code checks for NULL, so I suppose we should check here too. Fixes crash on exit with i915g instanced drawing.

Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>

---

 src/mesa/state_tracker/st_program.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/src/mesa/state_tracker/st_program.c b/src/mesa/state_tracker/st_program.c
index acd3b56..c419c40 100644
--- a/src/mesa/state_tracker/st_program.c
+++ b/src/mesa/state_tracker/st_program.c
@@ -1154,7 +1154,8 @@ destroy_shader_program_variants_cb(GLuint key, void *data, void *userData)
          }
 
 	 for (i = 0; i < Elements(shProg->_LinkedShaders); i++) {
-	    destroy_program_variants(st, shProg->_LinkedShaders[i]->Program);
+	    if (shProg->_LinkedShaders[i])
+               destroy_program_variants(st, shProg->_LinkedShaders[i]->Program);
 	 }
       }
       break;




More information about the mesa-commit mailing list