[MPlayer-dev-eng] [patch] libvorbis support matrsoka style headers

Nico Sabbi nicola_sabbi at fastwebnet.it
Sat Jun 10 00:14:57 CEST 2006


Nico Sabbi wrote:

>
> attached is the patch I'm going to commit if no one is against it.
> This was for me the chance to discover that there a lot of gross errors
> both in demux_mov.c and in parse_mp4.[ch], the main of which are:
> - esds atoms are not strictly related to aac (demux_mov.c makes 
> everywhere this assumption),
> because they simply contain information+extradata for whichever ES 
> stream they refer to
> - the length of mp4 descriptors is not limited to 1 byte: the length 
> is coded as uvlc and may
> span more bytes (for security reasons I limited decoderConfigLen to 16 
> bits, because bigger
> lengths are unlikely to ever happen).
>
> Patch works both with -ac vorbis and with -av ffvorbis.
>
>    Nico
>

committed



More information about the MPlayer-dev-eng mailing list