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@winehq.org Reporter: fgouget@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.
https://bugs.winehq.org/show_bug.cgi?id=40386
François Gouget fgouget@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |https://test.winehq.org/dat | |a/tests/wininet:url.html Keywords| |source, testcase
https://bugs.winehq.org/show_bug.cgi?id=40386
Bernhard Übelacker bernhardu@mailbox.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |bernhardu@mailbox.org
--- Comment #1 from Bernhard Übelacker bernhardu@mailbox.org --- Patch sent for review. https://www.winehq.org/pipermail/wine-patches/2016-August/153000.html
https://bugs.winehq.org/show_bug.cgi?id=40386
--- Comment #2 from Bernhard Übelacker bernhardu@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
https://bugs.winehq.org/show_bug.cgi?id=40386
Jacek Caban jacek@codeweavers.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |0fcecad8d84a6fbe0c16b786a9b | |639c3530cf272 Status|NEW |RESOLVED CC| |jacek@codeweavers.com
--- Comment #3 from Jacek Caban jacek@codeweavers.com --- wininet:url tests look clean now. Marking fixed.
https://bugs.winehq.org/show_bug.cgi?id=40386
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #4 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 2.0-rc2.