Skip to content

Commit f7c9030

Browse files
author
David Cooper
committed
Fix sub_cipherlists()
This commit fixes a bug in sub_cipherlists() when using sockets to test whether a server supports a set of ciphers with SSLv2. Code that is supposed to extract the list of SSLv2 ciphers to test for is incorrectly reading from the list of non-SSLv2 ciphers.
1 parent 11325ff commit f7c9030

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
@@ -5630,7 +5630,7 @@ sub_cipherlists() {
56305630
elif [[ -n "$7" ]]; then
56315631
sslv2_sockets "$7" "true"
56325632
if [[ $? -eq 3 ]] && [[ "$V2_HELLO_CIPHERSPEC_LENGTH" -ne 0 ]]; then
5633-
sslv2_cipherlist="$(strip_spaces "${6//,/}")"
5633+
sslv2_cipherlist="$(strip_spaces "${7//,/}")"
56345634
len=${#sslv2_cipherlist}
56355635
detected_ssl2_ciphers="$(grep "Supported cipher: " "$TEMPDIR/$NODEIP.parse_sslv2_serverhello.txt")"
56365636
for (( i=0; i<len; i=i+6 )); do

0 commit comments

Comments
 (0)