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.veli...@gmail.com>

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

Reviewed-by: Christian König <christian.koe...@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

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

Reply via email to