On Fri, Nov 7, 2014 at 6:08 PM, Luca Barbato <[email protected]> wrote: > On 30/10/14 01:52, Vittorio Giovara wrote: >> >> CC: [email protected] >> Bug-Id: CID 1087090 >> --- >> libavcodec/aacdec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c >> index cc74bd6..6fca9b1 100644 >> --- a/libavcodec/aacdec.c >> +++ b/libavcodec/aacdec.c >> @@ -143,7 +143,7 @@ static av_cold int che_configure(AACContext *ac, >> enum ChannelPosition che_pos, >> int type, int id, int *channels) >> { >> - if (*channels >= MAX_CHANNELS) >> + if (*channels >= MAX_CHANNELS - 2) >> return AVERROR_INVALIDDATA; >> if (che_pos) { >> if (!ac->che[type][id]) { >> > > why -2 ?
Because down in the function there are two "(*channels)++" that trigger the warning. -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
