[FFmpeg-devel] [PATCH] matroskadec: verify seekhead IDs

[Michael Niedermayer]

On Fri, Jun 12, 2015 at 01:11:41PM +0200, wm4 wrote:
> Some files have SeekHead elements with broken IDs. They mismatch with
> the ID of the destination element. These files are written by
> "IDMmkvlib0.1" (as identified by the MuxingApp and WritingApp elements),
> and the SeekHead IDs are actually endian-swapped.
> 
> This confuses the SeekHead logic of the demuxer. It will read some
> elements twice, because the SeekHead ID is used to identify and remember
> already read elements. With the file at hand, the stream list was
> duplicated by reading the Tracks element twice.
> 
> Fix this by rejecting invalid EBML IDs in SeekHead entries. (This fix is
> relatively specific to the broken file at hand, and doesn't protect
> against some other cases of broken SeekHead, such as valid but
> mismatching target element IDs.)
> ---
>  libavformat/matroskadec.c | 12 ++++++++++++
>  1 file changed, 12 insertions(+)

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150612/b1c1302d/attachment.asc>