diff options
author | Michael Niedermayer <michael@niedermayer.cc> | 2020-12-11 00:49:23 +0100 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2021-10-09 22:02:20 +0200 |
commit | a9fe6abe0a1c86717c1b80df00af458d2d6e97d4 (patch) | |
tree | e93edb9e03fb37b244779fed37d7634dfe515ae1 | |
parent | fe15426edda12b6d8345460f8f7d4be447e84387 (diff) | |
download | ffmpeg-a9fe6abe0a1c86717c1b80df00af458d2d6e97d4.tar.gz |
avformat/iff: Check data_size
Fixes: infinite loop
Fixes: 27834/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-5694930919620608
Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 001bc594d82f3df67a6e96c6ea022f4e39002385)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r-- | libavformat/iff.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavformat/iff.c b/libavformat/iff.c index 5024182a02..03d62fdafc 100644 --- a/libavformat/iff.c +++ b/libavformat/iff.c @@ -367,7 +367,7 @@ static int read_dst_frame(AVFormatContext *s, AVPacket *pkt) data_size = iff->is_64bit ? avio_rb64(pb) : avio_rb32(pb); data_pos = avio_tell(pb); - if (data_size < 1) + if (data_size < 1 || data_size >= INT64_MAX) return AVERROR_INVALIDDATA; switch (chunk_id) { |