Traneptora/FFmpeg
1
Fork 0
forked from FFmpeg/FFmpeg
Code Pull requests Activity

avformat/mxfdec: Check edit_unit for being larger than signed 64bit

Browse source 
Fixes: signed integer overflow: 2 * -4962931467012268000 cannot be represented in type 'long'
Fixes: 376496313/clusterfuzz-testcase-minimized-ffmpeg_dem_MXF_fuzzer-4921469185884160

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 2024-12-01 03:53:42 +01:00
parent 4bd9b5f520
commit f7cc023f06
No known key found for this signature in database
GPG key ID: B18E8928B3948D64
1 changed files with 2 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
libavformat/mxfdec.c
View file

@ -1928,6 +1928,8 @@ static int mxf_edit_unit_absolute_offset(MXFContext *mxf, MXFIndexTable *index_t
// clamp to actual range of index
index_end = av_sat_add64(last_segment->index_start_position, last_segment->index_duration);
edit_unit = FFMAX(FFMIN(edit_unit, index_end), first_segment->index_start_position);
if (edit_unit < 0)
return AVERROR_PATCHWELCOME;
// guess which table segment this edit unit is in
// saturation is fine since it's just a guess