aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNick Mathewson <nickm@torproject.org>2017-10-24 19:43:13 -0400
committerNick Mathewson <nickm@torproject.org>2017-10-24 19:43:13 -0400
commite5c341eb7c1189985d903f708ce91516da7f0c76 (patch)
tree7d7518037709815f7a3260bcf4c72a90fd6b7047
parent3498183807fa626289aab98ed8b2477176a70190 (diff)
parente7126a1030233fd656cff4a8ee998026065fb65b (diff)
downloadtor-e5c341eb7c1189985d903f708ce91516da7f0c76.tar.gz
tor-e5c341eb7c1189985d903f708ce91516da7f0c76.zip
Merge branch 'maint-0.3.2'
-rw-r--r--changes/bug205324
-rw-r--r--src/or/bridges.c8
2 files changed, 12 insertions, 0 deletions
diff --git a/changes/bug20532 b/changes/bug20532
new file mode 100644
index 0000000000..7c190ea032
--- /dev/null
+++ b/changes/bug20532
@@ -0,0 +1,4 @@
+ o Minor bugfixes (bridges):
+ - Overwrite the bridge address earlier in the process of directly
+ retrieving its descriptor, to make sure we reach it on the configured
+ address. Fixes bug 20532; bugfix on 0.2.0.10-alpha.
diff --git a/src/or/bridges.c b/src/or/bridges.c
index 1c748b4156..320f5ee632 100644
--- a/src/or/bridges.c
+++ b/src/or/bridges.c
@@ -54,6 +54,8 @@ struct bridge_info_t {
};
static void bridge_free(bridge_info_t *bridge);
+static void rewrite_node_address_for_bridge(const bridge_info_t *bridge,
+ node_t *node);
/** A list of configured bridges. Whenever we actually get a descriptor
* for one, we add it as an entry guard. Note that the order of bridges
@@ -574,6 +576,12 @@ launch_direct_bridge_descriptor_fetch(bridge_info_t *bridge)
return;
}
+ /* If we already have a node_t for this bridge, rewrite its address now. */
+ node_t *node = node_get_mutable_by_id(bridge->identity);
+ if (node) {
+ rewrite_node_address_for_bridge(bridge, node);
+ }
+
tor_addr_port_t bridge_addrport;
memcpy(&bridge_addrport.addr, &bridge->addr, sizeof(tor_addr_t));
bridge_addrport.port = bridge->port;