[FFmpeg-devel] [PATCH] Bugfix H.263+ encoder frame header corruption

[Panagiotis Issaris]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Michael Niedermayer wrote:
> Hi
> 
> On Fri, May 25, 2007 at 12:03:06PM +0200, Harald Karner wrote:
>> In libavcodec's h263p encoder under certain circumstances the picture 
>> header gets corrupted:
>> Due to wrong parameter order for an call to the "av_clip" function, the 
>> value for the  "clock divisor" in the "CPCFC" (Custom Picture Code 
>> Frequency Code) field of the H.263 header does not get clipped, which 
>> causes the preceeding fields in the header to be corrupted (namely the 
>> "Picture Height Indication" in case of custom picture format).
>> The problem seems only to appear at low framerates, since only this 
>> leads to large values of the "clock divisor".
>> E.g. encoding a 1024x768 video with 0.5 fps lead to a resolution of 
>> 1024x824 wrongly written to the frame header.
>>
>> The patch corrects the parameter order for the call to "av_clip".
> 
> patch ok

Applied.

With friendly regards,
Takis
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGVvYQ9kOxLuzz4CkRAskBAJ4wMsiDP5bSzjDsnXGNIl2BS5uDfACeJucd
BHtfimqCivmAbKxGvt+o7kU=
=1jLR
-----END PGP SIGNATURE-----