[Bug 36509] New: valgrind shows a potential leak in mscoree/tests/debugging.c

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

Bug ID: 36509
           Summary: valgrind shows a potential leak in
                    mscoree/tests/debugging.c
           Product: Wine
           Version: 1.7.19
          Hardware: x86
                OS: Linux
            Status: NEW
          Keywords: download, source, testcase
          Severity: normal
          Priority: P2
         Component: mscoree
          Assignee: [e-mail]
          Reporter: [e-mail]

../../../tools/runtest -q -P wine -T ../../.. -M mscoree.dll -p
mscoree_test.exe.so debugging && touch debugging.ok
preloader: Warning: failed to reserve range 00110000-68000000
preloader: Warning: failed to reserve range 7f000000-82000000
/bin/sh: /usr/bin/egrep: No such file or directory
==9100== Find PDB file: /tmp/valgrind_petmp9100_2afae468 is empty
==9100== Warning: Missing or un-stat-able
/home/austin/.wine-valgrind/drive_c/windows/mono/mono-2.0/bin/libmono-2.0-x86.pdb
fixme:mscoree:CorDebug_Initialize stub 0x4aa1808
==9100== 52 bytes in 1 blocks are possibly lost in loss record 296 of 646
==9100==    at 0x7BC4C6DF: notify_alloc (heap.c:255)
==9100==    by 0x7BC50F23: RtlAllocateHeap (heap.c:1716)
==9100==    by 0x62F2004: ???
==9100==    by 0x62F4544: ???
==9100==    by 0x62F4F5D: ???
==9100==    by 0x62F9A9F: ???
==9100==    by 0x4CA2FF1: test_createDebugger (debugging.c:450)
==9100==    by 0x4CA3237: func_debugging (debugging.c:488)
==9100==    by 0x4CA76F6: run_test (test.h:584)
==9100==    by 0x4CA7AE5: main (test.h:654)
==9100==

[Bug 36509] valgrind shows a potential leak in mscoree/tests/debugging.c

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

Alistair Leslie-Hughes  changed:

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

[Bug 36509] valgrind shows a potential leak in mscoree/tests/debugging.c

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

Austin English  changed:

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

[Bug 36509] valgrind shows a potential leak in mscoree/tests/debugging.c

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

--- Comment #1 from Vincent Povirk  ---
Is it possible to rerun this with debug symbols for mscoree.dll included?

[Bug 36509] valgrind shows a potential leak in mscoree/tests/debugging.c

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

--- Comment #2 from Austin English  ---
(In reply to Vincent Povirk from comment #1)
> Is it possible to rerun this with debug symbols for mscoree.dll included?

They should already be enabled, I build with

CFLAGS="-g -ggdb -Og -fno-inline"
https://github.com/austin987/wine-valgrind-scripts/blob/master/valgrind-full...

looking at valgrind's output, a debug build of mono may be useful (though
valgrind's PDB support is hit or miss, not sure how it will handle mono).

Anyway, if you've got a debug mono build I could try, I will.

[Bug 36509] valgrind shows a potential leak in mscoree/tests/debugging.c

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

--- Comment #3 from Vincent Povirk  ---
There's nothing in test_createDebugger that calls into Mono. The unknown frames
must be in mscoree.

[Bug 36509] valgrind shows a potential leak in mscoree/tests/debugging.c

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

--- Comment #4 from Austin English  ---
(In reply to Vincent Povirk from comment #3)
> There's nothing in test_createDebugger that calls into Mono. The unknown
> frames must be in mscoree.

Fair; looks like a valgrind thing, since debug symbols should be there:

austin@laptop:~/wine-valgrind/dlls/mscoree$ file *so
mscoree.dll.so: ELF 32-bit LSB pie executable, Intel 80386, version 1 (SYSV),
dynamically linked, with debug_info, not stripped

[Bug 36509] valgrind shows a potential leak in mscoree/tests/debugging.c

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

--- Comment #5 from Austin English  ---
*I realize though that this will be difficult to fix without.

[Bug 36509] valgrind shows a potential leak in mscoree/tests/debugging.c

[WineHQ Bugzilla]

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

Esme Povirk  changed:

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