On Sun, Sep 29, 2013 at 01:21:04AM +0300, Martin Storsjö wrote:
> Even if the sample rate is valid, an invalid bitrate could
> pass the mode combination test below.
> 
> CC: [email protected]
> ---
>  libavformat/vqf.c |    7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/libavformat/vqf.c b/libavformat/vqf.c
> index 162c753..a43829b 100644
> --- a/libavformat/vqf.c
> +++ b/libavformat/vqf.c
> @@ -182,6 +182,13 @@ static int vqf_read_header(AVFormatContext *s)
>          break;
>      }
>  
> +    if (read_bitrate / st->codec->channels <  8 ||
> +        read_bitrate / st->codec->channels > 48) {
> +        av_log(s, AV_LOG_ERROR, "Invalid bitrate per channel %d\n",
> +               read_bitrate / st->codec->channels);
> +        return AVERROR_INVALIDDATA;
> +    }
> +
>      switch (((st->codec->sample_rate/1000) << 8) +
>              read_bitrate/st->codec->channels) {
>      case (11<<8) + 8 :
> -- 

looks OK
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to