r15899@catbus: nickm | 2007-10-18 07:49:15 -0400

 "if (!router_get_trusted_dirservers())" is a bad test: router_get_trusted_dirservers() always returns a list.  Instead, check for whether the list is empty.

svn:r12013

diff --git a/src/or/config.c b/src/or/config.c
index 052a62668c13b54db6607dac038583e4efd974ed..7d64ede61cd811180f1d55b6ec33bf659424fd28 100644 (file)
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -973,7 +973,7 @@ options_act(or_options_t *old_options)
       }
     }
   } else {
-    if (!router_get_trusted_dir_servers())
+    if (!smartlist_len(router_get_trusted_dir_servers()))
       add_default_trusted_dirservers();
   }