[Mesa-dev] [PATCH] util: fix memory leak from the fragment shaders for SINT <-> UINT blits

[Brian Paul]

Looks good.  Reviewed-by: Brian Paul <brianp at vmware.com>


On Wed, Nov 23, 2016 at 6:33 PM, Charmaine Lee <charmainel at vmware.com>
wrote:

> This patch deletes those fragment shaders in util_blitter_destroy().
> ---
>  src/gallium/auxiliary/util/u_blitter.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/auxiliary/util/u_blitter.c
> b/src/gallium/auxiliary/util/u_blitter.c
> index 98b5421..45dc033 100644
> --- a/src/gallium/auxiliary/util/u_blitter.c
> +++ b/src/gallium/auxiliary/util/u_blitter.c
> @@ -453,7 +453,7 @@ void util_blitter_destroy(struct blitter_context
> *blitter)
>     }
>
>     for (i = 0; i < PIPE_MAX_TEXTURE_TYPES; i++) {
> -      for (unsigned type = 0; type < 3; ++type) {
> +      for (unsigned type = 0; type < ARRAY_SIZE(ctx->fs_texfetch_col);
> ++type) {
>           if (ctx->fs_texfetch_col[type][i])
>              ctx->delete_fs_state(pipe, ctx->fs_texfetch_col[type][i]);
>           if (ctx->fs_texfetch_col_msaa[type][i])
> --
> 1.9.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161123/0899718b/attachment-0001.html>