Andreas Rheinhardt: > On Sun, Jan 19, 2020 at 3:44 PM Andreas Rheinhardt < > andreas.rheinha...@gmail.com> wrote: > >> Michael Niedermayer: >>> On Tue, Oct 22, 2019 at 03:16:45PM +0200, Andreas Rheinhardt wrote: >>>> vobsub_read_packet() didn't check whether an index in array of AVPackets >>>> was valid and therefore used uninitialized values. >>>> >>>> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@gmail.com> >>>> --- >>>> Actually I only wanted to use Valgrind to check for memleaks... >>>> >>>> libavformat/mpeg.c | 4 ++++ >>>> 1 file changed, 4 insertions(+) >>>> >>>> diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c >>>> index 73ade71d95..474afd06b9 100644 >>>> --- a/libavformat/mpeg.c >>>> +++ b/libavformat/mpeg.c >>>> @@ -930,6 +930,10 @@ static int vobsub_read_packet(AVFormatContext *s, >> AVPacket *pkt) >>>> FFDemuxSubtitlesQueue *tmpq = &vobsub->q[i]; >>>> int64_t ts; >>>> av_assert0(tmpq->nb_subs); >>>> + >>>> + if (tmpq->current_sub_idx >= tmpq->nb_subs) >>>> + continue; >>> >>> How can this issue be reproduced ? >>> >>> thx >>> >>> [...] >> >> Read a VobSub subtitle till the end: >> ffmpeg -i <idx input file> -c copy -f null - >> >> - Andreas >> > > Ping. > > - Andreas > Ping.
- Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
