On Tue, Sep 17, 2013 at 7:34 AM, Anton Khirnov <[email protected]> wrote: > > 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. >
Maybe it could fail only when err_recognition has AV_EF_EXPLODE? Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
