RepoMirrors
/
ffmpeg
mirror of https://git.ffmpeg.org/ffmpeg.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

libavformat/iff: Check for overflow in body_end calculation 

Fixes: signed integer overflow: -6322983228386819992 - 5557477266266529857 cannot be represented in type 'long'
Fixes: 50112/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-6329186221948928

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 bcb4690304)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
Michael Niedermayer 2022-08-22 20:31:32 +02:00
parent 28f1396cfb
commit e84ddc90c7
No known key found for this signature in database
GPG Key ID: B18E8928B3948D64
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
libavformat/iff.c
View File

@ -502,6 +502,9 @@ static int iff_read_header(AVFormatContext *s)
case ID_DST: case ID_DST:
case ID_MDAT: case ID_MDAT:
iff->body_pos = avio_tell(pb); iff->body_pos = avio_tell(pb);
if (iff->body_pos < 0 || iff->body_pos + data_size > INT64_MAX)
return AVERROR_INVALIDDATA;
iff->body_end = iff->body_pos + data_size; iff->body_end = iff->body_pos + data_size;
iff->body_size = data_size; iff->body_size = data_size;
if (chunk_id == ID_DST) { if (chunk_id == ID_DST) {