http://bugs.winehq.org/show_bug.cgi?id=26050
Summary: winmm/mci tests show quite a few valgrind warnings Product: Wine Version: 1.3.13 Platform: x86 OS/Version: Linux Status: NEW Keywords: download, source, testcase Severity: minor Priority: P2 Component: winmm&mci AssignedTo: wine-bugs@winehq.org ReportedBy: austinenglish@gmail.com
Created an attachment (id=33222) --> (http://bugs.winehq.org/attachment.cgi?id=33222) valgrind log
Lots of: Use of uninitialised value of size 4 at wcstombs_sbcs (wctomb.c:155) by wine_cp_wcstombs (wctomb.c:456) by WideCharToMultiByte (locale.c:2010) by MCI_UnmapMsgAtoW (mci.c:466) by mciSendCommandA (mci.c:2285) by test_openCloseWAVE (mci.c:419) by func_mci (mci.c:1266) by run_test (test.h:556) by main (test.h:624) Uninitialised value was created by a client request at mark_block_uninitialized (heap.c:208) by initialize_block (heap.c:239) by RtlAllocateHeap (heap.c:1702) by MCI_MapMsgAtoW (mci.c:340) by mciSendCommandA (mci.c:2277) by test_openCloseWAVE (mci.c:419) by func_mci (mci.c:1266) by run_test (test.h:556) by main (test.h:624)
and a few like: 20 bytes in 1 blocks are definitely lost at notify_alloc (heap.c:254) by RtlAllocateHeap (heap.c:1701) by str_dup_upper (mci.c:85) by mciSendStringW (mci.c:1341) by mciSendStringA (mci.c:1590) by test_openCloseWAVE (mci.c:560) by func_mci (mci.c:1266) by run_test (test.h:556) by main (test.h:624)
full log attached
http://bugs.winehq.org/show_bug.cgi?id=26050
Jörg Höhle hoehle@users.sourceforge.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |hoehle@users.sourceforge.ne | |t
--- Comment #1 from Jörg Höhle hoehle@users.sourceforge.net 2011-02-23 09:20:41 CST --- commit fd8576bfdbaa6de55ba5bb5833f7f92173035a62 fixes the WideCharToMultiByte issue.
Confirming the other issue, devType = str_dup_upper(devType); is not freed in error paths.
I'm surprised that you did not mention the following excerpt of your log. IIRC I mentioned somewhere last year that MMIO looses its 8KB buffer every time mmioOpen fails, such as in open no-such-file-exists.wav
8,192 bytes in 1 blocks are definitely lost at notify_alloc (heap.c:254) by RtlAllocateHeap (heap.c:1701) by MMIO_SetBuffer (mmio.c:562) by MMIO_Open (mmio.c:650) by mmioOpenW (mmio.c:705) by wodWrite (wavemap.c:338) by wodPrepare (wavemap.c:386) by ??? (in /home/austin/wine-git/dlls/msacm32.drv/msacm32.drv.so) by DRIVER_SendMessage (driver.c:136) by SendDriverMessage (driver.c:156) by MCI_SendCommandFrom32 (mci.c:892) by MCI_FinishOpen (mci.c:927) by mciSendStringW (mci.c:1540) by mciSendStringA (mci.c:1590) by test_openCloseWAVE (mci.c:517)
http://bugs.winehq.org/show_bug.cgi?id=26050
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Attachment #33222|0 |1 is obsolete| |
--- Comment #2 from Austin English austinenglish@gmail.com 2011-02-23 17:54:43 CST --- Created an attachment (id=33427) --> (http://bugs.winehq.org/attachment.cgi?id=33427) valgrind log
Updated valgrind log attached, this is with wine-1.3.14-82-g85a578f
http://bugs.winehq.org/show_bug.cgi?id=26050
--- Comment #3 from Jörg Höhle hoehle@users.sourceforge.net 2011-02-25 09:13:09 CST --- I've just updated the Wiki about confused Valgrind logs because of unloaded dynamic libraries.
Here's a snippet that makes perfect sense after commenting out FreeLibrary. Actually it's one I mentioned in bug #20232, comment #12:
16 bytes in 1 blocks are definitely lost at RtlAllocateHeap (heap.c:254) by WAVE_mciOpenFile (mciwave.c:289) by MCIWAVE_DriverProc (mciwave.c:1452) by SendDriverMessage (driver.c:136) by MCI_SendCommandFrom32 (mci.c:892) by mciSendStringW (mci.c:1544) by mciSendStringA (mci.c:1590) by test_recordWAVE (mci.c:746) by func_mci (mci.c:1267)
http://bugs.winehq.org/show_bug.cgi?id=26050
--- Comment #4 from Austin English austinenglish@gmail.com 2011-02-28 13:22:16 CST --- http://source.winehq.org/git/wine.git/?a=commitdiff;h=90da8b749ebe228fd75b20...
fixes:
42 bytes in 1 blocks are definitely lost in loss record 222 of 557 at 0x7BC4832B: RtlAllocateHeap (heap.c:254) by 0x4C7EA13: ??? (in /home/austin/wine-git/dlls/midimap/midimap.dll.so) by 0x4C81F0C: ??? (in /home/austin/wine-git/dlls/midimap/midimap.dll.so) by 0x47B06FE: SendDriverMessage (driver.c:136) by 0x47B5A96: MCI_SendCommandFrom32 (mci.c:892) by 0x47B5BAB: MCI_FinishOpen (mci.c:927) by 0x47B8CEC: mciSendStringW (mci.c:1540) by 0x47B94C4: mciSendStringA (mci.c:1590) by 0x476FE34: test_openCloseWAVE (mci.c:524) by 0x4774FAC: func_mci (mci.c:1266) by 0x4789B5F: run_test (test.h:556) by 0x4789D65: main (test.h:624)
48 bytes in 1 blocks are definitely lost in loss record 247 of 557 at 0x7BC4832B: RtlAllocateHeap (heap.c:254) by 0x4C7EA13: ??? (in /home/austin/wine-git/dlls/midimap/midimap.dll.so) by 0x4C81F0C: ??? (in /home/austin/wine-git/dlls/midimap/midimap.dll.so) by 0x47B06FE: SendDriverMessage (driver.c:136) by 0x47B5A96: MCI_SendCommandFrom32 (mci.c:892) by 0x47B5BAB: MCI_FinishOpen (mci.c:927) by 0x47B8CEC: mciSendStringW (mci.c:1540) by 0x47B94C4: mciSendStringA (mci.c:1590) by 0x476FDD2: test_openCloseWAVE (mci.c:517) by 0x4774FAC: func_mci (mci.c:1266) by 0x4789B5F: run_test (test.h:556) by 0x4789D65: main (test.h:624)
48 bytes in 1 blocks are definitely lost in loss record 248 of 557 at 0x7BC4832B: RtlAllocateHeap (heap.c:254) by 0x53E0A13: ??? by 0x53E3F0C: ??? by 0x47B06FE: SendDriverMessage (driver.c:136) by 0x47B5A96: MCI_SendCommandFrom32 (mci.c:892) by 0x47B5BAB: MCI_FinishOpen (mci.c:927) by 0x47B8CEC: mciSendStringW (mci.c:1540) by 0x47B8E7F: mciSendStringW (mci.c:1416) by 0x47B94C4: mciSendStringA (mci.c:1590) by 0x4773FAE: test_AutoOpenWAVE (mci.c:1122) by 0x4774FC8: func_mci (mci.c:1270) by 0x4789B5F: run_test (test.h:556) by 0x4789D65: main (test.h:624)
http://bugs.winehq.org/show_bug.cgi?id=26050
--- Comment #5 from Austin English austinenglish@gmail.com 2011-05-12 15:44:40 CDT --- http://source.winehq.org/git/wine.git/commitdiff/7df286c13604e1add2f6da57492... improves things a bit more.
http://bugs.winehq.org/show_bug.cgi?id=26050
Michael Mc Donnell michael@mcdonnell.dk changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |michael@mcdonnell.dk
http://bugs.winehq.org/show_bug.cgi?id=26050
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #6 from Austin English austinenglish@gmail.com 2012-09-26 15:39:16 CDT --- No problems in wine-1.5.13-346-g0445393.
http://bugs.winehq.org/show_bug.cgi?id=26050
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #7 from Alexandre Julliard julliard@winehq.org 2012-09-28 13:43:49 CDT --- Closing bugs fixed in 1.5.14.
-
wine-bugs＠winehq.org