From 5c7e7bcbc790caeaddc238c3d1924013467bb877 Mon Sep 17 00:00:00 2001 From: David Cooper Date: Wed, 19 Feb 2025 12:47:35 -0800 Subject: [PATCH] Fix check for OpenSSL supported curves OpenSSL 3.X outputs a different error message than previous versions when $OPENSSL s_client -curves X ... is called with an unsupported curve. This was resulting in the check within find_openssl_binary() adding every curve to $OPENSSL_SUPPORTED_CURVES, even ones that were not supported. This commit changes to check in order to detect the new error message. --- testssl.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/testssl.sh b/testssl.sh index 76efce6..963d063 100755 --- a/testssl.sh +++ b/testssl.sh @@ -20505,7 +20505,7 @@ find_openssl_binary() { HAS_CURVES=true for curve in "${curves_ossl[@]}"; do # Same as above, we just don't need a port for invalid. - $OPENSSL s_client -curves $curve -connect $NXCONNECT &1 | grep -Eiaq "Error with command|unknown option" + $OPENSSL s_client -curves $curve -connect $NXCONNECT &1 | grep -Eiaq "Error with command|unknown option|Call to SSL_CONF_cmd(.*) failed" [[ $? -ne 0 ]] && OSSL_SUPPORTED_CURVES+=" $curve " done fi