r16034@catbus: nickm | 2007-10-22 12:30:38 -0400

 Move an XXX020 to the logical place.

svn:r12104

diff --git a/src/or/networkstatus.c b/src/or/networkstatus.c
index 589633c97c9c4c895ea28d54b708b783a0df1563..2735203dc5bd252fca2952103106f3cf6f3f9751 100644 (file)
--- a/src/or/networkstatus.c
+++ b/src/or/networkstatus.c
@@ -990,6 +990,9 @@ networkstatus_set_current_consensus(const char *consensus, int from_cache,
 
   download_status_reset(&consensus_dl_status); /*XXXX020 not quite right.*/
 
+  /* XXXX020 check dates for plausibility.  Don't trust a consensus whose
+   * valid-after date is very far in the future. */
+
   /* Are we missing any certificates at all? */
   if (r != 1)
     authority_certs_fetch_missing(c, now);

diff --git a/src/or/routerparse.c b/src/or/routerparse.c
index c634ade44d747f627a46b9e408cb58d86f9ab858..a2ba0d27b80ec239019f385a03abfa4f3f2d83bb 100644 (file)
--- a/src/or/routerparse.c
+++ b/src/or/routerparse.c
@@ -2220,8 +2220,6 @@ networkstatus_parse_vote_from_string(const char *s, const char **eos_out,
     goto err;
   }
 
-  /* XXXX020 check dates for plausibility.  ??? */
-
   if (eos_out)
     *eos_out = end_of_footer;