Michael Niedermayer (12020-06-23): > Fixes; signed integer overflow: 1 - -9223372036854775808 cannot be > represented in type 'long' > Fixes: > 23490/clusterfuzz-testcase-minimized-ffmpeg_DEMUXER_fuzzer-5133490093031424 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> > --- > libavformat/subtitles.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/subtitles.c b/libavformat/subtitles.c > index ad7f68938e..ccab80391f 100644 > --- a/libavformat/subtitles.c > +++ b/libavformat/subtitles.c > @@ -202,7 +202,7 @@ void ff_subtitles_queue_finalize(void *log_ctx, > FFDemuxSubtitlesQueue *q) > q->sort == SUB_SORT_TS_POS ? cmp_pkt_sub_ts_pos > : cmp_pkt_sub_pos_ts); > for (i = 0; i < q->nb_subs; i++) > - if (q->subs[i].duration < 0 && i < q->nb_subs - 1) > + if (q->subs[i].duration < 0 && i < q->nb_subs - 1 && q->subs[i].pts > != AV_NOPTS_VALUE) > q->subs[i].duration = q->subs[i + 1].pts - q->subs[i].pts; > > if (!q->keep_duplicates)
Having no PTS at this point makes no sense. We should examine why it arrived there. Can you share the test case? Regards, -- Nicolas George
signature.asc
Description: PGP signature
_______________________________________________ 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".
