[Bug 40106] New: ntdll:info regression
https://bugs.winehq.org/show_bug.cgi?id=40106 Bug ID: 40106 Summary: ntdll:info regression Product: Wine Version: 1.9.3 Hardware: x86 OS: Windows Status: NEW Severity: normal Priority: P2 Component: ntdll Assignee: wine-bugs(a)winehq.org Reporter: fgouget(a)codeweavers.com Since the commit below, the w2008s64 TestBot VM has a systematic failure in the 32 bit tests: Failure: info.c:524: Test failed: Expected to find event handle in handle list ntdll:info results on test.winehq.org: https://test.winehq.org/data/tests/ntdll:info.html commit e4e906c871ffe3e4acc055d5158a8b971f212aea Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Mon Dec 21 12:51:49 2015 +0100 ntdll/tests: Add more tests for SystemHandleInformation. Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> -- 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=40106 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download, source Regression SHA1| |e4e906c871ffe3e4acc055d5158 | |a8b971f212aea -- 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=40106 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |testcase -- 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=40106 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- URL| |https://test.winehq.org/dat | |a/tests/ntdll:info.html -- 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=40106 Sebastian Lackner <sebastian(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sebastian(a)fds-team.de -- 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=40106 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |regression -- 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=40106 --- Comment #1 from Dmitry Timoshkov <dmitry(a)baikal.ru> --- (In reply to François Gouget from comment #0)
Since the commit below, the w2008s64 TestBot VM has a systematic failure in the 32 bit tests:
Failure: info.c:524: Test failed: Expected to find event handle in handle list
I can't reproduce the failure: https://testbot.winehq.org/JobDetails.pl?Key=20273 -- 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=40106 --- Comment #2 from François Gouget <fgouget(a)codeweavers.com> --- Same here (damn TestBot has a short memory): https://testbot.winehq.org/JobDetails.pl?Key=20808 Yet the failure is very clearly still present: https://test.winehq.org/data/tests/ntdll:info.html Since it seems to only happen in WineTest runs it means it's likely a bad interaction with one of the previous tests. That does make it harder to figure out. -- 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=40106 Huw Davies <huw(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |huw(a)codeweavers.com --- Comment #3 from Huw Davies <huw(a)codeweavers.com> --- It turns out Vista and 2008 only initialize the first handle entry, so if the event handle isn't first in the list the test will fail. Commit 92fe612acb65b81b0038ef0c0555356cfcba7f41 skips the test if this broken behaviour is detected. -- 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=40106 Sebastian Lackner <sebastian(a)fds-team.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Fixed by SHA1| |92fe612acb65b81b0038ef0c055 | |5356cfcba7f41 Resolution|--- |FIXED --- Comment #4 from Sebastian Lackner <sebastian(a)fds-team.de> --- (In reply to Huw Davies from comment #3)
It turns out Vista and 2008 only initialize the first handle entry, so if the event handle isn't first in the list the test will fail.
Commit 92fe612acb65b81b0038ef0c0555356cfcba7f41 skips the test if this broken behaviour is detected.
Thanks for your work. As mentioned on the mailing list, I thought I had already excluded the possibility of a (partially) uninitialized buffer. According to https://test.winehq.org/data/tests/ntdll:info.html the systematic failure is gone, so I'm going to mark this bug as fixed. -- 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=40106 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #5 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 1.9.18. -- 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