[tor-dev] [PATCH] remove test for remote SOCKS Port at config validation

Martin Kepplinger martink at posteo.de
Sat Jun 28 12:51:29 UTC 2014


remove the check for a non-local SOCKS Port in the config by removing
the flag when calling parse_port_config() for it.

this should only prevent parse_port_config() from writing a warning
that is written when the listener is opened anyways, according to
https://trac.torproject.org/projects/tor/ticket/4019
---
this is UNTESTED and more of a question if that may suffice for closing
ticket 4019 for now.

 src/or/config.c |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/or/config.c b/src/or/config.c
index 10df839..7c22171 100644
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -6002,8 +6002,7 @@ parse_ports(or_options_t *options, int validate_only,
              options->SocksPort_lines, options->SocksListenAddress,
              "Socks", CONN_TYPE_AP_LISTENER,
              "127.0.0.1", 9050,
-             CL_PORT_WARN_NONLOCAL|CL_PORT_ALLOW_EXTRA_LISTENADDR|
-             CL_PORT_TAKES_HOSTNAMES) < 0) {
+             CL_PORT_ALLOW_EXTRA_LISTENADDR|CL_PORT_TAKES_HOSTNAMES) < 0) {
     *msg = tor_strdup("Invalid SocksPort/SocksListenAddress configuration");
     goto err;
   }
-- 
1.7.10.4



More information about the tor-dev mailing list