On Mon, 16 Sep 2013 22:06:25 +0300, Martin Storsjö <[email protected]> wrote: > Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind > CC: [email protected] > --- > libavformat/rmdec.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c > index afa37de..9dd32a7 100644 > --- a/libavformat/rmdec.c > +++ b/libavformat/rmdec.c > @@ -328,6 +328,8 @@ ff_rm_read_mdpr_codecdata (AVFormatContext *s, > AVIOContext *pb, > st->codec->codec_type = AVMEDIA_TYPE_VIDEO; > st->need_parsing = AVSTREAM_PARSE_TIMESTAMPS; > fps = avio_rb32(pb); > + if (fps < 0) > + return AVERROR_INVALIDDATA;
This looks overly restrictive, there should be no need to fail just because the framerate is not valid. Just skip setting the average framerate. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
