highperfocused/ffmpeg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

avformat/iff: check size against INT64_MAX

Browse Source 
Bigger sizes are misinterpreted as negative numbers by the API
Fixes: infinite loop
Fixes: 26611/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-4890614975692800

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
Michael Niedermayer
2020-10-27 21:51:08 +01:00
parent 27a99e2c7d
commit f291cd681b
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
libavformat/iff.c
View File

@@ -223,6 +223,9 @@ static int parse_dsd_diin(AVFormatContext *s, AVStream *st, uint64_t eof)
uint64_t orig_pos = avio_tell(pb); uint64_t orig_pos = avio_tell(pb);
const char * metadata_tag = NULL; const char * metadata_tag = NULL;
if (size >= INT64_MAX)
return AVERROR_INVALIDDATA;
switch(tag) { switch(tag) {
case MKTAG('D','I','A','R'): metadata_tag = "artist"; break; case MKTAG('D','I','A','R'): metadata_tag = "artist"; break;
case MKTAG('D','I','T','I'): metadata_tag = "title"; break; case MKTAG('D','I','T','I'): metadata_tag = "title"; break;