[Bug 10186] New: Scary heap warnings in ole32 tests
http://bugs.winehq.org/show_bug.cgi?id=10186 Summary: Scary heap warnings in ole32 tests Product: Wine Version: CVS/GIT Platform: Other OS/Version: other Status: NEW Severity: normal Priority: P2 Component: wine-ole AssignedTo: wine-bugs(a)winehq.org ReportedBy: dank(a)kegel.com Doing cd dlls/ole32/tests rm *.ok export WINEDEBUG=warn+heap make -k test yields a couple scary heap warnings. In particular, ../../../tools/runtest -P wine -M ole32.dll -T ../../.. -p ole32_test.exe.so hglobalstream.c && touch hglobalstream.ok warn:heap:GlobalLock invalid handle 0x125852 (Magic: 0x0098) ../../../tools/runtest -P wine -M ole32.dll -T ../../.. -p ole32_test.exe.so moniker.c warn:heap:HEAP_ValidateInUseArena Heap 0x110000: unaligned arena pointer 0x12ba7c warn:heap:HEAP_ValidateInUseArena Heap 0x110000: unaligned arena pointer 0x12bafc In tests/hglobalstream.c near line 286, there is the code hr = CreateStreamOnHGlobal(hglobal, FALSE, &pStream); ... GlobalFree(hglobal); ... hr = IStream_Read(pStream, buffer, sizeof(buffer), &read); and the IStream_Read is what triggers the invalid handle warning. The test code seems quite intentional, and the test passes on wine; is the warning spurious? In tests/moniker.c, there's a double release of a pointer at line 937, and removing one of them makes one warning go away; is this a typo, or intended? IEnumMoniker_Release(spEM1); IEnumMoniker_Release(spEM1); -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=10186 Rob Shearman <rob(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |rob(a)codeweavers.com Status|NEW |RESOLVED Resolution| |FIXED --- Comment #1 from Rob Shearman <rob(a)codeweavers.com> 2007-11-27 13:09:45 --- The first one you saw is intended. A patch for the second issue has been committed. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=10186 Dan Kegel <dank(a)kegel.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #2 from Dan Kegel <dank(a)kegel.com> 2008-01-28 05:43:16 --- Closing all RESOLVED FIXED bugs older than four weeks. -- 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=10186 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|CVS/GIT |unspecified -- 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.
https://bugs.winehq.org/show_bug.cgi?id=10186 Nikolay Sivov <bunglehead(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Component|ole |ole32 -- 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