note a bug in directories_have_accepted_server_descriptor() pointed

out by weasel

svn:r11914

diff --git a/src/or/directory.c b/src/or/directory.c
index 95143449a4de7b319aaa162fd661f5bda81d2043..d949ebb1ed737a1817f46d4b650b187e5e2e1b1c 100644 (file)
--- a/src/or/directory.c
+++ b/src/or/directory.c
@@ -192,6 +192,10 @@ directories_have_accepted_server_descriptor(void)
 {
   smartlist_t *servers = router_get_trusted_dir_servers();
   or_options_t *options = get_options();
+  /* XXX020 If any authority of the needed type is down, this
+   * function will never return true. Perhaps we need to be
+   * tolerant of down servers? Or even better, should we change
+   * this so one successful upload is enough? -RD */
   SMARTLIST_FOREACH(servers, trusted_dir_server_t *, d, {
     if ((d->type & options->_PublishServerDescriptor) &&
         !d->has_accepted_serverdesc) {