[FFmpeg-cvslog] avcodec/truemotion1: Check that the input has enough space for a minimal index_stream

Michael Niedermayer git at videolan.org
Tue Oct 8 17:28:24 EEST 2019


ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Tue Sep 24 01:43:35 2019 +0200| [4a660fac9899191d4121cde02f2a98977b1303b6] | committer: Michael Niedermayer

avcodec/truemotion1: Check that the input has enough space for a minimal index_stream

Fixes: Timeout (18sec -> 0.4sec)
Fixes: 17585/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_TRUEMOTION1_fuzzer-5117015135617024

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

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

 libavcodec/truemotion1.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavcodec/truemotion1.c b/libavcodec/truemotion1.c
index bcfe473852..b6481cbd7c 100644
--- a/libavcodec/truemotion1.c
+++ b/libavcodec/truemotion1.c
@@ -444,6 +444,8 @@ static int truemotion1_decode_header(TrueMotion1Context *s)
     if (s->flags & FLAG_KEYFRAME) {
         /* no change bits specified for a keyframe; only index bytes */
         s->index_stream = s->mb_change_bits;
+        if (s->avctx->width * s->avctx->height / 2048 + header.header_size > s->size)
+            return AVERROR_INVALIDDATA;
     } else {
         /* one change bit per 4x4 block */
         s->index_stream = s->mb_change_bits +



More information about the ffmpeg-cvslog mailing list