[Bug 25948] New: explorerframe/nstc tests shows some a leak in comctl32 under valgrind

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

Summary: explorerframe/nstc tests shows some a leak in comctl32
                    under valgrind
           Product: Wine
           Version: 1.3.12
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: minor
          Priority: P2
         Component: comctl32
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

140 bytes in 1 blocks are definitely lost
    at  notify_alloc (heap.c:254)
    by  RtlAllocateHeap (heap.c:1701)
    by  ImageListImpl_CreateInstance (imagelist.c:3654)
    by  ImageList_Create (imagelist.c:700)
    by  initialize_checkboxes (treeview.c:4956)
    by  TREEVIEW_StyleChanged (treeview.c:5443)
    by  TREEVIEW_WindowProc (treeview.c:5782)
    by  ??? (winproc.c:172)
    by  call_window_proc (winproc.c:242)
    by  CallWindowProcW (winproc.c:980)
    by  ???
    by  ??? (winproc.c:172)
    by  call_window_proc (winproc.c:242)
    by  WINPROC_call_window (winproc.c:899)
    by  call_window_proc (message.c:2201)
    by  send_message (message.c:3042)
    by  SendMessageW (message.c:3165)
    by  WIN_SetWindowLong (win.c:2333)
    by  SetWindowLongW (win.c:2488)
    by  ???

[Bug 25948] explorerframe/nstc tests shows some a leak in comctl32 under valgrind

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

--- Comment #1 from Nikolay Sivov  2011-02-01 02:11:23 CST ---
Yeah, I'm aware of that. As I remember we don't free checkboxes image list
currently because we don't track were state image list comes from. And we can't
free always because we're not responsible of freeing user image list.

All this needs is tests.

[Bug 25948] explorerframe/nstc tests shows some a leak in comctl32 under valgrind

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

Nikolay Sivov  changed:

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

--- Comment #2 from Nikolay Sivov  2011-10-01 14:40:32 CDT ---
Ok, I have a patch for that now, will submit next week.

[Bug 25948] explorerframe/nstc tests shows some a leak in comctl32 under valgrind

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

Vijay Kamuju  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |[e-mail]
         Resolution|                            |FIXED

--- Comment #3 from Vijay Kamuju  2011-10-03 17:43:59 CDT ---
fixed in latest git by
http://source.winehq.org/git/wine.git/commit/1bc331f31f44a382a7f909d79da794d...

[Bug 25948] explorerframe/nstc tests shows some a leak in comctl32 under valgrind

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

Alexandre Julliard  changed:

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

--- Comment #4 from Alexandre Julliard  2011-10-10 13:13:42 CDT ---
Closing bugs fixed in 1.3.30.

[Bug 25948] explorerframe/nstc tests shows some a leak in comctl32 under valgrind

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

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |1bc331f31f44a382a7f909d79da
                   |                            |794d28c233fd7

[Bug 25948] explorerframe/ nstc tests shows some a leak in comctl32 under valgrind

https://bugs.winehq.org/show_bug.cgi?id=25948

Nikolay Sivov  changed:

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