[Ffmpeg-devel] [PATCH] Cinepak header data length bug

[John Koleszar]

Hi all,

The attached one-liner fixes a bug in cinepak.c where the data length is 
parsed from the frame header as 16 bits instead of the required 24. For 
further info, see 
http://www.csse.monash.edu.au/~timf/videocodec/cinepak.txt, and grep for 
'Length of CVID data'. Note that this site is referenced in the header 
of cinepak.c.

Currently, if the data length of the frame is >=65536, the data length 
read from the header won't match the data length read from the demuxer, 
and the automagic sega detection introduced in rev 1.10 kicks in, 
causing the strip headers to be parsed with the wrong offset.

An example clip demonstrating this invalid behavior is available at:
ftp://opensource:opensource at ftp.on2.com/saturn_720_480_5sec_cinepak.avi

Cheers,

John Koleszar

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: cinepak_data_len.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20051212/97df22b1/attachment.asc>