har0ke/FFmpeg
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

avformat/mpeg: Don't use unintialized value

Browse Source 
vobsub_read_packet() didn't check whether an array of AVPackets was
valid and therefore used uninitialized values.

Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
This commit is contained in:
Andreas Rheinhardt 2019-10-22 14:54:09 +02:00
parent 46d2b2071b
commit a39536caee
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
libavformat/mpeg.c
View File

@ -927,6 +927,10 @@ static int vobsub_read_packet(AVFormatContext *s, AVPacket *pkt)
FFDemuxSubtitlesQueue *tmpq = &vobsub->q[i]; FFDemuxSubtitlesQueue *tmpq = &vobsub->q[i];
int64_t ts; int64_t ts;
av_assert0(tmpq->nb_subs); av_assert0(tmpq->nb_subs);
if (tmpq->current_sub_idx >= tmpq->nb_subs)
continue;
ts = tmpq->subs[tmpq->current_sub_idx].pts; ts = tmpq->subs[tmpq->current_sub_idx].pts;
if (ts < min_ts) { if (ts < min_ts) {
min_ts = ts; min_ts = ts;