On Sat, Jun 30, 2012 at 09:30:50PM +0300, Martin Storsjö wrote:
> ff_wma_init is used only by wmadec and wmaenc, and neither of them
> can handle more than 2 channels.
> 
> This fixes crashes with invalid files.
> 
> Based on patch by Piotr Bandurski and Michael Niedermayer.
> ---
>  libavcodec/wma.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/wma.c b/libavcodec/wma.c
> index 007653f..b61228b 100644
> --- a/libavcodec/wma.c
> +++ b/libavcodec/wma.c
> @@ -78,7 +78,7 @@ int ff_wma_init(AVCodecContext *avctx, int flags2)
>      int coef_vlc_table;
>  
>      if (   avctx->sample_rate <= 0 || avctx->sample_rate > 50000
> -        || avctx->channels    <= 0 || avctx->channels    > 8
> +        || avctx->channels    <= 0 || avctx->channels    > 2
>          || avctx->bit_rate    <= 0)
>          return -1;
>  
> -- 

LGTM (> 2ch is WMA 'Pro' and Lossless anyway)
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to