On Sun,  2 Oct 2011 12:50:44 -0400, Justin Ruggles <[email protected]> 
wrote:
> From: Laurent Aimar <[email protected]>
> 
> Signed-off-by: Michael Niedermayer <[email protected]>
> ---
>  libavcodec/qdm2.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/libavcodec/qdm2.c b/libavcodec/qdm2.c
> index 326ff79..6f9ff0d 100644
> --- a/libavcodec/qdm2.c
> +++ b/libavcodec/qdm2.c
> @@ -1354,6 +1354,8 @@ static void qdm2_fft_decode_tones (QDM2Context *q, int 
> duration, GetBitContext *
>              return;
>  
>          local_int_14 = (offset >> local_int_8);
> +        if (local_int_14 >= FF_ARRAY_ELEMS(fft_level_index_table))
> +            return;
>  
>          if (q->nb_channels > 1) {
>              channel = get_bits1(gb);
> -- 
> 1.7.1
> 

What's up with the variable names? Was this code written by a decompiler ;)

Patch looks fine.

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

Reply via email to