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

[Ronald S. Bultje]

Hi,

On Mon, May 10, 2010 at 11:22 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Mon, May 10, 2010 at 04:07:41PM +0200, Sebastian Vater wrote:
>> Ronald S. Bultje a ?crit :
>> > What we usually do is this:
>> > ? ?if ((err = avcodec_check_dimensions(avctx, avctx->width, avctx->height)))
>> > ? ? ? ?return err;
>>
>> Now it should be fine...
>
> avcodec_open() should already do avcodec_check_dimensions() if
> width and height are set.
> so this just checks for the 0 case in practice
>
> anyway, it doesnt hurt to check it like that, so ok

Applied.

Ronald