[FFmpeg-cvslog] avcodec/snowdec: Check qbias

Michael Niedermayer git at videolan.org
Sat May 6 03:12:30 EEST 2017


ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Sat May  6 01:08:54 2017 +0200| [523205ce1ed9415183c162998c68f573479e78fe] | committer: Michael Niedermayer

avcodec/snowdec: Check qbias

Fixes: signed integer overflow: -1094995529 * 131 cannot be represented in type 'int'
Fixes: 1353/clusterfuzz-testcase-minimized-5208180449607680

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpeg
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>

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

 libavcodec/snowdec.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/libavcodec/snowdec.c b/libavcodec/snowdec.c
index 042aecbbeb..97f55288c1 100644
--- a/libavcodec/snowdec.c
+++ b/libavcodec/snowdec.c
@@ -395,6 +395,11 @@ static int decode_header(SnowContext *s){
         s->block_max_depth= 0;
         return AVERROR_INVALIDDATA;
     }
+    if (FFABS(s->qbias) > 127) {
+        av_log(s->avctx, AV_LOG_ERROR, "qbias %d is too large\n", s->qbias);
+        s->qbias = 0;
+        return AVERROR_INVALIDDATA;
+    }
 
     return 0;
 }



More information about the ffmpeg-cvslog mailing list