Skip to content

Commit d7e9794

Browse files
authored
Merge pull request #1120 from dcooper16/avoid_clienthello_size_bug
Avoid ClientHello size bug
2 parents 9c07546 + bc3a812 commit d7e9794

1 file changed

Lines changed: 7 additions & 0 deletions

File tree

testssl.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11860,6 +11860,13 @@ socksend_tls_clienthello() {
1186011860
done
1186111861
len_extension=$len_extension+$len_padding_extension+0x4
1186211862
len_extension_hex=$(printf "%02x\n" $len_extension)
11863+
elif [[ ! "$extra_extensions_list" =~ " 0015 " ]] && ( [[ $((len_all%256)) -eq 10 ]] || [[ $((len_all%256)) -eq 14 ]] ); then
11864+
# Some servers fail if the length of the ClientHello is 522, 778, 1034, 1290, ... bytes.
11865+
# A few servers also fail if the length is 526, 782, 1038, 1294, ... bytes.
11866+
# So, if the ClientHello would be one of these length, add a 5-byte padding extension.
11867+
all_extensions="$all_extensions\\x00\\x15\\x00\\x01\\x00"
11868+
len_extension+=5
11869+
len_extension_hex=$(printf "%02x\n" $len_extension)
1186311870
fi
1186411871
len2twobytes "$len_extension_hex"
1186511872
all_extensions="

0 commit comments

Comments
 (0)