From 2e95054ebb73f95aa6620b76245313a6ccb62245 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Mon, 27 Jun 2016 22:22:21 +0300 Subject: [PATCH] checkasm: h264dsp: Initialize the padding area MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This fixes valgrind warnings about conditional jumps based on uninitialized data (even though the uninitialized data only ever was compared with a direct copy of the same uninitialized data). Signed-off-by: Martin Storsjö Signed-off-by: Michael Niedermayer --- tests/checkasm/h264dsp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/checkasm/h264dsp.c b/tests/checkasm/h264dsp.c index e2060db3bf..2c2603b2ca 100644 --- a/tests/checkasm/h264dsp.c +++ b/tests/checkasm/h264dsp.c @@ -36,7 +36,7 @@ static const uint32_t pixel_mask[3] = { 0xffffffff, 0x01ff01ff, 0x03ff03ff }; do { \ uint32_t mask = pixel_mask[bit_depth - 8]; \ for (y = 0; y < sz; y++) { \ - for (x = 0; x < sz * SIZEOF_PIXEL; x += 4) { \ + for (x = 0; x < PIXEL_STRIDE; x += 4) { \ AV_WN32A(src + y * PIXEL_STRIDE + x, rnd() & mask); \ AV_WN32A(dst + y * PIXEL_STRIDE + x, rnd() & mask); \ } \