[Vorbis-dev] Cover art

[Conrad Parker]

2009/3/28 Ian Malone <ibmalone at gmail.com>:
>
> The "size parsing error" actually occurs in oggz's reading of the mode
> section from the end of the decode setup packet and has nothing to do
> with the cover art.  Packetno 2 ends like this:
>
>    0996: 00010100 00010110 00011000 00011010 00011100 00011110  ......
>    099c: 00100000 00100010 00100100 00000000 00000000 00000000   "$...
>    09a2: 00000000 00000000 00000000 00000000 00000000 00000000  ......
>    09a8: 00000000 00000000 00000000 00000100                     ....
>
> It looks like the preceding mapping section ends with a sufficiently
> long string of zeros that oggz_auto.c gets frustrated in attempting to
> read back and until it finds non-zeros in the 32bits that would be the
> window and transform blocks. This is liboggz-0.9.8, I see the warning
> has been #ifdef-ed out in SVN; the size value is still used to read
> mode blocks in.

thanks for checking that out. Can you suggest a neater fix? Should
oggz_auto just check further back in the packet for non-zero values?

Conrad.