[FFmpeg-devel] Collection of patches

[Thorsten Jordan]

Michael Niedermayer schrieb:
> On Thu, Apr 24, 2008 at 11:52:07AM +0200, Thorsten Jordan wrote:
>> Michael Niedermayer schrieb:
>>> On Tue, Apr 22, 2008 at 12:36:02PM +0200, Thorsten Jordan wrote:
>>> [...]
>>>> Set pixel aspect ratio for xvid wrapper:
>>>> libxvid_set_pixelaspect.patch
>>> This is missing a check for sample_aspect_ratios validity.
>> how is this version? xvid limits for num/den are 1..255 according to xvid.h
>>

>     if (avctx->sample_aspect_ratio.num < 1 || avctx->sample_aspect_ratio.num > 255
>             || sample_aspect_ratio.den < 1 || sample_aspect_ratio.den > 255) {
> 
> at tis point you should see that there is something wrong ...
[...]
> vertical align

sorry, my mistake, here is a version with all things fixed, it compiles
also (should have checked before)

-- 
Regards, Thorsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libxvid_set_pixelaspect.patch
Type: text/x-patch
Size: 998 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080424/e9c20615/attachment.bin>