[FFmpeg-devel] [PATCH] IFF: Check for invalid width and height in decoder

Sebastian Vater cdgs.basty
Mon May 10 00:11:36 CEST 2010


M?ns Rullg?rd a ?crit :
> Sebastian Vater <cdgs.basty at googlemail.com> writes:
>
>   
>> +    if ((avctx->width <= 0) || (avctx->height <= 0)) {
>>     
>
> Too much () IMO.
>   

Fixed. Better this way? ;)

Although I personally prefer the method with the extra (), find it
myself a bit more readable.
So if you don't hesistate, please apply the other one.

-- 

Best regards,
                   :-) Basty/CDGS (-:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: iff-decoder-width-height-check.patch
Type: text/x-patch
Size: 616 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100510/edd769af/attachment.bin>



More information about the ffmpeg-devel mailing list