diff options
| -rw-r--r-- | changes/bug1107 | 5 | ||||
| -rw-r--r-- | src/or/config.c | 12 |
2 files changed, 17 insertions, 0 deletions
diff --git a/changes/bug1107 b/changes/bug1107 new file mode 100644 index 0000000000..c396b5d68a --- /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 a4e4f89c1a..fc8455656b 100644 --- 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 |