On 29/11/13 14:16, Tim W. wrote: > > 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? >
Looks nice, even NSYNCAUX would do. lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
