Re: gdiplus: Fix a memory leak in the tests.

[Nikolay Sivov]

On 12/15/2009 14:35, Huw Davies wrote:
> Found by Valgrind.
>
> dlls/gdiplus/tests/customlinecap.c |    1 +
>   1 files changed, 1 insertions(+), 0 deletions(-)

Hi, Huw.

This leak will be fixed with corrected  GdipCreateCustomLineCap(). 
Return code should never
be Ok here.

P.S. Same with previous patch for  GdipCreateFontFamilyFromNam().

gdiplus: Fix a memory leak in the tests.

[Huw Davies]

On Tue, Dec 15, 2009 at 02:40:26PM +0300, Nikolay Sivov wrote:
> On 12/15/2009 14:35, Huw Davies wrote:
> > Found by Valgrind.
> >
> > dlls/gdiplus/tests/customlinecap.c |    1 +
> >   1 files changed, 1 insertions(+), 0 deletions(-)
>
> Hi, Huw.
>
> This leak will be fixed with corrected  GdipCreateCustomLineCap().  
> Return code should never
> be Ok here.
>
> P.S. Same with previous patch for  GdipCreateFontFamilyFromNam().

Yes I know.  And when it's fixed the GdipDelete* calls could be
removed along with the todo_wine.  At the moment it's easier to
get them out of the Valgrind logs so that the real leaks are more
obvious.

Huw.