From: Nick Mathewson <nickm@torproject.org>
Date: Tue, 4 Nov 2014 05:48:25 +0000 (-0500)
Subject: Merge branch 'bug13315_squashed'
X-Git-Tag: tor-0.2.6.2-alpha~114
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=60c86a3b79d542543f191a78f5c7ea2c77c0d3f0;p=thirdparty%2Ftor.git

Merge branch 'bug13315_squashed'

Conflicts:
	src/or/buffers.c
---

60c86a3b79d542543f191a78f5c7ea2c77c0d3f0
diff --cc src/or/buffers.c
index 080d8fb6c7,691845ec10..bd33fe451d
--- a/src/or/buffers.c
+++ b/src/or/buffers.c
@@@ -2054,8 -2048,18 +2054,20 @@@ parse_socks(const char *data, size_t da
            req->address[len] = 0;
            req->port = ntohs(get_uint16(data+5+len));
            *drain_out = 5+len+2;
-           if (!tor_strisprint(req->address) || strchr(req->address,'\"')) {
+ 
+           if (string_is_valid_ipv4_address(req->address) ||
+               string_is_valid_ipv6_address(req->address)) {
+             log_unsafe_socks_warning(5,req->address,req->port,safe_socks);
+ 
+             if (safe_socks) {
+               socks_request_set_socks5_error(req, SOCKS5_NOT_ALLOWED);
+               return -1;
+             }
+           }
+ 
+           if (!string_is_valid_hostname(req->address)) {
 +            socks_request_set_socks5_error(req, SOCKS5_GENERAL_ERROR);
++
              log_warn(LD_PROTOCOL,
                       "Your application (using socks5 to port %d) gave Tor "
                       "a malformed hostname: %s. Rejecting the connection.",