1
0
mirror of https://github.com/drwetter/testssl.sh.git synced 2025-04-14 05:26:06 +02:00

Bugfix for extra extensions

If extra extensions are provided to socksend_tls_clienthello(), the socksend_tls_clienthello() needs to determine what extensions were provided so that it doesn't add any of these extensions a second time. The code that was looping through the extra extensions to get the extension IDs was using the value for the length of the string. This commit fixes the error.
This commit is contained in:
David Cooper 2018-02-15 14:13:04 -05:00 committed by GitHub
parent e564783fec
commit 41839ab221
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -10965,7 +10965,7 @@ socksend_tls_clienthello() {
# the provided values for those extensions. # the provided values for those extensions.
extra_extensions="$(tolower "$4")" extra_extensions="$(tolower "$4")"
code2network "$extra_extensions" code2network "$extra_extensions"
len_all=${#extra_extensions} len_all=${#NW_STR}
for (( i=0; i < len_all; i=i+16+4*0x$len_extension_hex )); do for (( i=0; i < len_all; i=i+16+4*0x$len_extension_hex )); do
part2=$i+4 part2=$i+4
extn_type="${NW_STR:i:2}${NW_STR:part2:2}" extn_type="${NW_STR:i:2}${NW_STR:part2:2}"