[tor-commits] [tor/master] Look at the right errno when sending reason for connect() failure

nickm at torproject.org nickm at torproject.org
Thu Apr 7 16:19:39 UTC 2011


commit 6a5b94de6c51478ca638ec03c40a6d3c27f2d674
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Mar 25 16:14:42 2011 -0400

    Look at the right errno when sending reason for connect() failure
    
    In afe414 (tor-0.1.0.1-rc~173), when we moved to
    connection_edge_end_errno(), we used it in handling errors from
    connection_connect().  That's not so good, since by the time
    connection_connect() returns, the socket is no longer set, and we're
    supposed to be looking at the socket_errno return value from
    connection_connect() instead.  So do what we should've done, and
    look at the socket_errno value that we get from connection_connect().
---
 changes/connect_err_reporting |    6 ++++++
 src/or/connection_edge.c      |    8 ++++----
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/changes/connect_err_reporting b/changes/connect_err_reporting
new file mode 100644
index 0000000..61a46b6
--- /dev/null
+++ b/changes/connect_err_reporting
@@ -0,0 +1,6 @@
+  o Minor bugfixes:
+    - Be more careful about reporting the correct error from a failed
+      connect() operation.  Under some circumstances, it was possible to
+      look at an incorrect value for errno when sending the end reason.
+      Bugfix on Tor-0.1.0.1-rc.
+
diff --git a/src/or/connection_edge.c b/src/or/connection_edge.c
index 82b7139..c75c06b 100644
--- a/src/or/connection_edge.c
+++ b/src/or/connection_edge.c
@@ -2823,13 +2823,13 @@ connection_exit_connect(edge_connection_t *edge_conn)
 
   log_debug(LD_EXIT,"about to try connecting");
   switch (connection_connect(conn, conn->address, addr, port, &socket_error)) {
-    case -1:
-      /* XXX022 use socket_error below rather than trying to piece things
-       * together from the current errno, which may have been clobbered. */
-      connection_edge_end_errno(edge_conn);
+    case -1: {
+      int reason = errno_to_stream_end_reason(socket_error);
+      connection_edge_end(edge_conn, reason);
       circuit_detach_stream(circuit_get_by_edge_conn(edge_conn), edge_conn);
       connection_free(conn);
       return;
+    }
     case 0:
       conn->state = EXIT_CONN_STATE_CONNECTING;
 





More information about the tor-commits mailing list