[FFmpeg-devel] [PATCH] Fix termination of mov_ch_layouts_wav array lookup

[Clément Bœsch]

On Mon, Mar 18, 2013 at 02:00:23PM -0700, Thierry Foucu wrote:
> Problem found using ASAN.
> In some case, the ff_mov_get_channel_layout_tag function will not find 0
> as termination for lookup in the array mov_ch_layouts_wav.
> ---
>  libavformat/mov_chan.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavformat/mov_chan.c b/libavformat/mov_chan.c
> index cf1f839..7078b4a 100644
> --- a/libavformat/mov_chan.c
> +++ b/libavformat/mov_chan.c
> @@ -426,6 +426,7 @@ static const enum MovChannelLayoutTag mov_ch_layouts_wav[] = {
>      MOV_CH_LAYOUT_MPEG_7_1_A,
>      MOV_CH_LAYOUT_MPEG_7_1_C,
>      MOV_CH_LAYOUT_SMPTE_DTV,
> +    0,
>  };
>  
>  static const struct {

Isn't it missing in mov_ch_layout_map as well by the way?

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130319/b4c769f0/attachment.asc>