diff options
| author | Brad King <brad.king@kitware.com> | 2023-10-12 23:06:14 (GMT) |
|---|---|---|
| committer | Brad King <brad.king@kitware.com> | 2023-10-12 23:06:14 (GMT) |
| commit | f2d8c369692354a2f75684ec0295d1cb19ef2a4a (patch) | |
| tree | 3477457c9595d7ffe51e61566d75a7cf8a732e40 /Utilities | |
| parent | 1d6fab5537c80117df799998cafbf31320a5016e (diff) | |
| parent | 701f52670f55830fbf0e592ffe5600a69889be8b (diff) | |
| download | CMake-f2d8c369692354a2f75684ec0295d1cb19ef2a4a.zip CMake-f2d8c369692354a2f75684ec0295d1cb19ef2a4a.tar.gz CMake-f2d8c369692354a2f75684ec0295d1cb19ef2a4a.tar.bz2 | |
Merge branch 'backport-curl-socks5-fix' into release-3.27
Merge-request: !8872
Diffstat (limited to 'Utilities')
| -rw-r--r-- | Utilities/cmcurl/lib/socks.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/Utilities/cmcurl/lib/socks.c b/Utilities/cmcurl/lib/socks.c index 53d798a..b74071a 100644 --- a/Utilities/cmcurl/lib/socks.c +++ b/Utilities/cmcurl/lib/socks.c @@ -588,9 +588,9 @@ static CURLproxycode do_SOCKS5(struct Curl_cfilter *cf, /* RFC1928 chapter 5 specifies max 255 chars for domain name in packet */ if(!socks5_resolve_local && hostname_len > 255) { - infof(data, "SOCKS5: server resolving disabled for hostnames of " - "length > 255 [actual len=%zu]", hostname_len); - socks5_resolve_local = TRUE; + failf(data, "SOCKS5: the destination hostname is too long to be " + "resolved remotely by the proxy."); + return CURLPX_LONG_HOSTNAME; } if(auth & ~(CURLAUTH_BASIC | CURLAUTH_GSSAPI)) @@ -904,7 +904,7 @@ CONNECT_RESOLVE_REMOTE: } else { socksreq[len++] = 3; - socksreq[len++] = (char) hostname_len; /* one byte address length */ + socksreq[len++] = (unsigned char) hostname_len; /* one byte length */ memcpy(&socksreq[len], sx->hostname, hostname_len); /* w/o NULL */ len += hostname_len; } |