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

Nico Sabbi nicola_sabbi at fastwebnet.it
Fri Jun 9 18:01:21 CEST 2006


Balatoni Denes wrote:

>Hi!
>
>szombat 06 május 2006 17.55-kor Nico Sabbi ezeket a bolcs gondolatokat 
>fogalmazta meg:
>  
>
>>has it changed?
>>I remember that I changed this file to make it accept only the layout of
>>extradata
>>used at the time by matroska (and I had to change all other demuxers
>>accordingly)
>>    
>>
>
>It didn't change, but libvorbis doesn't accept the two different kinds of 
>header format that ffvorbis does. I thought the new one was the one used by 
>matroska, but anyhow it is used by
>http://www.mplayerhq.hu/MPlayer/samples/MPEG-4/vorbis-in-mp4/mi2_vorbis51.mp4
>
>bye
>Denes
>
>  
>

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


-------------- next part --------------
A non-text attachment was scrubbed...
Name: movorbis.diff
Type: text/x-patch
Size: 5027 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20060609/b0c9d6c4/attachment.bin>


More information about the MPlayer-dev-eng mailing list