summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBrian Paul <[email protected]>2008-05-07 08:56:08 -0600
committerBrian Paul <[email protected]>2008-05-07 08:56:08 -0600
commit3e1a4c286936abdb4ce1b62a9ebdd93db1777aff (patch)
tree406f997a530e29e9f3367fa257098b16d8d0e126
parenta56a59ce74b7f18f25a13992d2a2c1ae7cf973ce (diff)
mesa: free shader program data before deleting shader objects.
Picked from master. Fixes mem corruption seen when glean/api2 test exits.
-rwxr-xr-xsrc/mesa/main/context.c16
1 files changed, 16 insertions, 0 deletions
diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index b9053344b26..893c79f28ca 100755
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/context.c
@@ -631,6 +631,21 @@ delete_arrayobj_cb(GLuint id, void *data, void *userData)
}
/**
+ * Callback for freeing shader program data. Call it before delete_shader_cb
+ * to avoid memory access error.
+ */
+static void
+free_shader_program_data_cb(GLuint id, void *data, void *userData)
+{
+ GLcontext *ctx = (GLcontext *) userData;
+ struct gl_shader_program *shProg = (struct gl_shader_program *) data;
+
+ if (shProg->Type == GL_SHADER_PROGRAM_MESA) {
+ _mesa_free_shader_program_data(ctx, shProg);
+ }
+}
+
+/**
* Callback for deleting shader and shader programs objects.
* Called by _mesa_HashDeleteAll().
*/
@@ -714,6 +729,7 @@ free_shared_state( GLcontext *ctx, struct gl_shared_state *ss )
_mesa_DeleteHashTable(ss->ArrayObjects);
#if FEATURE_ARB_shader_objects
+ _mesa_HashWalk(ss->ShaderObjects, free_shader_program_data_cb, ctx);
_mesa_HashDeleteAll(ss->ShaderObjects, delete_shader_cb, ctx);
_mesa_DeleteHashTable(ss->ShaderObjects);
#endif