From 3037d2bd348eb184ed924a590012a23f68a1b47f Mon Sep 17 00:00:00 2001 From: Selva Nair Date: Tue, 2 Nov 2021 00:23:14 -0400 Subject: [PATCH] Fix tls-version-min default once again commit 51be733ba236610dff6a1c361cf59172db97473a claimed to correct this but did not do it properly. (my fault). The check whether tls-version-min is set by the user or not was still wrong. Hope this fixes it for good. Signed-off-by: Selva Nair Acked-by: Gert Doering Message-Id: <20211102042314.19113-1-selva.nair@gmail.com> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg23091.html Signed-off-by: Gert Doering --- src/openvpn/options.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/openvpn/options.c b/src/openvpn/options.c index f8b910420..c912c8dd8 100644 --- a/src/openvpn/options.c +++ b/src/openvpn/options.c @@ -3165,7 +3165,9 @@ static void options_set_backwards_compatible_options(struct options *o) { /* TLS min version is not set */ - if ((o->ssl_flags & SSLF_TLS_VERSION_MIN_MASK) == 0) + int tls_ver_min = (o->ssl_flags >> SSLF_TLS_VERSION_MIN_SHIFT) + & SSLF_TLS_VERSION_MIN_MASK; + if (tls_ver_min == 0) { int tls_ver_max = (o->ssl_flags >> SSLF_TLS_VERSION_MAX_SHIFT) & SSLF_TLS_VERSION_MAX_MASK; -- 2.47.2