diff options
author | Michael Niedermayer <michael@niedermayer.cc> | 2022-09-18 16:35:41 +0200 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2022-09-25 13:52:02 +0200 |
commit | 2c737a2cb020f231d936cd6b8c1859f0ffdfa6a3 (patch) | |
tree | a6b1b0c48b8abdea09b9aa6f1bccb528ebd7f836 | |
parent | 1d9553542bafa06ec707f8590df0fa0efb930b00 (diff) | |
download | ffmpeg-2c737a2cb020f231d936cd6b8c1859f0ffdfa6a3.tar.gz |
avformat/sbgdec: Check ts_int in genrate_intervals
There is probably a better place to check for this, but better
here than nowhere
Fixes: signed integer overflow: -9223372036824775808 - 86400000000 cannot be represented in type 'long'
Fixes: 50993/clusterfuzz-testcase-minimized-ffmpeg_dem_SBG_fuzzer-6601162580688896
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 5f529e9147a5c5c8ecf8d5ef0dd569194ce30eed)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r-- | libavformat/sbgdec.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/libavformat/sbgdec.c b/libavformat/sbgdec.c index 36cfff20fc..c86bc40862 100644 --- a/libavformat/sbgdec.c +++ b/libavformat/sbgdec.c @@ -1316,6 +1316,8 @@ static int generate_intervals(void *log, struct sbg_script *s, int sample_rate, /* Pseudo event before the first one */ ev0 = s->events[s->nb_events - 1]; + if (av_sat_sub64(ev0.ts_int, period) != (uint64_t)ev0.ts_int - period) + return AVERROR_INVALIDDATA; ev0.ts_int -= period; ev0.ts_trans -= period; ev0.ts_next -= period; |