On 06/30/2012 08:37 PM, Kostya Shishkov wrote:
> 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)

Can we encode those?

lu

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero

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

Reply via email to