Bug #1107: Complain if PublishServerDescriptor receives 0 or 1 in a list

0 or 1 should only be passed as the sole argument. Warn for now, reject
in 0.2.3.x.

diff --git a/changes/bug1107 b/changes/bug1107
new file mode 100644 (file)
index 0000000..c396b5d
--- /dev/null
+++ b/changes/bug1107
@@ -0,0 +1,5 @@
+  o Minor bugfixes:
+    - Complain if PublishServerDescriptor is given multiple arguments that
+      include 0 or 1. This configuration will be rejected in future.
+      Bugfix in 0.2.0.1-alpha, closes bug 1107.
+

diff --git a/src/or/config.c b/src/or/config.c
index a4e4f89c1af67e312355a92b5d0c5d3ce7f23c22..fc8455656b676972308c7cc21b6aacf5ea2cfd5a 100644 (file)
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -3578,6 +3578,18 @@ options_validate(or_options_t *old_options, or_options_t *options,
   if (options->AccelDir && !options->AccelName)
     REJECT("Can't use hardware crypto accelerator dir without engine name.");
 
+  if (options->PublishServerDescriptor)
+    SMARTLIST_FOREACH(options->PublishServerDescriptor, const char *, pubdes, {
+      if (!strcmp(pubdes, "1") || !strcmp(pubdes, "0"))
+        if (smartlist_len(options->PublishServerDescriptor) > 1) {
+          COMPLAIN("You have passed a list of multiple arguments to the "
+                   "PublishServerDescriptor option that includes 0 or 1. "
+                   "0 or 1 should only be used as the sole argument. "
+                   "This configuration will be rejected in a future release.");
+          break;
+        }
+    });
+
   return 0;
 #undef REJECT
 #undef COMPLAIN