r15848@catbus: nickm | 2007-10-16 10:17:02 -0400

Only drop routers for not appearing in the consensus if their purpose is GENERAL.


svn:r11984
This commit is contained in:
Nick Mathewson 2007-10-16 14:25:14 +00:00
parent 9ca2625833
commit 03ce9d3ee7

View File

@ -2667,11 +2667,10 @@ router_add_to_routerlist(routerinfo_t *router, const char **msg,
} }
} }
/*XXXX020 I had suspicions about whether this was correct, but now I if (router->purpose == ROUTER_PURPOSE_GENERAL &&
* can't remember why. :( -NM */ consensus && !in_consensus && !authdir_mode(get_options())) {
if (consensus && !in_consensus && !authdir_mode(get_options())) { /* If it's a general router not listed in the consensus, then don't
/* If it's not listed in the consensus, then don't consider replacing * consider replacing the latest router with it. */
* the latest router with it. */
if (!from_cache && should_cache_old_descriptors()) if (!from_cache && should_cache_old_descriptors())
signed_desc_append_to_journal(&router->cache_info, signed_desc_append_to_journal(&router->cache_info,
router_get_store(routerlist, router)); router_get_store(routerlist, router));