diff options
author | Nick Mathewson <nickm@torproject.org> | 2013-02-19 15:02:22 -0500 |
---|---|---|
committer | Nick Mathewson <nickm@torproject.org> | 2013-02-19 15:02:22 -0500 |
commit | 8ddf4e218e24167d1c595b74af62dd83b60d99a7 (patch) | |
tree | 05dbacfd35c4a79fef1d2f0ac0f4da52df1bac81 /src | |
parent | 804be10b096de6e6a7204cf3268857d3b59ef51e (diff) | |
parent | 3340d3279de0bb411c506a759778ee7a9b45165b (diff) | |
download | tor-8ddf4e218e24167d1c595b74af62dd83b60d99a7.tar.gz tor-8ddf4e218e24167d1c595b74af62dd83b60d99a7.zip |
Merge branch 'bug8065_v2'
Diffstat (limited to 'src')
-rw-r--r-- | src/or/circuitbuild.c | 2 | ||||
-rw-r--r-- | src/or/circuitbuild.h | 1 | ||||
-rw-r--r-- | src/or/connection_edge.c | 11 |
3 files changed, 12 insertions, 2 deletions
diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c index 40751e02b1..8a3a36accb 100644 --- a/src/or/circuitbuild.c +++ b/src/or/circuitbuild.c @@ -1269,7 +1269,7 @@ pathbias_get_scale_use_threshold(const or_options_t *options) /** * Convert a Guard's path state to string. */ -static const char * +const char * pathbias_state_to_string(path_state_t state) { switch (state) { diff --git a/src/or/circuitbuild.h b/src/or/circuitbuild.h index 3ca8d1531d..a3091707e8 100644 --- a/src/or/circuitbuild.h +++ b/src/or/circuitbuild.h @@ -66,6 +66,7 @@ int pathbias_check_probe_response(circuit_t *circ, const cell_t *cell); void pathbias_count_use_attempt(origin_circuit_t *circ); void pathbias_mark_use_success(origin_circuit_t *circ); void pathbias_mark_use_rollback(origin_circuit_t *circ); +const char *pathbias_state_to_string(path_state_t state); #endif diff --git a/src/or/connection_edge.c b/src/or/connection_edge.c index b4fa3e6fe2..abde2e19f2 100644 --- a/src/or/connection_edge.c +++ b/src/or/connection_edge.c @@ -651,7 +651,16 @@ connection_ap_expire_beginning(void) } continue; } - tor_assert(circ->purpose == CIRCUIT_PURPOSE_C_GENERAL); + if (circ->purpose != CIRCUIT_PURPOSE_C_GENERAL) { + log_warn(LD_BUG, "circuit->purpose == CIRCUIT_PURPOSE_C_GENERAL failed. " + "The purpose on the circuit was %s; it was in state %s, " + "path_state %s.", + circuit_purpose_to_string(circ->purpose), + circuit_state_to_string(circ->state), + CIRCUIT_IS_ORIGIN(circ) ? + pathbias_state_to_string(TO_ORIGIN_CIRCUIT(circ)->path_state) : + "none"); + } log_fn(cutoff < 15 ? LOG_INFO : severity, LD_APP, "We tried for %d seconds to connect to '%s' using exit %s." " Retrying on a new circuit.", |