From owner-svn-ports-head@freebsd.org Sat May 18 16:23:57 2019 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 140B01591C91; Sat, 18 May 2019 16:23:57 +0000 (UTC) (envelope-from arrowd@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id AB5B388BBC; Sat, 18 May 2019 16:23:56 +0000 (UTC) (envelope-from arrowd@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 84B25220EE; Sat, 18 May 2019 16:23:56 +0000 (UTC) (envelope-from arrowd@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id x4IGNuvF000466; Sat, 18 May 2019 16:23:56 GMT (envelope-from arrowd@FreeBSD.org) Received: (from arrowd@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id x4IGNu40000465; Sat, 18 May 2019 16:23:56 GMT (envelope-from arrowd@FreeBSD.org) Message-Id: <201905181623.x4IGNu40000465@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: arrowd set sender to arrowd@FreeBSD.org using -f From: Gleb Popov Date: Sat, 18 May 2019 16:23:56 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r501973 - in head/net/qt5-network: . files X-SVN-Group: ports-head X-SVN-Commit-Author: arrowd X-SVN-Commit-Paths: in head/net/qt5-network: . files X-SVN-Commit-Revision: 501973 X-SVN-Commit-Repository: ports MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Rspamd-Queue-Id: AB5B388BBC X-Spamd-Bar: -- Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-2.96 / 15.00]; local_wl_from(0.00)[FreeBSD.org]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; NEURAL_HAM_SHORT(-0.96)[-0.960,0]; ASN(0.00)[asn:11403, ipnet:2610:1c1:1::/48, country:US] X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 18 May 2019 16:23:57 -0000 Author: arrowd Date: Sat May 18 16:23:55 2019 New Revision: 501973 URL: https://svnweb.freebsd.org/changeset/ports/501973 Log: net/qt5-network: Add patch from Qt upstream fixing QDnsLookup. This should fix bug in net-im/psi, when connecting to some servers was failing. Reviewed by: tcberner (mentor) Added: head/net/qt5-network/files/patch-qdnslookup (contents, props changed) Modified: head/net/qt5-network/Makefile Modified: head/net/qt5-network/Makefile ============================================================================== --- head/net/qt5-network/Makefile Sat May 18 16:11:02 2019 (r501972) +++ head/net/qt5-network/Makefile Sat May 18 16:23:55 2019 (r501973) @@ -2,6 +2,7 @@ PORTNAME= network DISTVERSION= ${QT5_VERSION} +PORTREVISION= 1 CATEGORIES= net ipv6 PKGNAMEPREFIX= qt5- Added: head/net/qt5-network/files/patch-qdnslookup ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/net/qt5-network/files/patch-qdnslookup Sat May 18 16:23:55 2019 (r501973) @@ -0,0 +1,109 @@ +QDnsLookup: fix "Resolver functions not found" error on FreeBSD + +The current code only tries to load the required functions from +LIBRESOLV_SO (if defined) and resolv, but on FreeBSD they are in libc: +https://www.freebsd.org/cgi/man.cgi?query=res_query&sektion=3&apropos=0&manpath=freebsd + +This commit changes the code so that, after failing to load the +non-existent libraries, it attempts to load the functions with dlsym() +using the special handle RTLD_DEFAULT, which searches for the specified +symbol in the loaded libraries. + +Task-number: QTBUG-74844 + +diff --git a/src/network/kernel/kernel.pri b/src/network/kernel/kernel.pri +index 11b80d59d5e..d7a92a12ebe 100644 +--- src/network/kernel/kernel.pri ++++ src/network/kernel/kernel.pri +@@ -38,7 +38,11 @@ qtConfig(dnslookup) { + } + + unix { +- !integrity:qtConfig(dnslookup): SOURCES += kernel/qdnslookup_unix.cpp ++ !integrity:qtConfig(dnslookup) { ++ SOURCES += kernel/qdnslookup_unix.cpp ++ qtConfig(dlopen): QMAKE_USE_PRIVATE += libdl ++ } ++ + SOURCES += kernel/qhostinfo_unix.cpp + + qtConfig(linux-netlink): SOURCES += kernel/qnetworkinterface_linux.cpp +diff --git a/src/network/kernel/qdnslookup_unix.cpp b/src/network/kernel/qdnslookup_unix.cpp +index ce1ec6442a6..ee7484ab35f 100644 +--- src/network/kernel/qdnslookup_unix.cpp ++++ src/network/kernel/qdnslookup_unix.cpp +@@ -59,6 +59,10 @@ + # include + #endif + ++#if defined(Q_OS_FREEBSD) || QT_CONFIG(dlopen) ++# include ++#endif ++ + #include + + QT_BEGIN_NAMESPACE +@@ -87,6 +91,18 @@ struct QDnsLookupStateDeleter + } + }; + ++static QFunctionPointer resolveSymbol(QLibrary &lib, const char *sym) ++{ ++ if (lib.isLoaded()) ++ return lib.resolve(sym); ++ ++#if defined(RTLD_DEFAULT) && (defined(Q_OS_FREEBSD) || QT_CONFIG(dlopen)) ++ return reinterpret_cast(dlsym(RTLD_DEFAULT, sym)); ++#else ++ return nullptr; ++#endif ++} ++ + static bool resolveLibraryInternal() + { + QLibrary lib; +@@ -96,31 +112,30 @@ static bool resolveLibraryInternal() + #endif + { + lib.setFileName(QLatin1String("resolv")); +- if (!lib.load()) +- return false; ++ lib.load(); + } + +- local_dn_expand = dn_expand_proto(lib.resolve("__dn_expand")); ++ local_dn_expand = dn_expand_proto(resolveSymbol(lib, "__dn_expand")); + if (!local_dn_expand) +- local_dn_expand = dn_expand_proto(lib.resolve("dn_expand")); ++ local_dn_expand = dn_expand_proto(resolveSymbol(lib, "dn_expand")); + +- local_res_nclose = res_nclose_proto(lib.resolve("__res_nclose")); ++ local_res_nclose = res_nclose_proto(resolveSymbol(lib, "__res_nclose")); + if (!local_res_nclose) +- local_res_nclose = res_nclose_proto(lib.resolve("res_9_nclose")); ++ local_res_nclose = res_nclose_proto(resolveSymbol(lib, "res_9_nclose")); + if (!local_res_nclose) +- local_res_nclose = res_nclose_proto(lib.resolve("res_nclose")); ++ local_res_nclose = res_nclose_proto(resolveSymbol(lib, "res_nclose")); + +- local_res_ninit = res_ninit_proto(lib.resolve("__res_ninit")); ++ local_res_ninit = res_ninit_proto(resolveSymbol(lib, "__res_ninit")); + if (!local_res_ninit) +- local_res_ninit = res_ninit_proto(lib.resolve("res_9_ninit")); ++ local_res_ninit = res_ninit_proto(resolveSymbol(lib, "res_9_ninit")); + if (!local_res_ninit) +- local_res_ninit = res_ninit_proto(lib.resolve("res_ninit")); ++ local_res_ninit = res_ninit_proto(resolveSymbol(lib, "res_ninit")); + +- local_res_nquery = res_nquery_proto(lib.resolve("__res_nquery")); ++ local_res_nquery = res_nquery_proto(resolveSymbol(lib, "__res_nquery")); + if (!local_res_nquery) +- local_res_nquery = res_nquery_proto(lib.resolve("res_9_nquery")); ++ local_res_nquery = res_nquery_proto(resolveSymbol(lib, "res_9_nquery")); + if (!local_res_nquery) +- local_res_nquery = res_nquery_proto(lib.resolve("res_nquery")); ++ local_res_nquery = res_nquery_proto(resolveSymbol(lib, "res_nquery")); + + return true; + } +