[FFmpeg-devel] [PATCH] vp9/parser: change size type to unsigned.

Ronald S. Bultje rsbultje at gmail.com
Sat Feb 8 12:50:43 CET 2014


Fixes valgrind errors in fuzzed8.ivf because size >= 0x80000000.
---
 libavcodec/vp9_parser.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcodec/vp9_parser.c b/libavcodec/vp9_parser.c
index 2de8937..af033c2 100644
--- a/libavcodec/vp9_parser.c
+++ b/libavcodec/vp9_parser.c
@@ -73,12 +73,12 @@ static int parse(AVCodecParserContext *ctx,
 #define case_n(a, rd) \
             case a: \
                 while (n_frames--) { \
-                    int sz = rd; \
+                    unsigned sz = rd; \
                     idx += a; \
                     if (sz > size) { \
                         s->n_frames = 0; \
                         av_log(avctx, AV_LOG_ERROR, \
-                               "Superframe packet size too big: %d > %d\n", \
+                               "Superframe packet size too big: %u > %d\n", \
                                sz, size); \
                         return size; \
                     } \
-- 
1.8.4



More information about the ffmpeg-devel mailing list