[5/5] gdiplus: Implement GdipImageSelectActiveFrame.

Dmitry Timoshkov dmitry at baikal.ru
Tue Jun 19 09:29:15 CDT 2012


Vincent Povirk <madewokherd at gmail.com> wrote:

> > +    if (stat == Ok)
> > +    {
> > +        memcpy(&new_image->format, &codec->info.FormatID, sizeof(GUID));
> > +        free_image_data(image);
> > +        if (image->type == ImageTypeBitmap)
> > +            *(GpBitmap *)image = *(GpBitmap *)new_image;
> > +        else if (image->type == ImageTypeMetafile)
> > +            *(GpMetafile *)image = *(GpMetafile *)new_image;
> > +        return Ok;
> > +    }
> 
> You appear to be leaking new_image.

Thanks. I hope that won't prevent accepting other patches in the series.

-- 
Dmitry.



More information about the wine-devel mailing list