summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoger Dingledine <arma@torproject.org>2016-03-24 19:05:36 -0400
committerRoger Dingledine <arma@torproject.org>2016-03-24 19:14:32 -0400
commit98abd49f6f688d1bc35bfa4329dc7d649463f8bf (patch)
tree9dc56dcd4b71826e8db6d24adc13b2553fe3d201
parentfbd79f38c232aa6b739a81480830eebb5e49af4e (diff)
downloadtor-98abd49f6f688d1bc35bfa4329dc7d649463f8bf.tar.gz
tor-98abd49f6f688d1bc35bfa4329dc7d649463f8bf.zip
remove the extraneous dir_port variable
we already are using "port" to describe the place we're going to ask to connect to.
-rw-r--r--src/or/directory.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/src/or/directory.c b/src/or/directory.c
index 2c10c03cf7..023d3f6ace 100644
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -1079,8 +1079,6 @@ directory_initiate_command_rend(const tor_addr_port_t *or_addr_port,
tor_addr_t addr;
tor_addr_copy(&addr, &(use_begindir ? or_addr_port : dir_addr_port)->addr);
uint16_t port = (use_begindir ? or_addr_port : dir_addr_port)->port;
- /* XXX dir_port is redundant and we can get rid of it. Just use port. */
- uint16_t dir_port = dir_addr_port->port;
log_debug(LD_DIR, "anonymized %d, use_begindir %d.",
anonymized_connection, use_begindir);
@@ -1150,11 +1148,11 @@ directory_initiate_command_rend(const tor_addr_port_t *or_addr_port,
if (options->HTTPProxy) {
tor_addr_copy(&addr, &options->HTTPProxyAddr);
- dir_port = options->HTTPProxyPort;
+ port = options->HTTPProxyPort;
}
switch (connection_connect(TO_CONN(conn), conn->base_.address, &addr,
- dir_port, &socket_error)) {
+ port, &socket_error)) {
case -1:
connection_mark_for_close(TO_CONN(conn));
return;