ffmpeg | branch: release/3.2 | James Almer <jamr...@gmail.com> | Thu Nov  3 
22:34:58 2016 -0300| [ee5677737942ed0ab898f696d39164ee234d2f0a] | committer: 
James Almer

avcodec/rawdec: check for side data before checking its size

Fixes valgrind warnings about usage of uninitialized values.

Reviewed-by: Michael Niedermayer <mich...@niedermayer.cc>
Signed-off-by: James Almer <jamr...@gmail.com>
(cherry picked from commit 51e329918dc1826de7451541cb15bef3b9bfe138)

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

 libavcodec/rawdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/rawdec.c b/libavcodec/rawdec.c
index 45cf27f..e53eb2e 100644
--- a/libavcodec/rawdec.c
+++ b/libavcodec/rawdec.c
@@ -369,7 +369,7 @@ static int raw_decode(AVCodecContext *avctx, void *data, 
int *got_frame,
                                                      &pal_size);
         int ret;
 
-        if (pal_size != AVPALETTE_SIZE) {
+        if (pal && pal_size != AVPALETTE_SIZE) {
             av_log(avctx, AV_LOG_ERROR, "Palette size %d is wrong\n", 
pal_size);
             pal = NULL;
         }

_______________________________________________
ffmpeg-cvslog mailing list
ffmpeg-cvslog@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-cvslog

Reply via email to