[Bug 36285] New: valgrind shows several leaks in kernel32/tests/debugger.c

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

Bug ID: 36285
           Summary: valgrind shows several leaks in
                    kernel32/tests/debugger.c
           Product: Wine
           Version: 1.7.18
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: normal
          Priority: P2
         Component: kernel32
          Assignee: [e-mail]
          Reporter: [e-mail]

==28825== 99 bytes in 1 blocks are definitely lost in loss record 135 of 229
==28825==    at 0x7BC4C735: notify_alloc (heap.c:255)
==28825==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==28825==    by 0x4B4AB31: doChildren (debugger.c:703)
==28825==    by 0x4B4B71F: func_debugger (debugger.c:840)
==28825==    by 0x4C087A8: run_test (test.h:584)
==28825==    by 0x4C08B97: main (test.h:654)
==28825==

[Bug 36285] valgrind shows several leaks in kernel32/tests/debugger.c

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

--- Comment #1 from Austin English  ---
Also, one case of:
==17743== 12 bytes in 1 blocks are definitely lost in loss record 18 of 284
==17743==    at 0x7BC4C735: notify_alloc (heap.c:255)
==17743==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==17743==    by 0x7BC8E7FF: RtlCreateUserThread (thread.c:517)
==17743==    by 0x7B87CFF0: CreateRemoteThread (thread.c:88)
==17743==    by 0x7B87CF4D: CreateThread (thread.c:54)
==17743==    by 0x4CF59D3: test_thread_actctx (thread.c:1587)
==17743==    by 0x4CF6401: func_thread (thread.c:1772)
==17743==    by 0x4D147AC: run_test (test.h:584)
==17743==    by 0x4D14B9B: main (test.h:654)
==17743== 
{

[Bug 36285] valgrind shows several leaks in kernel32/tests/debugger.c

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

--- Comment #2 from Austin English  ---
Another:
==16199== 276 bytes in 1 blocks are definitely lost in loss record 78 of 94
==16199==    at 0x7BC4C6DF: notify_alloc (heap.c:255)
==16199==    by 0x7BC50F23: RtlAllocateHeap (heap.c:1716)
==16199==    by 0x4E8E6A5: elf_locate_debug_link (elf_module.c:873)
==16199==    by 0x4E8F033: elf_check_alternate (elf_module.c:1045)
==16199==    by 0x4E8F146: elf_load_debug_info_from_map (elf_module.c:1095)
==16199==    by 0x4E8F4BA: elf_load_debug_info (elf_module.c:1159)
==16199==    by 0x4E94969: module_get_debug (module.c:350)
==16199==    by 0x4E8B73F: dwarf2_virtual_unwind (dwarf.c:3194)
==16199==    by 0x4E7B209: fetch_next_frame32 (cpu_i386.c:103)
==16199==    by 0x4E7CCF1: i386_stack_walk (cpu_i386.c:481)
==16199==    by 0x4EACAFA: StackWalk64 (stack.c:240)
==16199==    by 0x49DC32D: stack_fetch_frames (stack.c:212)
==16199==    by 0x49DF0A5: dbg_exception_prolog (tgt_active.c:137)
==16199==    by 0x49DF71C: dbg_handle_exception (tgt_active.c:274)
==16199==    by 0x49DFA8F: dbg_handle_debug_event (tgt_active.c:354)
==16199==    by 0x49E0882: wait_exception (tgt_active.c:556)
==16199==    by 0x49E09D7: dbg_active_wait_for_first_exception
(tgt_active.c:587)
==16199==    by 0x49E130C: dbg_active_auto (tgt_active.c:835)
==16199==    by 0x49E786F: main (winedbg.c:673)
==16199==

[Bug 36285] valgrind shows several leaks in kernel32/tests/debugger.c

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |valgrind

[Bug 36285] valgrind shows several leaks in kernel32/tests/debugger.c

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

Sebastian Lackner  changed:

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

--- Comment #3 from Sebastian Lackner  ---
Created attachment 48853
  --> https://bugs.winehq.org/attachment.cgi?id=48853
kernel32/tests: Add missing HeapFree() in doChildren().

This should help for the first one.

[Bug 36285] valgrind shows several leaks in kernel32/tests/debugger.c

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

--- Comment #4 from Austin English  ---
(In reply to Sebastian Lackner from comment #3)
> Created attachment 48853 [details]
> kernel32/tests: Add missing HeapFree() in doChildren().
>
> This should help for the first one.

Yep, thanks. I'll file new bugs for the other issues.

[Bug 36285] valgrind shows several leaks in kernel32/tests/debugger.c

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |e5e4ef07b6e2a54000bb8572fb8
                   |                            |bf69953cdfaea
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Austin English  ---
(In reply to Austin English from comment #4)
> (In reply to Sebastian Lackner from comment #3)
> > Created attachment 48853 [details]
> > kernel32/tests: Add missing HeapFree() in doChildren().
> >
> > This should help for the first one.
>
> Yep, thanks. I'll file new bugs for the other issues.

https://source.winehq.org/git/wine.git/commitdiff/e5e4ef07b6e2a54000bb8572fb...

[Bug 36285] valgrind shows several leaks in kernel32/tests/debugger.c

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

Alexandre Julliard  changed:

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

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.21.