[FFmpeg-devel] [PATCH] AAC: give an error message when returning due to an unallocated channel element

[Baptiste Coudurier]

Hi,

Alex Converse wrote:
> I've run into this a few times. It's worth a message.
> 
> Regards,
> Alex Converse
> 
> 
> ------------------------------------------------------------------------
> 
> diff --git a/libavcodec/aac.c b/libavcodec/aac.c
> index 7275430..64c2ae6 100644
> --- a/libavcodec/aac.c
> +++ b/libavcodec/aac.c
> @@ -1576,7 +1576,10 @@ static int aac_decode_frame(AVCodecContext * avccontext, void * data, int * data
>          }
>          if(elem_type < TYPE_DSE) {
>              if(!ac->che[elem_type][elem_id])
> +            {
> +                av_log(ac->avccontext, AV_LOG_ERROR, "channel element %d.%d is unallocated\n", elem_type, elem_id);
>                  return -1;
> +            }
>              if(elem_type != TYPE_CCE)
>                  ac->che[elem_type][elem_id]->coup.coupling_point = 4;
>          }
> 

Braces are misplaced. Should be "if () {"

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
checking for life_signs in -lkenny... no
FFmpeg maintainer                                  http://www.ffmpeg.org