diff options
| author | Nick Mathewson <nickm@torproject.org> | 2013-05-24 12:23:21 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2013-05-24 12:23:21 -0400 |
| commit | b4b0063e48a2687e7971730defc9b5a99a396cc4 (patch) | |
| tree | af3475affb2c5e80108a62673b4a24a664fbe9c0 | |
| parent | d25e77f2c330d855cf1e3b8fa44387e2500a3ac4 (diff) | |
| download | tor-b4b0063e48a2687e7971730defc9b5a99a396cc4.tar tor-b4b0063e48a2687e7971730defc9b5a99a396cc4.tar.gz | |
Tweak fix for #8789 a bit; avoid double-close and add changes file
| -rw-r--r-- | changes/bug8789 | 3 | ||||
| -rw-r--r-- | src/or/connection.c | 7 |
2 files changed, 8 insertions, 2 deletions
diff --git a/changes/bug8789 b/changes/bug8789 new file mode 100644 index 000000000..e3bcc4074 --- /dev/null +++ b/changes/bug8789 @@ -0,0 +1,3 @@ + o Code simplifications and refactoring: + - Clean up exit path from connection_listener_new. Closes bug + 8789. Patch from Arlo Breault. diff --git a/src/or/connection.c b/src/or/connection.c index 82d03106c..aa3cc330d 100644 --- a/src/or/connection.c +++ b/src/or/connection.c @@ -940,7 +940,7 @@ connection_listener_new(const struct sockaddr *listensockaddr, const port_cfg_t *port_cfg) { listener_connection_t *lis_conn; - connection_t *conn; + connection_t *conn = NULL; tor_socket_t s = TOR_INVALID_SOCKET; /* the socket we're going to make */ or_options_t const *options = get_options(); #if defined(HAVE_PWD_H) && defined(HAVE_SYS_UN_H) @@ -1106,6 +1106,7 @@ connection_listener_new(const struct sockaddr *listensockaddr, conn = TO_CONN(lis_conn); conn->socket_family = listensockaddr->sa_family; conn->s = s; + s = TOR_INVALID_SOCKET; /* Prevent double-close */ conn->address = tor_strdup(address); conn->port = gotPort; tor_addr_copy(&conn->addr, &addr); @@ -1141,7 +1142,6 @@ connection_listener_new(const struct sockaddr *listensockaddr, if (connection_add(conn) < 0) { /* no space, forget it */ log_warn(LD_NET,"connection_add for listener failed. Giving up."); - connection_free(conn); goto err; } @@ -1162,6 +1162,9 @@ connection_listener_new(const struct sockaddr *listensockaddr, err: if (SOCKET_OK(s)) tor_close_socket(s); + if (conn) + connection_free(conn); + return NULL; } |