From 1411842933afba657599dd1cccbc61040dfe9b4b Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Mon, 20 Sep 2010 13:33:57 -0400 Subject: [PATCH] Count unknown authorities as unknown only once, not once per signature. Do not double-report signatures from unrecognized authorities both as "from unknown authority" and "not present". Fixes bug 1956, bugfix on 0.2.2.16-alpha. --- changes/bug1956 | 5 +++++ src/or/networkstatus.c | 8 ++++++-- 2 files changed, 11 insertions(+), 2 deletions(-) create mode 100644 changes/bug1956 diff --git a/changes/bug1956 b/changes/bug1956 new file mode 100644 index 0000000000..3e402b7765 --- /dev/null +++ b/changes/bug1956 @@ -0,0 +1,5 @@ + o Minor bugfixes + - Do not double-report signatures from unrecognized authorities both + as "from unknown authority" and "not present". Fixes bug 1956, + bugfix on 0.2.2.16-alpha. + diff --git a/src/or/networkstatus.c b/src/or/networkstatus.c index 9bb5546d97..1d8a20be11 100644 --- a/src/or/networkstatus.c +++ b/src/or/networkstatus.c @@ -482,6 +482,7 @@ networkstatus_check_consensus_signature(networkstatus_t *consensus, voter) { int good_here = 0; int bad_here = 0; + int unknown_here = 0; int missing_key_here = 0, dl_failed_key_here = 0; SMARTLIST_FOREACH_BEGIN(voter->sigs, document_signature_t *, sig) { if (!sig->good_signature && !sig->bad_signature && @@ -497,7 +498,7 @@ networkstatus_check_consensus_signature(networkstatus_t *consensus, if (!is_v3_auth) { smartlist_add(unrecognized, voter); - ++n_unknown; + ++unknown_here; continue; } else if (!cert || cert->expires < now) { smartlist_add(need_certs_from, voter); @@ -527,8 +528,11 @@ networkstatus_check_consensus_signature(networkstatus_t *consensus, ++n_missing_key; if (dl_failed_key_here) ++n_dl_failed_key; - } else + } else if (unknown_here) { + ++n_unknown; + } else { ++n_no_signature; + } } SMARTLIST_FOREACH_END(voter); /* Now see whether we're missing any voters entirely. */