nihilist/tor
1
0
Fork 0
mirror of https://gitlab.torproject.org/tpo/core/tor.git synced 2024-11-10 21:23:58 +01:00
Code Issues Packages Projects Releases Wiki Activity

Change an assertion into a warning in connection_or_handle_event_cb()

Browse Source 
Possibly addresses bug 4873, though IMO that's likely not a real
bug: it seems likely to have been an ssl version mismatch.
This commit is contained in:
Nick Mathewson 2012-05-24 11:14:28 -04:00
parent 466276faa5
commit 6b7c3b42ee
2 changed files with 19 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
changes/bug4873 Normal file
View File

@ -0,0 +1,3 @@
o Minor bugfixes:
- Turn an assertion (that the number of handshakes received as a
server is not < 1) into a warning. Bug 4873.

16
src/or/connection_or.c
View File

@ -1286,7 +1286,10 @@ connection_or_handle_event_cb(struct bufferevent *bufev, short event,
return; /* ???? */
}
}
} else if (tor_tls_get_num_server_handshakes(conn->tls) == 1) {
} else {
const int handshakes = tor_tls_get_num_server_handshakes(conn->tls);
if (handshakes == 1) {
/* v2 or v3 handshake, as a server. Only got one handshake, so
* wait for the next one. */
tor_tls_set_renegotiate_callback(conn->tls,
@ -1295,18 +1298,19 @@ connection_or_handle_event_cb(struct bufferevent *bufev, short event,
conn->_base.state = OR_CONN_STATE_TLS_SERVER_RENEGOTIATING;
/* return 0; */
return; /* ???? */
} else {
const int handshakes = tor_tls_get_num_server_handshakes(conn->tls);
tor_assert(handshakes >= 2);
if (handshakes == 2) {
} else if (handshakes == 2) {
/* v2 handshake, as a server. Two handshakes happened already,
* so we treat renegotiation as done.
*/
connection_or_tls_renegotiated_cb(conn->tls, conn);
} else {
} else if (handshakes > 2) {
log_warn(LD_OR, "More than two handshakes done on connection. "
"Closing.");
connection_mark_for_close(TO_CONN(conn));
} else {
log_warn(LD_BUG, "We were unexpectedly unexpectedly told that "
"a connection got %d handshakes. Closing.", handshakes);
connection_mark_for_close(TO_CONN(conn));
}
return;
}