diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2021-06-24 18:57:51 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2021-10-06 14:41:42 +0200 |
| commit | 953f316b1bb3e631a5992e69baf7a0336339bd99 (patch) | |
| tree | c47c53a860e8d4a793905b579f7cf5332efd2c66 | |
| parent | bd947157eb904222056103ea6bcd6d204edf45d8 (diff) | |
| download | ffmpeg-953f316b1bb3e631a5992e69baf7a0336339bd99.tar.gz | |
avformat/sbgdec: Check opt_duration and start for overflow
Fixes: signed integer overflow: 2788626175500000000 + 7118941284000000000 cannot be represented in type 'long'
Fixes: 35215/clusterfuzz-testcase-minimized-ffmpeg_dem_SBG_fuzzer-6123272247836672
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 2768928624793f66694f7f2b0824f052e69e3557)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
| -rw-r--r-- | libavformat/sbgdec.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/libavformat/sbgdec.c b/libavformat/sbgdec.c index 54279be5ca..b880f4aa1e 100644 --- a/libavformat/sbgdec.c +++ b/libavformat/sbgdec.c @@ -933,6 +933,9 @@ static int expand_timestamps(void *log, struct sbg_script *s) } if (s->start_ts == AV_NOPTS_VALUE) s->start_ts = (s->opt_start_at_first && s->tseq) ? s->tseq[0].ts.t : now; + if (s->start_ts > INT64_MAX - s->opt_duration) + return AVERROR_INVALIDDATA; + s->end_ts = s->opt_duration ? s->start_ts + s->opt_duration : AV_NOPTS_VALUE; /* may be overridden later by -E option */ cur_ts = now; |