On Sunday, May 17, 2015 05:32:24 PM Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin <imir...@alum.mit.edu>
> ---
>  src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp 
> b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> index 74c1932..0c582f5 100644
> --- a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> +++ b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
> @@ -5736,6 +5736,7 @@ get_mesa_program(struct gl_context *ctx,
>      */
>     _mesa_associate_uniform_storage(ctx, shader_program, prog->Parameters);
>     if (!shader_program->LinkStatus) {
> +      free_glsl_to_tgsi_visitor(v);
>        return NULL;
>     }
>  
> 

Huh, seems odd that st_program holds on to the visitor itself, rather
than some data structure produced by it.  But I guess it works...

This is clearly right, at any rate.

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to