Note that there is incorrect behavior in directory.c code. Arma, please check this too.

svn:r5161
This commit is contained in:
Nick Mathewson 2005-09-30 01:38:09 +00:00
parent 92451f74a8
commit edd15cb781

View File

@ -1011,7 +1011,13 @@ connection_dir_client_reached_eof(connection_t *conn)
if (which) { if (which) {
/* We only call these if it's a "fp/" request, since /* We only call these if it's a "fp/" request, since
* it's the only way we'll be adding new server descriptors * it's the only way we'll be adding new server descriptors
* and thus the only way we'd affect has_fetched_directory. */ * and thus the only way we'd affect has_fetched_directory.
*
* XXXX Not so. We will fetch "all" routers if we have zero or one
* networkstatus, so we can bootstrap faster. See
* update_router_descriptor_downloads(). One of these behaviors is
* wrong.
**/
/* as we learn from them, we remove them from 'which' */ /* as we learn from them, we remove them from 'which' */
router_load_routers_from_string(body, 0, which); router_load_routers_from_string(body, 0, which);