diff options
| author | Brian Paul <brianp@vmware.com> | 2013-02-20 07:39:04 -0700 |
|---|---|---|
| committer | Brian Paul <brianp@vmware.com> | 2013-02-22 16:49:05 -0700 |
| commit | 8bb291b0f59e8e13bc252cb35672eb468e53881a (patch) | |
| tree | baf2cb42f52e6bce25bd7b90372e8f68ec14569a | |
| parent | 8589cc41b36b89bc80ad6a4f671e47f81a89d556 (diff) | |
st/mesa: check for dummy programs in destroy_program_variants()
When we destroy an ARB vp/fp whose ID was gen'd but not otherwise used we
get a pointer to the dummy/placeholder program. We can't destroy that one
so just skip it. This only failed during context tear-down because
glDeleteProgramsARB() was already aware of dummy programs.
Fixes https://bugs.freedesktop.org/show_bug.cgi?id=38086
Note: This is a candidate for the stable branches.
Tested-by: Andreas Boll <andreas.boll.dev@gmail.com>
| -rw-r--r-- | src/mesa/state_tracker/st_program.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/state_tracker/st_program.c b/src/mesa/state_tracker/st_program.c index a9111b523a4..f56f7cb1c5d 100644 --- a/src/mesa/state_tracker/st_program.c +++ b/src/mesa/state_tracker/st_program.c @@ -1142,7 +1142,7 @@ st_print_shaders(struct gl_context *ctx) static void destroy_program_variants(struct st_context *st, struct gl_program *program) { - if (!program) + if (!program || program == &_mesa_DummyProgram) return; switch (program->Target) { |