[Bug 40386] New: wininet:url regression caused by new cross-compiler
https://bugs.winehq.org/show_bug.cgi?id=40386 Bug ID: 40386 Summary: wininet:url regression caused by new cross-compiler Product: Wine Version: 1.9.6 Hardware: x86 OS: Windows Status: NEW Severity: normal Priority: P2 Component: wininet Assignee: wine-bugs(a)winehq.org Reporter: fgouget(a)codeweavers.com The WineHQ upgrade changed the cross-compiler used to generate the reference WineTest executables. Since the upgrade the wininet:url 32 bit test has these failures on Windows XP + Internet Explorer 6: url.c:507: Test failed: InternetCrackUrl returned 0 with GLE=87 (expected to return 0 and ERROR_INVALID_HANDLE or ERROR_INSUFFICIENT_BUFFER) url.c:521: Test failed: InternetCrackUrl failed with 87 url.c:522: Test failed: lpszScheme was "" instead of "about" url.c:523: Test failed: lpszHostName was "" instead of "host" url.c:524: Test failed: lpszUrlPath was "`ù"" instead of "/blank" See: https://test.winehq.org/data/tests/wininet:url.html https://www.winehq.org/pipermail/wine-devel/2016-March/112371.html I can reproduce the problem with this compiler: $ i686-w64-mingw32-gcc --version i686-w64-mingw32-gcc (GCC) 5.3.1 20160205 But if I recompile the tests with -O0 the failures go away which points to a compiler bug. We need to either: * Switch to a non buggy compiler to build the reference WineTest executables. (which might involve reporting the bug in the first place) * Add 'CFLAGS = -g -O0' to the test's Makefile.in as a workaround. (preferably once we can point to a relevant compiler bug) * Find some other nicer workaround in the C file. -- 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=40386 François Gouget <fgouget(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- URL| |https://test.winehq.org/dat | |a/tests/wininet:url.html 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=40386 Bernhard Übelacker <bernhardu(a)mailbox.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bernhardu(a)mailbox.org --- Comment #1 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- Patch sent for review. https://www.winehq.org/pipermail/wine-patches/2016-August/153000.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=40386 --- Comment #2 from Bernhard Übelacker <bernhardu(a)mailbox.org> --- Patch got committed: http://source.winehq.org/git/wine.git/commit/0fcecad8d84a6fbe0c16b786a9b639c... First two test runs after commit look good to me: https://test.winehq.org/data/tests/wininet:url.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=40386 Jacek Caban <jacek(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |0fcecad8d84a6fbe0c16b786a9b | |639c3530cf272 Status|NEW |RESOLVED CC| |jacek(a)codeweavers.com --- Comment #3 from Jacek Caban <jacek(a)codeweavers.com> --- wininet:url tests look clean now. Marking 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=40386 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #4 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 2.0-rc2. -- 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