aboutsummaryrefslogtreecommitdiff
path: root/src/feature
diff options
context:
space:
mode:
authorGeorge Kadianakis <desnacked@riseup.net>2020-10-20 13:26:19 +0300
committerGeorge Kadianakis <desnacked@riseup.net>2020-10-20 13:26:19 +0300
commit19302a1dfd8da73ff86b0c36534221b71c501c32 (patch)
tree22fab1d6f4fbc963a0c263b2ea19357a92efdea5 /src/feature
parentf7adf3653ff313d45d70bed7c0fdcdd0d64f29ab (diff)
parentcb4cedae686bd227d42997840b3a6b0b3bc5e936 (diff)
downloadtor-19302a1dfd8da73ff86b0c36534221b71c501c32.tar.gz
tor-19302a1dfd8da73ff86b0c36534221b71c501c32.zip
Merge remote-tracking branch 'tor-gitlab/mr/148' into master
Diffstat (limited to 'src/feature')
-rw-r--r--src/feature/control/control_bootstrap.c12
-rw-r--r--src/feature/relay/circuitbuild_relay.c1
2 files changed, 13 insertions, 0 deletions
diff --git a/src/feature/control/control_bootstrap.c b/src/feature/control/control_bootstrap.c
index d4f2adde81..d6dfdad94e 100644
--- a/src/feature/control/control_bootstrap.c
+++ b/src/feature/control/control_bootstrap.c
@@ -348,6 +348,18 @@ control_event_bootstrap_prob_or, (const char *warn, int reason,
{
int dowarn = 0;
+ if (! or_conn->potentially_used_for_bootstrapping) {
+ /* We never decided that this channel was a good match for one of our
+ * origin_circuit_t objects. That means that we probably launched it
+ * for somebody else, most likely in response to an EXTEND cell.
+ *
+ * Since EXTEND cells can contain arbitrarily broken descriptions of
+ * relays, a failure on this connection here won't necessarily indicate a
+ * bootstrapping problem.
+ */
+ return;
+ }
+
if (or_conn->have_noted_bootstrap_problem)
return;
diff --git a/src/feature/relay/circuitbuild_relay.c b/src/feature/relay/circuitbuild_relay.c
index 64f3c341ae..289a5be557 100644
--- a/src/feature/relay/circuitbuild_relay.c
+++ b/src/feature/relay/circuitbuild_relay.c
@@ -475,6 +475,7 @@ circuit_extend(struct cell_t *cell, struct circuit_t *circ)
&ec.ed_pubkey,
ipv4_valid ? &ec.orport_ipv4.addr : NULL,
ipv6_valid ? &ec.orport_ipv6.addr : NULL,
+ false,
&msg,
&should_launch);