Skip to content

Commit f929e39

Browse files
authored
Fix #2271
This commit fixes #2271 by adding the `-no_ssl2` option to the call to get_host_cert() in run_drown(). There is at least one server that causes OpenSSL to hang if this call to get_host_cert() results in an SSLv2 ClientHello being sent. Since this call to get_host_cert() only needs to find the server's certificate in cases in which the server does not support SSLv2, there is no need to send an SSLv2 ClientHello.
1 parent 52fe210 commit f929e39

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

testssl.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15646,7 +15646,7 @@ run_drown() {
1564615646
# to $CERT_FINGERPRINT_SHA2, so if $CERT_FINGERPRINT_SHA2 is not empty, but
1564715647
# $RSA_CERT_FINGERPRINT_SHA2 is empty, then the server doesn't have an RSA certificate.
1564815648
if [[ -z "$CERT_FINGERPRINT_SHA2" ]]; then
15649-
get_host_cert "-cipher aRSA"
15649+
get_host_cert "-cipher aRSA -no_ssl2"
1565015650
[[ $? -eq 0 ]] && cert_fingerprint_sha2="$($OPENSSL x509 -noout -in $HOSTCERT -fingerprint -sha256 2>>$ERRFILE | sed -e 's/^.*Fingerprint=//' -e 's/://g' )"
1565115651
else
1565215652
cert_fingerprint_sha2="$RSA_CERT_FINGERPRINT_SHA2"

0 commit comments

Comments
 (0)