diff options
| author | Ilia Mirkin <[email protected]> | 2015-05-17 17:56:44 -0400 |
|---|---|---|
| committer | Ilia Mirkin <[email protected]> | 2015-05-25 16:52:11 -0400 |
| commit | 5646f0f18a620292524eebcd77353ff3d3687eb2 (patch) | |
| tree | b3618998e360ae9b0b536865a28ab554c9a9a5eb /src/glsl/linker.cpp | |
| parent | 6a111e54d7578abee6bce4a75ce1399ed369ab5f (diff) | |
glsl: avoid leaking linked gl_shader when there's a late linker error
This makes piglit mixing-clip-distance-and-clip-vertex-disallowed have 0
definitely lost blocks with valgrind. (Same non-0 number of possibly
lost blocks though.)
Signed-off-by: Ilia Mirkin <[email protected]>
Reviewed-by: Tobias Klausmann <[email protected]>
Cc: "10.5 10.6" <[email protected]>
Diffstat (limited to 'src/glsl/linker.cpp')
| -rw-r--r-- | src/glsl/linker.cpp | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp index 9798afefc98..99e0a388bb4 100644 --- a/src/glsl/linker.cpp +++ b/src/glsl/linker.cpp @@ -2829,8 +2829,11 @@ link_shaders(struct gl_context *ctx, struct gl_shader_program *prog) link_intrastage_shaders(mem_ctx, ctx, prog, shader_list[stage], num_shaders[stage]); - if (!prog->LinkStatus) + if (!prog->LinkStatus) { + if (sh) + ctx->Driver.DeleteShader(ctx, sh); goto done; + } switch (stage) { case MESA_SHADER_VERTEX: @@ -2843,8 +2846,11 @@ link_shaders(struct gl_context *ctx, struct gl_shader_program *prog) validate_fragment_shader_executable(prog, sh); break; } - if (!prog->LinkStatus) + if (!prog->LinkStatus) { + if (sh) + ctx->Driver.DeleteShader(ctx, sh); goto done; + } _mesa_reference_shader(ctx, &prog->_LinkedShaders[stage], sh); } |