summaryrefslogtreecommitdiff
path: root/src/or/circuitbuild.c
diff options
context:
space:
mode:
authorMike Perry <mikeperry-git@fscked.org>2012-12-08 16:37:22 -0800
committerMike Perry <mikeperry-git@fscked.org>2012-12-08 16:37:22 -0800
commit686fc222593fd46ec82d62f0fa62ca02900c1014 (patch)
treee7bcb41cb31078acb8c127f8006216e4a9cddea4 /src/or/circuitbuild.c
parentb599a6ed07fd588500a256ef815e87729449626a (diff)
downloadtor-686fc222593fd46ec82d62f0fa62ca02900c1014.tar.gz
tor-686fc222593fd46ec82d62f0fa62ca02900c1014.zip
Allow any valid 'end' cell to mean a circuit was used successfully.
Also improve some log messages.
Diffstat (limited to 'src/or/circuitbuild.c')
-rw-r--r--src/or/circuitbuild.c9
1 files changed, 6 insertions, 3 deletions
diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c
index 9b1236f34d..c3a5827589 100644
--- a/src/or/circuitbuild.c
+++ b/src/or/circuitbuild.c
@@ -1411,16 +1411,18 @@ pathbias_check_close(origin_circuit_t *ocirc, int reason)
// state.. Can we use that? Does optimistic data change this?
log_info(LD_CIRC,
- "Circuit closed without successful use for reason %d. "
+ "Circuit %d closed without successful use for reason %d. "
"Circuit purpose %d currently %s.",
+ ocirc->global_identifier,
reason, circ->purpose, circuit_state_to_string(circ->state));
pathbias_count_unusable(ocirc);
} else {
if (reason & END_CIRC_REASON_FLAG_REMOTE) {
/* Unused remote circ close reasons all could be bias */
log_info(LD_CIRC,
- "Circuit remote-closed without successful use for reason %d. "
+ "Circuit %d remote-closed without successful use for reason %d. "
"Circuit purpose %d currently %s.",
+ ocirc->global_identifier,
reason, circ->purpose, circuit_state_to_string(circ->state));
pathbias_count_collapse(ocirc);
} else if ((reason & ~END_CIRC_REASON_FLAG_REMOTE)
@@ -1432,8 +1434,9 @@ pathbias_check_close(origin_circuit_t *ocirc, int reason)
/* FIXME: Only count bias if the network is live?
* What about clock jumps/suspends? */
log_info(LD_CIRC,
- "Circuit's channel closed without successful use for reason %d, "
+ "Circuit %d's channel closed without successful use for reason %d, "
"channel reason %d. Circuit purpose %d currently %s.",
+ ocirc->global_identifier,
reason, circ->n_chan->reason_for_closing,
circ->purpose, circuit_state_to_string(circ->state));
pathbias_count_collapse(ocirc);