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

Mike Melanson mike
Mon Dec 12 21:16:13 CET 2005


John Koleszar wrote:
> 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

	That's a big Cinepak frame. Good catch. I will validate this and apply it.

	Thanks...
-- 
	-Mike Melanson





More information about the ffmpeg-devel mailing list