[FFmpeg-cvslog] checkasm: h264dsp: Initialize the padding area

Martin Storsjö git at videolan.org
Thu Aug 11 20:58:47 EEST 2016


ffmpeg | branch: master | Martin Storsjö <martin at martin.st> | Mon Jun 27 22:22:21 2016 +0300| [2e95054ebb73f95aa6620b76245313a6ccb62245] | committer: Michael Niedermayer

checkasm: h264dsp: Initialize the padding area

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ö <martin at martin.st>
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=2e95054ebb73f95aa6620b76245313a6ccb62245
---

 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 e2060db..2c2603b 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);          \
             }                                                                \



More information about the ffmpeg-cvslog mailing list