On Mit, 2013-02-20 at 08:20 +0100, Vinson Lee wrote: 
> Fixes resource leak defect reported by Coverity.
> 
> Signed-off-by: Vinson Lee <v...@freedesktop.org>
> ---
>  src/gallium/drivers/radeonsi/si_state.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/gallium/drivers/radeonsi/si_state.c 
> b/src/gallium/drivers/radeonsi/si_state.c
> index d20e3ff..7f76cb5 100644
> --- a/src/gallium/drivers/radeonsi/si_state.c
> +++ b/src/gallium/drivers/radeonsi/si_state.c
> @@ -1940,6 +1940,7 @@ int si_shader_select(struct pipe_context *ctx,
>                       R600_ERR("Failed to build shader variant (type=%u) 
> %d\n",
>                                sel->type, r);
>                       sel->current = NULL;
> +                     FREE(shader);
>                       return r;
>               }

Reviewed-by: Michel Dänzer <michel.daen...@amd.com>

Can you fix the same problem in r600g while you're at it? :) Thanks
Vinson.


-- 
Earthling Michel Dänzer           |                   http://www.amd.com
Libre software enthusiast         |          Debian, X and DRI developer
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to