From 928c584ec219f4d2b5b60a22842390e4057e7dd0 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Sun, 4 Nov 2007 02:29:27 +0000 Subject: [PATCH] r14700@tombo: nickm | 2007-11-03 22:29:21 -0400 Fix sign error in networkstatus_note_certs_arrived(). svn:r12371 --- ChangeLog | 5 +++++ src/or/networkstatus.c | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index 8050421556..e8434aba0c 100644 --- a/ChangeLog +++ b/ChangeLog @@ -67,6 +67,11 @@ Changes in version 0.2.0.10-alpha - 2007-11-0? the Tor process. Patch from Marco Bonetti with tweaks from Mike Perry. + o Major bugfixes: + - Consider replace the current consensus when certificates arrive + that make the pending consensus valid. Previously, we were only + considering replacement when the new certs _didn't_ help. + o Minor bugfixes: - Refuse to start if both ORPort and UseBridges are set. Bugfix on 0.2.0.x. diff --git a/src/or/networkstatus.c b/src/or/networkstatus.c index cacb1307c3..2bb653e3ef 100644 --- a/src/or/networkstatus.c +++ b/src/or/networkstatus.c @@ -1384,7 +1384,7 @@ networkstatus_note_certs_arrived(void) { if (consensus_waiting_for_certs) { if (networkstatus_check_consensus_signature( - consensus_waiting_for_certs, 0)<0) { + consensus_waiting_for_certs, 0)>=0) { if (!networkstatus_set_current_consensus( consensus_waiting_for_certs_body, 0, 1)) { tor_free(consensus_waiting_for_certs_body);