Re: [PATCH v18 0/4] MR4661: gdiplus: Implement Gdiplus effects

[Esme Povirk (＠madewokherd)]

Esme Povirk (@madewokherd) commented about dlls/gdiplus/image.c:
> -
> - if (effect->type == RedEyeCorrectionEffect)
> - {
> -
>    if ((paramsize-size > 0) || (((size-paramsize)%sizeof(RECT)) != 0))
> -
>        return InvalidParameter;
> - }
> - else
> - {
> -
>    if (paramsize != size)
> -
>        return InvalidParameter;
> - }
> -
> - switch (effect->type)
> - {
> - case BlurEffect:
> -
>    alloc_copy(params, effect->p.blurparams, size);

I think this will leak memory if parameters were already set.