On 29 Nov 2013, at 13:50, Diego Biurrun <[email protected]> wrote:

> On Fri, Nov 29, 2013 at 01:47:48PM +0100, Tim Walker wrote:
>> --- a/libavcodec/dcadec.c
>> +++ b/libavcodec/dcadec.c
>> @@ -1301,8 +1301,10 @@ static int dca_subframe_footer(DCAContext *s, int 
>> base_channel)
>> 
>> -            if (get_bits_long(&s->gb, 32) != 0x9A1105A0) // nSYNCAUX
>> +            if (get_bits_long(&s->gb, 32) != 0x9A1105A0) { // nSYNCAUX
>> +                av_log(s->avctx,AV_LOG_ERROR, "nSYNCAUX mismatch\n");
>>                 return AVERROR_INVALIDDATA;
>> +            }
> 
> I guess a #define for this magic number would not hurt.
> 
> Diego

#define DCA_NSYNCAUX 0x9A1105A0 all right?

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

Reply via email to