[Bug 26050] New: winmm/mci tests show quite a few valgrind warnings
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(a)winehq.org ReportedBy: austinenglish(a)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 -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26050 Jörg Höhle <hoehle(a)users.sourceforge.net> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |hoehle(a)users.sourceforge.ne | |t --- Comment #1 from Jörg Höhle <hoehle(a)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) -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26050 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #33222|0 |1 is obsolete| | --- Comment #2 from Austin English <austinenglish(a)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 -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26050 --- Comment #3 from Jörg Höhle <hoehle(a)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) -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26050 --- Comment #4 from Austin English <austinenglish(a)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) -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26050 --- Comment #5 from Austin English <austinenglish(a)gmail.com> 2011-05-12 15:44:40 CDT --- http://source.winehq.org/git/wine.git/commitdiff/7df286c13604e1add2f6da57492... improves things a bit more. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26050 Michael Mc Donnell <michael(a)mcdonnell.dk> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |michael(a)mcdonnell.dk -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26050 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED --- Comment #6 from Austin English <austinenglish(a)gmail.com> 2012-09-26 15:39:16 CDT --- No problems in wine-1.5.13-346-g0445393. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=26050 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #7 from Alexandre Julliard <julliard(a)winehq.org> 2012-09-28 13:43:49 CDT --- Closing bugs fixed in 1.5.14. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
participants (1)
-
wine-bugs＠winehq.org