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

[Diego Biurrun]

On Mon, Dec 12, 2005 at 03:05:42PM -0500, John Koleszar wrote:
> 
> 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

This fixes the sample for me on my G4.

Diego