[FFmpeg-devel] [PATCH 5/5] avformat/bintext: Check width in idf_read_header()

Michael Niedermayer michael at niedermayer.cc
Mon Nov 2 02:21:28 EET 2020


Fixes: division by 0
Fixes: 26802/clusterfuzz-testcase-minimized-ffmpeg_dem_IDF_fuzzer-5180591554953216.fuzz

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
---
 libavformat/bintext.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavformat/bintext.c b/libavformat/bintext.c
index 810d870637..bc0f6bd099 100644
--- a/libavformat/bintext.c
+++ b/libavformat/bintext.c
@@ -346,6 +346,8 @@ static int idf_read_header(AVFormatContext *s)
 
     bin->fsize = avio_size(pb) - 12 - 4096 - 48;
     ff_sauce_read(s, &bin->fsize, &got_width, 0);
+    if (st->codecpar->width < 8)
+        return AVERROR_INVALIDDATA;
     if (!bin->width)
         calculate_height(st->codecpar, bin->fsize);
     avio_seek(pb, 12, SEEK_SET);
-- 
2.17.1



More information about the ffmpeg-devel mailing list