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

[Denis Charmet]

Le jeudi 10 janvier 2013 à 07:06:10, Edward Wang a écrit :
> 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!
>
I know, I don't post patch for the lulz :p
 
> (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]

Yeah already fixed locally with %"PRIu64".

-- 
TypX
Le mauvais esprit est un art de vivre