[Bug 36268] New: valgrind shows several leaks in dmloader/tests/loader.c

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

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

==4220== 880 bytes in 1 blocks are definitely lost in loss record 783 of 872
==4220==    at 0x7BC4C735: notify_alloc (heap.c:255)
==4220==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==4220==    by 0x5F8C7C8: create_dmcontainer (container.c:917)
==4220==    by 0x5F8DA8F: ClassFactory_CreateInstance (dmloader_main.c:101)
==4220==    by 0x4D3AE26: CoCreateInstance (unknwn.h:226)
==4220==    by 0x49558D7: test_COM_container (loader.c:189)
==4220==    by 0x4955CB4: func_loader (loader.c:234)
==4220==    by 0x4956AB0: run_test (test.h:584)
==4220==    by 0x4956E9E: main (test.h:654)
==4220== 
==4280== 104 bytes in 1 blocks are possibly lost in loss record 566 of 872
==4280==    at 0x7BC4C735: notify_alloc (heap.c:255)
==4280==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==4280==    by 0x5FD0389: create_dmsegment (segment.c:1340)
==4280==    by 0x5FBC088: ClassFactory_CreateInstance (dmime_main.c:100)
==4280==    by 0x4D3AE26: CoCreateInstance (unknwn.h:226)
==4280==    by 0x5F8FA9B: IDirectMusicLoaderImpl_GetObject (loader.c:312)
==4280==    by 0x495525E: test_simple_playing (loader.c:110)
==4280==    by 0x4955CAA: func_loader (loader.c:232)
==4280==    by 0x4956AB0: run_test (test.h:584)
==4280==    by 0x4956E9E: main (test.h:654)
==4280== 
==4327== 56 bytes in 1 blocks are possibly lost in loss record 431 of 876
==4327==    at 0x7BC4C735: notify_alloc (heap.c:255)
==4327==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==4327==    by 0xC1632EC: MMDRV_InitPerType (lolvldrv.c:384)
==4327==    by 0xC163C4D: MMDRV_Install (lolvldrv.c:476)
==4327==    by 0xC16416F: MMDRV_Init (lolvldrv.c:547)
==4327==    by 0xC161F4B: MMDRV_InitSingleType (lolvldrv.c:78)
==4327==    by 0xC161FEE: MMDRV_GetNum (lolvldrv.c:89)
==4327==    by 0xC17F7C8: midiOutGetNumDevs (winmm.c:284)
==4327==    by 0x800E004: create_system_ports_list (dmusic.c:328)
==4327==    by 0x800E5C7: DMUSIC_CreateDirectMusicImpl (dmusic.c:436)
==4327==    by 0x800EABB: ClassFactory_CreateInstance (dmusic_main.c:100)
==4327==    by 0x4D4AE26: CoCreateInstance (unknwn.h:226)
==4327==    by 0x5FD5DA9: IDirectMusicPerformance8Impl_Init (performance.c:286)
==4327==    by 0x5FD8498: IDirectMusicPerformance8Impl_InitAudio
(performance.c:930)
==4327==    by 0x4954F82: test_simple_playing (loader.c:83)
==4327==    by 0x4955CAA: func_loader (loader.c:232)
==4327==    by 0x4956AB0: run_test (test.h:584)
==4327==    by 0x4956E9E: main (test.h:654)
==4327== 
==4366== 16 bytes in 1 blocks are definitely lost in loss record 150 of 880
==4366==    at 0x7BC4C735: notify_alloc (heap.c:255)
==4366==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==4366==    by 0x5FA2379: create_dmloader (loader.c:899)
==4366==    by 0x5F9DA8F: ClassFactory_CreateInstance (dmloader_main.c:101)
==4366==    by 0x4D4AE26: CoCreateInstance (unknwn.h:226)
==4366==    by 0x4955605: test_COM (loader.c:152)
==4366==    by 0x4955CAF: func_loader (loader.c:233)
==4366==    by 0x4956AB0: run_test (test.h:584)
==4366==    by 0x4956E9E: main (test.h:654)
==4366== 
==4511== 44 bytes in 1 blocks are definitely lost in loss record 384 of 872
==4511==    at 0x7BC4C735: notify_alloc (heap.c:255)
==4511==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==4511==    by 0x5FA3B1E: create_dmaudiopath (audiopath.c:642)
==4511==    by 0x5FB09FE: IDirectMusicPerformance8Impl_CreateStandardAudioPath
(performance.c:1033)
==4511==    by 0x5FB05BF: IDirectMusicPerformance8Impl_InitAudio
(performance.c:952)
==4511==    by 0x4944F82: test_simple_playing (loader.c:83)
==4511==    by 0x4945CAA: func_loader (loader.c:232)
==4511==    by 0x4946AB0: run_test (test.h:584)
==4511==    by 0x4946E9E: main (test.h:654)
==4511== 
==4877== 44 bytes in 1 blocks are definitely lost in loss record 389 of 876
==4877==    at 0x7BC4C735: notify_alloc (heap.c:255)
==4877==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==4877==    by 0x5FCBB1E: create_dmaudiopath (audiopath.c:642)
==4877==    by 0x5FD89FE: IDirectMusicPerformance8Impl_CreateStandardAudioPath
(performance.c:1033)
==4877==    by 0x495508B: test_simple_playing (loader.c:93)
==4877==    by 0x4955CAA: func_loader (loader.c:232)
==4877==    by 0x4956AB0: run_test (test.h:584)
==4877==    by 0x4956E9E: main (test.h:654)
==4877== 
==4985== 40 bytes in 1 blocks are definitely lost in loss record 366 of 870
==4985==    at 0x7BC4C735: notify_alloc (heap.c:255)
==4985==    by 0x7BC50F79: RtlAllocateHeap (heap.c:1716)
==4985==    by 0x5F9CCE0: DMUSIC_CreateDirectMusicLoaderResourceStream
(loaderstream.c:553)
==4985==    by 0x5F9771D: IDirectMusicLoaderImpl_GetObject (loader.c:278)
==4985==    by 0x49651A7: test_simple_playing (loader.c:103)
==4985==    by 0x4965CAA: func_loader (loader.c:232)
==4985==    by 0x4966AB0: run_test (test.h:584)
==4985==    by 0x4966E9E: main (test.h:654)
==4985==

