tor

commit 33989cd002c761c37d2ed551f8aae0955dde4cd4
parent 834e1f8085af43c8f7bd07c346cf6f760a519f13
Author: Nick Mathewson <nickm@torproject.org>
Date:   Mon,  4 Sep 2017 12:48:26 -0400

Merge remote-tracking branch 'dgoulet/bug23366_032_01'

Diffstat:
A
changes/bug23366
 | 
4
++++
M
src/or/config.c
 | 
27
+++++++++++++++++++++------
M
src/test/test_options.c
 | 
2
++

3 files changed, 27 insertions(+), 6 deletions(-)
diff --git a/changes/bug23366 b/changes/bug23366
@@ -0,0 +1,4 @@
+  o Minor bugfixes (test):
+    - Fix a broken OutboundAddress option unit test because the parsing
+      function was never returning an error on failure. Fixes bug #23366.;
+      bugfix on tor-0.3.0.3-alpha.
diff --git a/src/or/config.c b/src/or/config.c
@@ -7921,13 +7921,28 @@ parse_outbound_addresses(or_options_t *options, int validate_only, char **msg)
     memset(&options->OutboundBindAddresses, 0,
            sizeof(options->OutboundBindAddresses));
   }
-  parse_outbound_address_lines(options->OutboundBindAddress,
-                      OUTBOUND_ADDR_EXIT_AND_OR, options, validate_only, msg);
-  parse_outbound_address_lines(options->OutboundBindAddressOR,
-                      OUTBOUND_ADDR_OR, options, validate_only, msg);
-  parse_outbound_address_lines(options->OutboundBindAddressExit,
-                      OUTBOUND_ADDR_EXIT, options, validate_only, msg);
+
+  if (parse_outbound_address_lines(options->OutboundBindAddress,
+                                   OUTBOUND_ADDR_EXIT_AND_OR, options,
+                                   validate_only, msg) < 0) {
+    goto err;
+  }
+
+  if (parse_outbound_address_lines(options->OutboundBindAddressOR,
+                                   OUTBOUND_ADDR_OR, options, validate_only,
+                                   msg) < 0) {
+    goto err;
+  }
+
+  if (parse_outbound_address_lines(options->OutboundBindAddressExit,
+                                   OUTBOUND_ADDR_EXIT, options, validate_only,
+                                   msg)  < 0) {
+    goto err;
+  }
+
   return 0;
+ err:
+  return -1;
 }
 
 /** Load one of the geoip files, <a>family</a> determining which
diff --git a/src/test/test_options.c b/src/test/test_options.c
@@ -528,6 +528,8 @@ test_options_validate__outbound_addresses(void *ignored)
 
   ret = options_validate(tdata->old_opt, tdata->opt, tdata->def_opt, 0, &msg);
   tt_int_op(ret, OP_EQ, -1);
+  tt_str_op(msg, OP_EQ, "Multiple outbound bind addresses configured: "
+                        "xxyy!!!sdfaf");
 
  done:
   free_options_test_data(tdata);