[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