Bruce Mitchener identified (bug report #1099640) the never-ending SOCKS5

problem with the version byte and the check for bad versions. Bruce has lots
of clues on this, and based on his suggestion I've now removed the check of
that byte since it seems to be able to contain 1 or 5.

1 files changed, 2 insertions, 2 deletions

diff --git a/lib/url.c b/lib/url.c
index 29c35bf05..6c3b662b4 100644
--- a/lib/url.c
+++ b/lib/url.c
@@ -1829,8 +1829,8 @@ static int handleSock5Proxy(const char *proxy_name,
       return 1;
     }
 
-    if ((socksreq[0] != 5) || /* version */
-        (socksreq[1] != 0)) { /* status */
+    /* ignore the first (VER) byte */
+    if (socksreq[1] != 0) { /* status */
       failf(conn->data, "User was rejected by the SOCKS5 server (%d %d).",
             socksreq[0], socksreq[1]);
       return 1;