On Sun, 17 Feb 2013 09:25:09 +0100, Reinhard Tartler <[email protected]> 
wrote:
> From: Michael Niedermayer <[email protected]>
> 
> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> Signed-off-by: Michael Niedermayer <[email protected]>
> (cherry picked from commit a7ee6281f7ef1c29284e3a4cadfe0f227ffde1ed)
> 
> Signed-off-by: Reinhard Tartler <[email protected]>
> ---
>  libavcodec/qdm2.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/libavcodec/qdm2.c b/libavcodec/qdm2.c
> index 1286cc5..61ac0a8 100644
> --- a/libavcodec/qdm2.c
> +++ b/libavcodec/qdm2.c
> @@ -1232,6 +1232,11 @@ static void qdm2_decode_super_block (QDM2Context *q)
>      for (i = 0; packet_bytes > 0; i++) {
>          int j;
>  
> +        if (i >= FF_ARRAY_ELEMS(q->sub_packet_list_A)) {
> +            SAMPLES_NEEDED_2("too many packet bytes");
> +            return;
> +        }
> +
>          q->sub_packet_list_A[i].next = NULL;
>  
>          if (i > 0) {
> -- 
> 1.7.9.5
> 

Looks mostly ok.

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

Reply via email to