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)

Reply via email to