[Bug 36268] valgrind shows several leaks in dmloader/tests/loader.c

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

Austin English  changed:

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

[Bug 36268] valgrind shows several leaks in dmloader/tests/loader.c

[WineHQ Bugzilla]

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

joaopa  changed:

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

--- Comment #1 from joaopa  ---
Does the bug still occur with wine-4.19?

[Bug 36268] valgrind shows several leaks in dmloader/tests/loader.c

[WineHQ Bugzilla]

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

--- Comment #2 from Austin English  ---
(In reply to joaopa from comment #1)
> Does the bug still occur with wine-4.19?

Yes:
../../../tools/runtest -q -P wine -T ../../.. -M dmloader.dll -p
dmloader_test.exe.so loader && touch loader.ok
==21129== 4 bytes in 2 blocks are possibly lost in loss record 58 of 988
==21129==    at 0x40345DC: malloc (vg_replace_malloc.c:307)
==21129==    by 0x42F0EB1: strdup (strdup.c:42)
==21129==    by 0x80C881D: ???
==21129==    by 0x80C8A45: ???
==21129==    by 0x80C8E2C: ???
==21129==    by 0x80C8F75: ???
==21129==    by 0x80C9A00: ???
==21129==    by 0x80C9C2A: ???
==21129==    by 0x80CCEE6: ???
==21129==    by 0x80CD0D0: ???
==21129==    by 0x8151568: ???
==21129==    by 0x7C61769: midiOpenSeq (midi.c:235)
==21129==    by 0x7C61E5F: ALSA_MidiInit (midi.c:1293)
==21129==    by 0x7C64185: ALSA_midMessage (midi.c:1348)
==21129==    by 0x69CEDCC: MMDRV_InitPerType (lolvldrv.c:345)
==21129==    by 0x69CF539: MMDRV_Install (lolvldrv.c:471)
==21129==    by 0x69CF742: MMDRV_Init (lolvldrv.c:543)
==21129==    by 0x69CF8B7: MMDRV_InitSingleType (lolvldrv.c:74)
==21129==    by 0x69CF8D8: MMDRV_GetNum (lolvldrv.c:85)
==21129==    by 0x69E360C: midiOutGetNumDevs (winmm.c:291)
==21129== 
==21129== 4 bytes in 2 blocks are possibly lost in loss record 59 of 988
==21129==    at 0x40345DC: malloc (vg_replace_malloc.c:307)
==21129==    by 0x42F0EB1: strdup (strdup.c:42)
==21129==    by 0x80C881D: ???
==21129==    by 0x80C8A45: ???
==21129==    by 0x80C8E2C: ???
==21129==    by 0x80C8F75: ???
==21129==    by 0x80C88F9: ???
==21129==    by 0x80C8A45: ???
==21129==    by 0x80C8E2C: ???
==21129==    by 0x80C8F75: ???
==21129==    by 0x80C9A00: ???
==21129==    by 0x80C9C2A: ???
==21129==    by 0x80CCEE6: ???
==21129==    by 0x80CD0D0: ???
==21129==    by 0x8151568: ???
==21129==    by 0x7C61769: midiOpenSeq (midi.c:235)
==21129==    by 0x7C61E5F: ALSA_MidiInit (midi.c:1293)
==21129==    by 0x7C64185: ALSA_midMessage (midi.c:1348)
==21129==    by 0x69CEDCC: MMDRV_InitPerType (lolvldrv.c:345)
==21129==    by 0x69CF539: MMDRV_Install (lolvldrv.c:471)
==21129==

et al

wine-4.20-213-gddec23013e

[Bug 36268] valgrind shows several leaks in dmloader/tests/loader.c

[WineHQ Bugzilla]

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

Alistair Leslie-Hughes  changed:

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

--- Comment #3 from Alistair Leslie-Hughes  ---
(In reply to Austin English from comment #2)
> (In reply to joaopa from comment #1)
> > Does the bug still occur with wine-4.19?

These traces aren't very helpful in working out where the memory leaks are.