[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
[or-cvs] r17985: {tor} simplify. no actual changes i think. ;) (tor/trunk/src/or)
Author: arma
Date: 2009-01-06 16:31:46 -0500 (Tue, 06 Jan 2009)
New Revision: 17985
Modified:
tor/trunk/src/or/circuituse.c
Log:
simplify. no actual changes i think. ;)
Modified: tor/trunk/src/or/circuituse.c
===================================================================
--- tor/trunk/src/or/circuituse.c 2009-01-06 21:28:48 UTC (rev 17984)
+++ tor/trunk/src/or/circuituse.c 2009-01-06 21:31:46 UTC (rev 17985)
@@ -801,32 +801,19 @@
circ->cpath->state != CPATH_STATE_OPEN) {
/* We failed at the first hop. If there's an OR connection
to blame, blame it. */
- or_connection_t *n_conn = NULL;
const char *n_conn_id = circ->_base.n_hop->identity_digest;
if (circ->_base.n_conn) {
- n_conn = circ->_base.n_conn;
- if (n_conn) n_conn_id = n_conn->identity_digest;
- } else if (circ->_base.state == CIRCUIT_STATE_OR_WAIT &&
- circ->_base.n_hop) {
- n_conn_id = circ->_base.n_hop->identity_digest;
- /* Do not set n_conn. If we're in state_or_wait, it's wrong to blame a
- * particular connection for our failure to extend and set its
- * is_bad_for_new_circs field, since no connection ever got a chance to
- * hear our CREATE cell. */
- }
- if (n_conn) {
+ or_connection_t *n_conn = circ->_base.n_conn;
log_info(LD_OR,
"Our circuit failed to get a response from the first hop "
"(%s:%d). I'm going to try to rotate to a better connection.",
n_conn->_base.address, n_conn->_base.port);
n_conn->is_bad_for_new_circs = 1;
}
- if (n_conn_id) {
- entry_guard_register_connect_status(n_conn_id, 0, time(NULL));
- /* if there are any one-hop streams waiting on this circuit, fail
- * them now so they can retry elsewhere. */
- connection_ap_fail_onehop(n_conn_id, circ->build_state);
- }
+ entry_guard_register_connect_status(n_conn_id, 0, time(NULL));
+ /* if there are any one-hop streams waiting on this circuit, fail
+ * them now so they can retry elsewhere. */
+ connection_ap_fail_onehop(n_conn_id, circ->build_state);
}
switch (circ->_base.purpose) {