[vlc-devel] [PATCH] Avoid inifite loops and stack explosion when parsing broken files and ignore lvl 1 unknown elements.

Edward Wang edward.c.wang at compdigitec.com
Fri Jan 11 01:06:10 CET 2013


On 13-01-10 07:00 PM, Denis Charmet <typx at dinauz.org> wrote:
> Fix #8013
> ---
>   modules/demux/mkv/Ebml_parser.cpp |   29 ++++++++++++++++++++++++-----
>   1 file changed, 24 insertions(+), 5 deletions(-)
Just tested it, and it does fix the problem!

(Btw, the following warnings popped up during compile, any action required?)
../../../modules/demux/mkv/Ebml_parser.cpp: In member function 
‘libebml::EbmlElement* EbmlParser::Get(int)’:
../../../modules/demux/mkv/Ebml_parser.cpp:231:217: warning: format 
‘%lu’ expects argument of type ‘long unsigned int’, but argument 5 has 
type ‘uint64 {aka long long unsigned int}’ [-Wformat]
../../../modules/demux/mkv/Ebml_parser.cpp:238:255: warning: format 
‘%lu’ expects argument of type ‘long unsigned int’, but argument 5 has 
type ‘uint64 {aka long long unsigned int}’ [-Wformat]

Regards,
         Edward Wang



More information about the vlc-devel mailing list