[Mesa-dev] [PATCH 5/7] vl/idct: cleanup all idct buffers

[Christian König]

Am 17.08.2013 23:51, schrieb Emil Velikov:
> Code should loop through and cleanup the three (VL_NUM_COMPONENTS) idct
> buffers, rather than doing the first one three times.
>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>

Thanks, I was already wondering for quite a while where that memory leak 
actually is.

Reviewed-by: Christian König <christian.koenig at amd.com>

> ---
>   src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c b/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c
> index de890fe..cff326f 100644
> --- a/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c
> +++ b/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c
> @@ -198,7 +198,7 @@ cleanup_idct_buffer(struct vl_mpeg12_buffer *buf)
>      assert(buf);
>   
>      for (i = 0; i < 3; ++i)
> -      vl_idct_cleanup_buffer(&buf->idct[0]);
> +      vl_idct_cleanup_buffer(&buf->idct[i]);
>   }
>   
>   static bool