aboutsummaryrefslogtreecommitdiff
path: root/src/or/config.c
diff options
context:
space:
mode:
Diffstat (limited to 'src/or/config.c')
-rw-r--r--src/or/config.c11
1 files changed, 3 insertions, 8 deletions
diff --git a/src/or/config.c b/src/or/config.c
index 383ecc62f6..67c6140748 100644
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -2286,10 +2286,8 @@ options_validate(or_options_t *old_options, or_options_t *options,
if (options->NatdPort == 0 && options->NatdListenAddress != NULL)
REJECT("NatdPort must be defined if NatdListenAddress is defined.");
-#if 0 /* don't complain, since a standard configuration does this! */
- if (options->SocksPort == 0 && options->SocksListenAddress != NULL)
- REJECT("SocksPort must be defined if SocksListenAddress is defined.");
-#endif
+ /* Don't gripe about SocksPort 0 with SocksListenAddress set; a standard
+ * configuration does this. */
for (i = 0; i < 3; ++i) {
int is_socks = i==0;
@@ -3090,10 +3088,7 @@ config_register_addressmaps(or_options_t *options)
if (smartlist_len(elts) >= 2) {
from = smartlist_get(elts,0);
to = smartlist_get(elts,1);
- if (!is_plausible_address(from)) {
- log_warn(LD_CONFIG,
- "Skipping invalid argument '%s' to MapAddress", from);
- } else if (!is_plausible_address(to)) {
+ if (address_is_invalid_destination(to)) {
log_warn(LD_CONFIG,
"Skipping invalid argument '%s' to MapAddress", to);
} else {