From e09f8d256f60fd0af62e510f3eaab9e9936f3a6a Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Thu, 11 Jun 2020 16:47:50 +0100 Subject: [PATCH] Don't send supported groups if no-ec and we're doing DTLS The supported_groups extension only supported EC groups in DTLS. Therefore we shouldn't send it in a no-ec build. Reviewed-by: Shane Lontis (Merged from https://github.com/openssl/openssl/pull/11914) --- ssl/statem/extensions_clnt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ssl/statem/extensions_clnt.c b/ssl/statem/extensions_clnt.c index dbdedad1ab..abff069ec9 100644 --- a/ssl/statem/extensions_clnt.c +++ b/ssl/statem/extensions_clnt.c @@ -211,7 +211,7 @@ EXT_RETURN tls_construct_ctos_supported_groups(SSL *s, WPACKET *pkt, } #if defined(OPENSSL_NO_EC) - if (max_version < TLS1_3_VERSION) + if (SSL_IS_DTLS(s) || max_version < TLS1_3_VERSION) return EXT_RETURN_NOT_SENT; #else if (!use_ecc(s, min_version, max_version) && max_version < TLS1_3_VERSION) -- 2.25.1