diff options
| -rw-r--r-- | changes/bug8065 | 6 | ||||
| -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 |
4 files changed, 18 insertions, 2 deletions
diff --git a/changes/bug8065 b/changes/bug8065 new file mode 100644 index 000000000..06dbae8cd --- /dev/null +++ b/changes/bug8065 @@ -0,0 +1,6 @@ + o Minor bugfixes: + - Downgrade an assertion in connection_ap_expire_beginning to + an LD_BUG message. The fix for bug 8024 should prevent this + message from displaying, but just in case a warn that we can + diagnose is better than more assert crashes. Fix for bug 8065; + bugfix on 0.2.4.8-alpha. diff --git a/src/or/circuitbuild.c b/src/or/circuitbuild.c index 40751e02b..8a3a36acc 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 3ca8d1531..a3091707e 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 b4fa3e6fe..abde2e19f 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.", |