Do not erase the announcement port by assigning an accouncement IP address.

With 11(!) assignment operators (not even counting the implicit ones),
no wonder IPAddress assignment semantics gets lost on developers.

diff --git a/src/send-announce.cc b/src/send-announce.cc
index 2914fbf5a8e0609baa8b62ebd11f919c2e087847..4d94c3b29691747813b8620d1923ccbc6b80aa7f 100644 (file)
--- a/src/send-announce.cc
+++ b/src/send-announce.cc
@@ -109,8 +109,8 @@ send_announce(const ipcache_addrs * ia, void *junk)
         }
     }
 
-    S.SetPort(port);
     S = ia->in_addrs[0];
+    S.SetPort(port);
     assert(theOutIcpConnection > 0);
     x = comm_udp_sendto(theOutIcpConnection, S, sndbuf, strlen(sndbuf) + 1);