[Bug 44296] New: Qt 5 programs fail to resolve DNS SRV records
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(a)winehq.org Reporter: mikkel(a)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. [1]: https://www.mumble.info -- 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=44296 --- Comment #1 from Mikkel Krautz <mikkel(a)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. -- 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=44296 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gijsvrm(a)gmail.com --- Comment #2 from Gijs Vermeulen <gijsvrm(a)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. -- 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=44296 --- Comment #3 from Mikkel Krautz <mikkel(a)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. -- 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=44296 --- Comment #4 from Mikkel Krautz <mikkel(a)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... -- 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=44296 --- Comment #5 from Hans Leidekker <hans(a)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? -- 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=44296 --- Comment #6 from Gijs Vermeulen <gijsvrm(a)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. -- 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=44296 --- Comment #7 from Mikkel Krautz <mikkel(a)krautz.dk> --- I'll also chime in and say that the patch fixes the problem for me on my original test system. Thanks! -- 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=44296 --- Comment #8 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- The patch was committed as: https://source.winehq.org/git/wine.git/commit/dfe48bab07aee196284b2c02641d35... So this should now be 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=44296 Hans Leidekker <hans(a)meelstraat.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |dfe48bab07aee196284b2c02641 | |d353eb390f08d Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED --- Comment #9 from Hans Leidekker <hans(a)meelstraat.net> --- Fixed by dfe48bab07aee196284b2c02641d353eb390f08d. -- 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=44296 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #10 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 3.1. -- 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=44296 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |3.0.x -- 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=44296 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|3.0.x |--- --- Comment #11 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 3.0.x milestone from bugs included in 3.0.1. -- 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