[FFmpeg-cvslog] Revert "avcodec/svq1: zero initialize entries array"

Clément Bœsch git at videolan.org
Mon Apr 10 17:09:14 EEST 2017


ffmpeg | branch: master | Clément Bœsch <cboesch at gopro.com> | Mon Apr 10 13:46:52 2017 +0200| [8839cbf91188098d91573fc5876454e2e0874707] | committer: Clément Bœsch

Revert "avcodec/svq1: zero initialize entries array"

This reverts commit aed84ee4d1b0c9e315a84b1ee0918fa49ee9cc09.

It is a false positive from Valgrind with recent GCC and
-funswitch-loops (present in -O3).

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

 libavcodec/svq1dec.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcodec/svq1dec.c b/libavcodec/svq1dec.c
index e5e43fc8e2..d3e60c3a4a 100644
--- a/libavcodec/svq1dec.c
+++ b/libavcodec/svq1dec.c
@@ -160,7 +160,7 @@ static int svq1_decode_block_intra(GetBitContext *bitbuf, uint8_t *pixels,
     uint8_t *list[63];
     uint32_t *dst;
     const uint32_t *codebook;
-    int entries[6] = { 0 };
+    int entries[6];
     int i, j, m, n;
     int stages;
     unsigned mean;
@@ -227,7 +227,7 @@ static int svq1_decode_block_non_intra(GetBitContext *bitbuf, uint8_t *pixels,
     uint8_t *list[63];
     uint32_t *dst;
     const uint32_t *codebook;
-    int entries[6] = { 0 };
+    int entries[6];
     int i, j, m, n;
     int stages;
     unsigned mean;



More information about the ffmpeg-cvslog mailing list