[Bug 27938] New: GdipDrawString Resource leak (GDI Font object).

http://bugs.winehq.org/show_bug.cgi?id=27938

Summary: GdipDrawString Resource leak (GDI Font object).
           Product: Wine
           Version: 1.2.3
          Platform: x86
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: gdiplus
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

The GDI font resource leaks to the GdipDrawString function.

correct it as follows ?.

Before corrects (line: 3920)
  DeleteObject(SelectObject(graphics->hdc, CreateFontIndirectW(&lfw)));

After corrects
  DeleteObject(SelectObject(graphics->hdc, gdifont));

[Bug 27938] GdipDrawString Resource leak (GDI Font object).

http://bugs.winehq.org/show_bug.cgi?id=27938

--- Comment #1 from m.matsubara  2011-07-29 12:33:49 CDT ---
The file is "dlls/gdiplus/Graphics.c" .

[Bug 27938] GdipDrawString Resource leak (GDI Font object).

http://bugs.winehq.org/show_bug.cgi?id=27938

Vincent Povirk  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |1.2.x

--- Comment #2 from Vincent Povirk  2011-07-29 13:29:45 CDT ---
I don't know if we're doing another 1.2.x release, but
d0e6aa0204a5cd6c7d2f4696019f6f87897cf478 would be the appropriate commit.

[Bug 27938] GdipDrawString Resource leak (GDI Font object).

http://bugs.winehq.org/show_bug.cgi?id=27938

Wylda  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #3 from Wylda  2011-07-29 13:54:27 CDT ---

Thus fixed right, Vincent? Or invalid??

Still perplexed this one was fixed on 2011-03-09, but the bug was opened
2011-07-29? I would say invalid, but AJ would beat me again like day or two
before ;)

[Bug 27938] GdipDrawString Resource leak (GDI Font object).

http://bugs.winehq.org/show_bug.cgi?id=27938

--- Comment #4 from Vincent Povirk  2011-07-29 14:35:54 CDT ---
Note the reported version. It was apparently never fixed in a stable release.

[Bug 27938] GdipDrawString Resource leak (GDI Font object).

http://bugs.winehq.org/show_bug.cgi?id=27938

--- Comment #5 from Wylda  2011-07-29 14:44:27 CDT ---
> Note the reported version. It was apparently never fixed in a stable release.

But that's not how things works. This one should be FIXED/INVALID. For stable
there is "Target Milestone:" which says something like "When/If the time comes
backport this one".

If there won't be 1.2 stable anymore, then this one would be UNCONFIRMED
forever? I don't think so.

[Bug 27938] GdipDrawString Resource leak (GDI Font object).

http://bugs.winehq.org/show_bug.cgi?id=27938

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
           Severity|normal                      |minor

--- Comment #6 from Austin English  2011-08-01 13:56:33 CDT ---
It's fixed.

[Bug 27938] GdipDrawString Resource leak (GDI Font object).

http://bugs.winehq.org/show_bug.cgi?id=27938

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #7 from Alexandre Julliard  2011-08-05 12:39:15 CDT ---
Closing bugs fixed in 1.3.26.