Module Name: src Committed By: andvar Date: Mon May 29 21:16:58 UTC 2023
Modified Files: src/external/apache2/mDNSResponder/dist/mDNSPosix: PosixDaemon.c src/external/apache2/mDNSResponder/dist/mDNSShared: PlatformCommon.c Log Message: mdnsd(8): restore fixes for PR bin/46758, lost on resolving merge conflicts. Original commit message from Roy Marples: "Derive our primary interface and address by trying to connect to an address in the TEST-NET-2 network as noted in RFC5737 instead of using the 1.1.1.1 address. Also, use port 7 (echo) for better style. Fixes PR bin/46758 thanks to Lloyd Parkes." pullups needed for netbsd-9, netbsd-10. To generate a diff of this commit: cvs rdiff -u -r1.15 -r1.16 \ src/external/apache2/mDNSResponder/dist/mDNSPosix/PosixDaemon.c cvs rdiff -u -r1.6 -r1.7 \ src/external/apache2/mDNSResponder/dist/mDNSShared/PlatformCommon.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/external/apache2/mDNSResponder/dist/mDNSPosix/PosixDaemon.c diff -u src/external/apache2/mDNSResponder/dist/mDNSPosix/PosixDaemon.c:1.15 src/external/apache2/mDNSResponder/dist/mDNSPosix/PosixDaemon.c:1.16 --- src/external/apache2/mDNSResponder/dist/mDNSPosix/PosixDaemon.c:1.15 Sat May 27 17:58:58 2023 +++ src/external/apache2/mDNSResponder/dist/mDNSPosix/PosixDaemon.c Mon May 29 21:16:58 2023 @@ -92,7 +92,7 @@ mDNSlocal void mDNS_StatusCallback(mDNS static void Reconfigure(mDNS *m) { mDNSAddr DynDNSIP; - const mDNSAddr dummy = { mDNSAddrType_IPv4, { { { 1, 1, 1, 1 } } } };; + const mDNSAddr dummy = { mDNSAddrType_IPv4, { { { 198, 51, 100, 42 } } } };; mDNS_SetPrimaryInterfaceInfo(m, NULL, NULL, NULL); mDNS_Lock(m); if (ParseDNSServers(m, uDNS_SERVERS_FILE) < 0) Index: src/external/apache2/mDNSResponder/dist/mDNSShared/PlatformCommon.c diff -u src/external/apache2/mDNSResponder/dist/mDNSShared/PlatformCommon.c:1.6 src/external/apache2/mDNSResponder/dist/mDNSShared/PlatformCommon.c:1.7 --- src/external/apache2/mDNSResponder/dist/mDNSShared/PlatformCommon.c:1.6 Tue Jul 21 14:04:07 2020 +++ src/external/apache2/mDNSResponder/dist/mDNSShared/PlatformCommon.c Mon May 29 21:16:58 2023 @@ -52,7 +52,7 @@ mDNSexport void mDNSPlatformSourceAddrFo addr.a4.sin_len = inner_len; #endif addr.a4.sin_family = AF_INET; - addr.a4.sin_port = 1; // Not important, any port will do + addr.a4.sin_port = 7; // Not important, any port will do addr.a4.sin_addr.s_addr = dst->ip.v4.NotAnInteger; } else if (dst->type == mDNSAddrType_IPv6) @@ -63,7 +63,7 @@ mDNSexport void mDNSPlatformSourceAddrFo #endif addr.a6.sin6_family = AF_INET6; addr.a6.sin6_flowinfo = 0; - addr.a6.sin6_port = 1; // Not important, any port will do + addr.a6.sin6_port = 7; // Not important, any port will do addr.a6.sin6_addr = *(struct in6_addr*)&dst->ip.v6; addr.a6.sin6_scope_id = 0; } @@ -71,9 +71,17 @@ mDNSexport void mDNSPlatformSourceAddrFo if ((connect(sock, &addr.s, inner_len)) < 0) { - if (errno != EADDRNOTAVAIL) - LogMsg("mDNSPlatformSourceAddrForDest: connect %#a failed errno %d (%s)", dst, errno, strerror(errno)); - goto exit; + static mDNSv4Addr dummy = { 198, 51, 100, 42 }; + + // Don't spam if we can't connect to 198.51.100.42 to the console. + // That is our test address to out which interfaces/address should be primary and is also + // configured in mDNSPosix/PosixDaemon.c:Reconfigure() + // Failing to connect to it with EADDRNOTAVAIL is a common situation, especially on boot up. + if (dst->type == mDNSAddrType_IPv4 && dst->ip.v4.NotAnInteger == dummy.NotAnInteger && errno == EADDRNOTAVAIL) + LogInfo("mDNSPlatformSourceAddrForDest: connect %#a failed errno %d (%s)", dst, errno, strerror(errno)); + else + LogMsg("mDNSPlatformSourceAddrForDest: connect %#a failed errno %d (%s)", dst, errno, strerror(errno)); + goto exit; } if ((getsockname(sock, &addr.s, &len)) < 0)