Check for port config before addr config in client port options

Otherwise, we'll fail, since "9050" looks like a perfectly fine address.

diff --git a/src/or/config.c b/src/or/config.c
index 7a96304e182d5bee7e6eb4586cab0ec6ab1d3dd0..96d50954babccd8116714986b51cb9401e7f7b43 100644 (file)
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -5039,20 +5039,23 @@ parse_client_port_config(smartlist_t *out,
         tor_free(addrtmp);
         goto err;
       }
-    } else if (tor_addr_port_parse(addrport, &addr, &ptmp) == 0) {
-      if (ptmp == 0) {
-        log_warn(LD_CONFIG, "%sPort line has address but no port", portname);
-        goto err;
-      }
-      port = ptmp;
     } else {
+      /* Try parsing integer port before address, because, who knows?
+         "9050" might be a valid address. */
       port = (int) tor_parse_long(addrport, 10, 0, 65535, &ok, NULL);
-      if (!ok) {
+      if (ok) {
+        tor_addr_from_str(&addr, defaultaddr);
+      } else if (tor_addr_port_parse(addrport, &addr, &ptmp) == 0) {
+        if (ptmp == 0) {
+          log_warn(LD_CONFIG, "%sPort line has address but no port", portname);
+          goto err;
+        }
+        port = ptmp;
+      } else {
         log_warn(LD_CONFIG, "Couldn't parse address '%s' for %sPort",
                  escaped(addrport), portname);
         goto err;
       }
-      tor_addr_from_str(&addr, defaultaddr);
     }
 
     /* Now parse the rest of the options, if any. */