From: Nick Mathewson <nickm@torproject.org>
Date: Mon, 2 Jul 2018 20:11:02 +0000 (-0400)
Subject: Partially revert "Use tor_addr_from_getsockname() in several places"
X-Git-Tag: tor-0.3.4.4-rc~11^2^2
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=dfdf32404cd93671646481f6f3c129591a6901d7;p=thirdparty%2Ftor.git

Partially revert "Use tor_addr_from_getsockname() in several places"

This reverts part of commit 6ed384b827dce21ea3a44b587, in order to
fix bug 26568.  Bugfix on 0.3.4.1-alpha.
---

diff --git a/changes/bug26568 b/changes/bug26568
new file mode 100644
index 0000000000..0c4c05d043
--- /dev/null
+++ b/changes/bug26568
@@ -0,0 +1,3 @@
+  o Minor bugfixes (controller):
+    - Report the port correctly when a port is configured to bind to "auto".
+      Fixes bug 26568; bugfix on 0.3.4.1-alpha.
diff --git a/src/or/connection.c b/src/or/connection.c
index 5185b45b14..7283a81043 100644
--- a/src/or/connection.c
+++ b/src/or/connection.c
@@ -1260,12 +1260,15 @@ connection_listener_new(const struct sockaddr *listensockaddr,
       gotPort = usePort;
     } else {
       tor_addr_t addr2;
-      if (tor_addr_from_getsockname(&addr2, s)<0) {
+      struct sockaddr_storage ss;
+      socklen_t ss_len=sizeof(ss);
+      if (getsockname(s, (struct sockaddr*)&ss, &ss_len)<0) {
         log_warn(LD_NET, "getsockname() couldn't learn address for %s: %s",
                  conn_type_to_string(type),
                  tor_socket_strerror(tor_socket_errno(s)));
         gotPort = 0;
       }
+      tor_addr_from_sockaddr(&addr2, (struct sockaddr*)&ss, &gotPort);
     }
 #ifdef HAVE_SYS_UN_H
   /*