Hi, On Wed, Jan 25, 2017 at 8:12 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote:
> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com> > --- > libavformat/nistspheredec.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/libavformat/nistspheredec.c b/libavformat/nistspheredec.c > index 782d1dfbfb..3386497682 100644 > --- a/libavformat/nistspheredec.c > +++ b/libavformat/nistspheredec.c > @@ -21,6 +21,7 @@ > > #include "libavutil/avstring.h" > #include "libavutil/intreadwrite.h" > +#include "libavcodec/internal.h" > #include "avformat.h" > #include "internal.h" > #include "pcm.h" > @@ -90,6 +91,11 @@ static int nist_read_header(AVFormatContext *s) > return 0; > } else if (!memcmp(buffer, "channel_count", 13)) { > sscanf(buffer, "%*s %*s %"SCNd32, &st->codecpar->channels); > + if (st->codecpar->channels > FF_SANE_NB_CHANNELS) { > + av_log(s, AV_LOG_ERROR, "Too many channels %d > %d\n", > + st->codecpar->channels, FF_SANE_NB_CHANNELS); > + return AVERROR(ENOSYS); > + } I've said this before, but again - please don't add useless log messages. These don't help end users at all, since these samples are exceedingly unlikely to be real. Most likely, they derive from fuzzing, and stderr cramming is one of the things that makes fuzzing slower. Ronald _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel