[FFmpeg-devel] [PATCH] mkv 0-byte integer parsing

Reimar Döffinger Reimar.Doeffinger
Tue Sep 7 21:08:26 CEST 2010


On Tue, Sep 07, 2010 at 08:14:41PM +0200, Sebastian Hagen wrote:
> >Attached patch adds a length check for that case (this is independant
> >of this patch though).
> 
> *reads patch*
> >+        [EBML_UINT]  = 4,
> Please don't. According to the mkv specs[1], those can go up to 8
> bytes

No need for a long explanation, that was just a typo, it wasn't
meant to restrict the allowed values further.
Still, attached updated patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: maxlen.diff
Type: text/x-diff
Size: 1432 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100907/5cd8f7ad/attachment.diff>



More information about the ffmpeg-devel mailing list