aboutsummaryrefslogtreecommitdiffstats
path: root/libavformat
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2020-10-20 21:55:13 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2021-10-09 22:02:19 +0200
commit5a1cc01fdf4b6f9d5cf2cd7cffa62d9dae5d0c72 (patch)
treee87bd494de9ed69f4c9e30545c62233fe44c1989 /libavformat
parent2c6b8fadfc039814ba9f71402d1f3b0026cd3c63 (diff)
downloadffmpeg-5a1cc01fdf4b6f9d5cf2cd7cffa62d9dae5d0c72.tar.gz
avformat/lrcdec: Clip timestamps
Fixes: signed integer overflow: 7111111111111531010 - -7335632962598013506 cannot be represented in type 'long' Fixes: 26463/clusterfuzz-testcase-minimized-ffmpeg_dem_LRC_fuzzer-6015558333759488 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 80bc2ac3c06319cf85428c58c471d105d25ae987) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavformat')
-rw-r--r--libavformat/lrcdec.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/libavformat/lrcdec.c b/libavformat/lrcdec.c
index 45f0bf24a7..fa379372a1 100644
--- a/libavformat/lrcdec.c
+++ b/libavformat/lrcdec.c
@@ -185,6 +185,8 @@ static int lrc_read_header(AVFormatContext *s)
sscanf(comma_offset + 1, "%"SCNd64, &lrc->ts_offset) != 1) {
av_dict_set(&s->metadata, line.str + 1, comma_offset + 1, 0);
}
+ lrc->ts_offset = av_clip64(lrc->ts_offset, INT64_MIN/4, INT64_MAX/4);
+
*comma_offset = ':';
*right_bracket_offset = ']';
}
@@ -198,6 +200,7 @@ static int lrc_read_header(AVFormatContext *s)
while((ts_stroffset_incr = read_ts(line.str + ts_stroffset,
&ts_start)) != 0) {
+ ts_start = av_clip64(ts_start, INT64_MIN/4, INT64_MAX/4);
ts_stroffset += ts_stroffset_incr;
sub = ff_subtitles_queue_insert(&lrc->q, line.str + ts_strlength,
line.len - ts_strlength, 0);