[Ffmpeg-devel] [PATCH] Cinepak header data length bug
Diego Biurrun
diego
Tue Dec 13 16:10:43 CET 2005
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
More information about the ffmpeg-devel
mailing list