Quoting Thomas Guillem (2015-04-10 19:04:51)
> ---
> Here is a new patch that move embl_parse check and returns 
> AVERROR_INVALIDDATA.
> 
> Maybe a proper patch is to return AVERROR_INVALIDDATA instead of 0 in
> ebml_parse_id. But then I don't know the correct behavior when this function 
> is
> called from ebml_parse_elem with EBML_PASS syntax type.
> 

Perhaps it'd be even more correct to check whether MATROSKA_ID_CLUSTER
is expected (present in the syntax) at that point, but as we have no way
to test it, I'd rather keep this patch as it seems safest.

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

Reply via email to