[Bug 54866] New: ieframe:webbrowser - test_SetQueryNetSessionCount() sometimes gets an unexpected session count on Windows
https://bugs.winehq.org/show_bug.cgi?id=54866 Bug ID: 54866 Summary: ieframe:webbrowser - test_SetQueryNetSessionCount() sometimes gets an unexpected session count on Windows Product: Wine Version: unspecified Hardware: x86-64 OS: Windows Status: NEW Severity: normal Priority: P2 Component: ieframe Assignee: wine-bugs(a)winehq.org Reporter: fgouget(a)codeweavers.com ieframe:webbrowser - test_SetQueryNetSessionCount() sometimes gets an unexpected session count on Windows: webbrowser.c:4440: init_count 1 webbrowser.c:4443: Test failed: count = 3 webbrowser.c:4446: Test failed: count = 3 webbrowser.c:4449: Test failed: count = 2 webbrowser.c:4452: Test failed: count = 2 See https://test.winehq.org/data/patterns.html#ieframe:webbrowser These failures only happen on Windows 10 21H1 to 22H2. On Windows 10 init_count is either 1 or 2 (on w8adm is can be 0 too), but all the failures so far happened with an init_count of 1. That said there are also a lot of successful runs with init_count == 1. This looks a lot like the browser creates a net session for its own purposes: * Either init_count is 1 and the extra net session is never created or gets created after test_SetQueryNetSessionCount() is done so all is fine. * Or the extra net session is created it before test_SetQueryNetSessionCount() starts and outlives it so our tests pass. * Or the extra net session gets created after we get init_count but before the next test so that all our tests are off by one. * But it seems the extra net session never gets created after the first SESSION_INCREMENT test. Note that on my Windows 10 VM when I start iexplorer.exe by hand and then run the test I always get init_count == 2. But if IE is not already running I get init_count == 0. In any case if it's a race condition I could send a patch to retry a couple of times. -- 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=54866 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |source, 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=54866 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Status|NEW |RESOLVED --- Comment #1 from François Gouget <fgouget(a)codeweavers.com> --- Eventhough it does not mention it, this bug is supposed to be fixed by the commit below (see also MR!3577): commit 3f276c836b43a720256559870b059399fb3f4663 Author: Jacek Caban <jacek(a)codeweavers.com> Date: Wed Aug 16 10:15:55 2023 +0200 ieframe/tests: Use less strict checks in session count tests. -- 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=54866 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #2 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 8.14. -- 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)
-
WineHQ Bugzilla