On Sat, Jul 11, 2015 at 12:39:09AM +0200, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com> > --- > libavformat/riffdec.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/libavformat/riffdec.c b/libavformat/riffdec.c > index eebd8ed..be55699 100644 > --- a/libavformat/riffdec.c > +++ b/libavformat/riffdec.c > @@ -106,6 +106,11 @@ int ff_get_wav_header(AVIOContext *pb, AVCodecContext > *codec, int size, int big_ > codec->bit_rate = avio_rb32(pb) * 8; > codec->block_align = avio_rb16(pb); > } > + if (codec->bit_rate < 0) { > + av_log(NULL, AV_LOG_ERROR, > + "Invalid bit rate: %d\n", codec->bit_rate); > + return AVERROR_INVALIDDATA; > + }
i think it might be better to just warn on a invalid bitrate and only error out if AV_EF_EXPLODE is set because erroring out from header reading coul end all further decoding of that stream (which might in principle be fine otherwise) [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Opposition brings concord. Out of discord comes the fairest harmony. -- Heraclitus
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel