https://bugs.winehq.org/show_bug.cgi?id=44296
Bug ID: 44296 Summary: Qt 5 programs fail to resolve DNS SRV records Product: Wine Version: 1.8.5 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: dnsapi Assignee: wine-bugs@winehq.org Reporter: mikkel@krautz.dk Distribution: ---
Created attachment 60129 --> https://bugs.winehq.org/attachment.cgi?id=60129 ZIP file containing reproducible test case.
One of our test cases from Mumble [1] tests our SRV resolving code.
The test passes on Windows 10 (all I've tested), but fails on Wine (wine-1.8.5 (Ubuntu 1.8.5-1ubuntu2)).
On Wine, the test fails due to Qt's QDnsLookup failing with QDnsLookup::ServerFailureError.
The code in Qt 5 is here: https://github.com/qt/qtbase/blob/19b0ce5daa31e2ffebfcf2701143742302f1deb4/s...
So it seems like a call to DnsQuery_W for resolving SRV records is failing with DNS_ERROR_RCODE_SERVER_FAILURE.
I've not dug deeper yet, I just wanted to file this upstream early.
https://bugs.winehq.org/show_bug.cgi?id=44296
--- Comment #1 from Mikkel Krautz mikkel@krautz.dk --- Just to document the attached reproducible test case:
When running the test binary (TestServerResolver.exe) on Wine, the test-case should fail:
FAIL! : TestServerResolver::srvCustomPort() Compared values are not the same
On Windows, all tests should pass.
https://bugs.winehq.org/show_bug.cgi?id=44296
Gijs Vermeulen gijsvrm@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |gijsvrm@gmail.com
--- Comment #2 from Gijs Vermeulen gijsvrm@gmail.com --- Created attachment 60130 --> https://bugs.winehq.org/attachment.cgi?id=60130 output
For me, on macOS 10.12.6 running Wine-3.0-rc4, both TestServerResolver::simpleSrv() and TestServerResolver::srvCustomPort() fail. Output is attached. Maybe try with 3.0-rc4 yourself. Providing the full source for the test exe would probably be a good idea as well.
https://bugs.winehq.org/show_bug.cgi?id=44296
--- Comment #3 from Mikkel Krautz mikkel@krautz.dk --- The test can be checked out and built via:
$ git clone --recursive https://github.com/mumble-voip/mumble.git $ cd mumble $ git checkout 3ffd9ad3ed18176774d8e1c64a96dffe0de69655 # pin to a commit where we know the test fails.... $ cd src/tests/TestServerResolver $ qmake CONFIG+=release TestServerResolver.pro $ make
for example via MXE.
https://bugs.winehq.org/show_bug.cgi?id=44296
--- Comment #4 from Mikkel Krautz mikkel@krautz.dk --- The source of the test can also be viewed at:
https://github.com/mumble-voip/mumble/blob/3ffd9ad3ed18176774d8e1c64a96dffe0...
Other potentially relevant files:
https://github.com/mumble-voip/mumble/blob/3ffd9ad3ed18176774d8e1c64a96dffe0...
https://bugs.winehq.org/show_bug.cgi?id=44296
--- Comment #5 from Hans Leidekker hans@meelstraat.net --- Created attachment 60143 --> https://bugs.winehq.org/attachment.cgi?id=60143 patch
Thanks for the nice bug report. Can you try this patch?
https://bugs.winehq.org/show_bug.cgi?id=44296
--- Comment #6 from Gijs Vermeulen gijsvrm@gmail.com --- (In reply to Hans Leidekker from comment #5)
Created attachment 60143 [details] patch
Thanks for the nice bug report. Can you try this patch?
I can confirm that this patch fixes the issue on Ubuntu.
The weird thing is, on Ubuntu I only get one test failure, like described in the bugreport, while on macOS I get two. I can't test this patch there because I haven't been able to build Wine correctly on it.
https://bugs.winehq.org/show_bug.cgi?id=44296
--- Comment #7 from Mikkel Krautz mikkel@krautz.dk --- I'll also chime in and say that the patch fixes the problem for me on my original test system. Thanks!
https://bugs.winehq.org/show_bug.cgi?id=44296
--- Comment #8 from Gijs Vermeulen gijsvrm@gmail.com --- The patch was committed as: https://source.winehq.org/git/wine.git/commit/dfe48bab07aee196284b2c02641d35... So this should now be fixed.
https://bugs.winehq.org/show_bug.cgi?id=44296
Hans Leidekker hans@meelstraat.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |dfe48bab07aee196284b2c02641 | |d353eb390f08d Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED
--- Comment #9 from Hans Leidekker hans@meelstraat.net --- Fixed by dfe48bab07aee196284b2c02641d353eb390f08d.
https://bugs.winehq.org/show_bug.cgi?id=44296
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #10 from Alexandre Julliard julliard@winehq.org --- Closing bugs fixed in 3.1.
https://bugs.winehq.org/show_bug.cgi?id=44296
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x
https://bugs.winehq.org/show_bug.cgi?id=44296
Michael Stefaniuc mstefani@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |---
--- Comment #11 from Michael Stefaniuc mstefani@winehq.org --- Removing the 3.0.x milestone from bugs included in 3.0.1.
-
wine-bugs＠winehq.org