[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