[Bug 20551] New: user32 clipboard memory leaks

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

Summary: user32 clipboard memory leaks
           Product: Wine
           Version: 1.1.32
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Keywords: download
          Severity: normal
          Priority: P2
         Component: user32
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Valgrind reports a definite leak in chromium's test suite.
To reproduce:
  mkdir demo
  cd demo
  wget -c http://kegel.com/wine/chromium/chromium-tests.tar.bz2
  tar -xjvf chromium-tests.tar.bz2
  valgrind --trace-children=yes --leak-check=full wine
src/chrome/Debug/app_unittests.exe --gtest_filter=ClipboardTest.WriteEverything

The warning is

16 bytes in 1 blocks are definitely lost in loss record 186 of 1,084
   by RtlAllocateHeap (heap.c:1421)
   by HeapAlloc (heap.c:276)
   by GlobalAlloc (heap.c:395)
   by `anonymous namespace'::CreateGlobalData<wchar_t> (clipboard_win.cc:118)
   by Clipboard::WriteText (clipboard_win.cc:177)
   by Clipboard::DispatchObject (clipboard.cc:25)
   by Clipboard::WriteObjects (clipboard_win.cc:170)
   by Clipboard::WriteObjects (clipboard_win.cc:152)
   by ScopedClipboardWriter::~ScopedClipboardWriter
(scoped_clipboard_writer.cc:21)
   by ClipboardTest_WriteEverything_Test::TestBody (clipboard_unittest.cc:352)

and the attached +relay log shows that the data is passed to
SetClipboardData(),
and CloseClipboard() is called.  According to MSDN, that should free
the memory passed to SetClipboardData().

(This seems to be over and above the leaks shown in
http://kegel.com/wine/valgrind/logs/2009-10-28-08.35/vg-user32_clipboard.txt )

[Bug 20551] user32 clipboard memory leaks

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

--- Comment #1 from Dan Kegel   2009-11-01 15:29:05 ---
Created an attachment (id=24512)
 --> (http://bugs.winehq.org/attachment.cgi?id=24512)
+relay log of test case running, showing clipboard calls

[Bug 20551] user32 clipboard memory leaks

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

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WONTFIX

--- Comment #2 from Alexandre Julliard   2009-11-11 05:48:32 ---
CloseClipboard doesn't free the data, it keeps it around until it's replaced by
new data. That's the way it's supposed to work.

[Bug 20551] user32 clipboard memory leaks

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

Dmitry Timoshkov  changed:

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

--- Comment #3 from Dmitry Timoshkov   2009-11-16 06:42:29 ---
Closing WONTFIX.