socks.c: Do not modify and invalidate calculated response length

Second commit to fix issue #944 regarding SOCKS5 error handling.

Reported-by: David Kalnischkies

diff --git a/lib/socks.c b/lib/socks.c
index a3c6ee088252748f1884d081e524c283d07b6749..bb93e424a13877bda32b2074890f6f1fc2670d75 100644 (file)
--- a/lib/socks.c
+++ b/lib/socks.c
@@ -702,10 +702,9 @@ CURLcode Curl_SOCKS5(const char *proxy_name,
     /* decrypt_gssapi_blockread already read the whole packet */
 #endif
     if(len > 10) {
-      len -= 10;
       result = Curl_blockread_all(conn, sock, (char *)&socksreq[10],
-                                  len, &actualread);
-      if(result || (len != actualread)) {
+                                  len - 10, &actualread);
+      if(result || ((len - 10) != actualread)) {
         failf(data, "Failed to receive SOCKS5 connect request ack.");
         return CURLE_COULDNT_CONNECT;
       }