On 07/07/2012 08:39 PM, Anton Khirnov wrote:
> ---
>  libavformat/matroskadec.c |    5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
> index 3b78ae3..40c2eb1 100644
> --- a/libavformat/matroskadec.c
> +++ b/libavformat/matroskadec.c
> @@ -783,8 +783,11 @@ static int ebml_parse_id(MatroskaDemuxContext *matroska, 
> EbmlSyntax *syntax,
>          matroska->num_levels > 0 &&
>          matroska->levels[matroska->num_levels-1].length == 0xffffffffffffff)
>          return 0;  // we reached the end of an unknown size cluster
> -    if (!syntax[i].id && id != EBML_ID_VOID && id != EBML_ID_CRC32)
> +    if (!syntax[i].id && id != EBML_ID_VOID && id != EBML_ID_CRC32) {
>          av_log(matroska->ctx, AV_LOG_INFO, "Unknown entry 0x%X\n", id);
> +        if (matroska->ctx->error_recognition & AV_EF_EXPLODE)
> +            return AVERROR_INVALIDDATA;
> +    }
>      return ebml_parse_elem(matroska, &syntax[i], data);
>  }
>  

Looks ok to me.

-- 

Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero

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

Reply via email to