mirror of
https://github.com/FFmpeg/FFmpeg.git
synced 2024-12-23 12:43:46 +02:00
avcodec/xsubdec: Check parse_timecode()
Fixes: CID1604490 Overflowed constant
Sponsored-by: Sovereign Tech Fund
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
(cherry picked from commit 96fd9417e2
)
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
This commit is contained in:
parent
f99867ef79
commit
343a76e123
@ -59,6 +59,7 @@ static int decode_frame(AVCodecContext *avctx, AVSubtitle *sub,
|
||||
int64_t packet_time = 0;
|
||||
GetBitContext gb;
|
||||
int has_alpha = avctx->codec_tag == MKTAG('D','X','S','A');
|
||||
int64_t start_display_time, end_display_time;
|
||||
|
||||
// check that at least header fits
|
||||
if (buf_size < 27 + 7 * 2 + 4 * (3 + has_alpha)) {
|
||||
@ -73,8 +74,14 @@ static int decode_frame(AVCodecContext *avctx, AVSubtitle *sub,
|
||||
}
|
||||
if (avpkt->pts != AV_NOPTS_VALUE)
|
||||
packet_time = av_rescale_q(avpkt->pts, AV_TIME_BASE_Q, (AVRational){1, 1000});
|
||||
sub->start_display_time = parse_timecode(buf + 1, packet_time);
|
||||
sub->end_display_time = parse_timecode(buf + 14, packet_time);
|
||||
|
||||
sub->start_display_time = start_display_time = parse_timecode(buf + 1, packet_time);
|
||||
sub->end_display_time = end_display_time = parse_timecode(buf + 14, packet_time);
|
||||
if (sub->start_display_time != start_display_time ||
|
||||
sub-> end_display_time != end_display_time) {
|
||||
av_log(avctx, AV_LOG_ERROR, "time code not representable in 32bit\n");
|
||||
return -1;
|
||||
}
|
||||
buf += 27;
|
||||
|
||||
// read header
|
||||
|
Loading…
Reference in New Issue
Block a user