diff options
| author | Jay Satiro <raysatiro@yahoo.com> | 2018-09-18 16:35:36 -0400 |
|---|---|---|
| committer | Jay Satiro <raysatiro@yahoo.com> | 2018-09-20 14:12:25 -0400 |
| commit | 2e5651a5ce0ee18d610cacfaa2b3ec5073a40c34 (patch) | |
| tree | 65f4bd915f2efa2c975f9aa78e32020fb6a64feb /lib/vtls/nss.c | |
| parent | ba782baac3009e44295589743bb8ae8220793e74 (diff) | |
| download | curl-2e5651a5ce0ee18d610cacfaa2b3ec5073a40c34.tar.gz | |
vtls: fix ssl version "or later" behavior change for many backends
- Treat CURL_SSLVERSION_MAX_NONE the same as
CURL_SSLVERSION_MAX_DEFAULT. Prior to this change NONE would mean use
the minimum version also as the maximum.
This is a follow-up to 6015cef which changed the behavior of setting
the SSL version so that the requested version would only be the minimum
and not the maximum. It appears it was (mostly) implemented in OpenSSL
but not other backends. In other words CURL_SSLVERSION_TLSv1_0 used to
mean use just TLS v1.0 and now it means use TLS v1.0 *or later*.
- Fix CURL_SSLVERSION_MAX_DEFAULT for OpenSSL.
Prior to this change CURL_SSLVERSION_MAX_DEFAULT with OpenSSL was
erroneously treated as always TLS 1.3, and would cause an error if
OpenSSL was built without TLS 1.3 support.
Co-authored-by: Daniel Gustafsson
Fixes https://github.com/curl/curl/issues/2969
Closes https://github.com/curl/curl/pull/3012
Diffstat (limited to 'lib/vtls/nss.c')
| -rw-r--r-- | lib/vtls/nss.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/lib/vtls/nss.c b/lib/vtls/nss.c index 0c5a806f1..c945b453c 100644 --- a/lib/vtls/nss.c +++ b/lib/vtls/nss.c @@ -1715,8 +1715,6 @@ static CURLcode nss_init_sslver(SSLVersionRange *sslver, failf(data, "unsupported min version passed via CURLOPT_SSLVERSION"); return result; } - if(max == CURL_SSLVERSION_MAX_NONE) - sslver->max = sslver->min; } switch(max) { |