summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Goulet <dgoulet@torproject.org>2017-08-24 09:00:18 -0400
committerDavid Goulet <dgoulet@torproject.org>2017-08-24 13:03:29 -0400
commit1077e9894cd60d9b991e38d965e57be00fd83277 (patch)
treed91aabd5b81ae1347e4c7cbfb122e4fdeb84287c
parentdca105d58853c7ba6f8ef594cca34230dbb87394 (diff)
downloadtor-1077e9894cd60d9b991e38d965e57be00fd83277.tar.gz
tor-1077e9894cd60d9b991e38d965e57be00fd83277.zip
circuit: Avoid needless log info in circuit_get_best()
When looking for an introduction circuit in circuit_get_best(), we log an info message if we are about to launch a new intro circuit in parallel. However, the condition was considering marked for close circuit leading to the function triggering the log info even though there is actually no valid intro circuit. Signed-off-by: David Goulet <dgoulet@torproject.org>
-rw-r--r--src/or/circuituse.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/or/circuituse.c b/src/or/circuituse.c
index 93bccf0eb8..2472ac5452 100644
--- a/src/or/circuituse.c
+++ b/src/or/circuituse.c
@@ -337,7 +337,8 @@ circuit_get_best(const entry_connection_t *conn,
/* Log an info message if we're going to launch a new intro circ in
* parallel */
if (purpose == CIRCUIT_PURPOSE_C_INTRODUCE_ACK_WAIT &&
- !must_be_open && origin_circ->hs_circ_has_timed_out) {
+ !must_be_open && origin_circ->hs_circ_has_timed_out &&
+ !circ->marked_for_close) {
intro_going_on_but_too_old = 1;
continue;
}