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
