Skip to content

Commit

Permalink
Quick fix for drwetter#2590
Browse files Browse the repository at this point in the history
Proper fix need another refactoring/cleanup of the renego test.
  • Loading branch information
emmanuelfuste committed Nov 4, 2024
1 parent 4c706d3 commit 73fa97a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion testssl.sh
Original file line number Diff line number Diff line change
Expand Up @@ -17189,7 +17189,7 @@ run_renego() {
sec_client_renego=1
else
# second try in the foreground as we are sure now it won't hang
(echo R; sleep 1) | $OPENSSL s_client $(s_client_options "$proto $legacycmd $STARTTLS $BUGS -connect $NODEIP:$PORT $PROXY $SNI") >$TMPFILE 2>>$ERRFILE
(echo R; sleep 1) | $OPENSSL s_client $(s_client_options "$proto $legacycmd $STARTTLS $BUGS -connect $NODEIP:$PORT $PROXY $SNI") >$TMPFILE 2>>$ERRFILE
sec_client_renego=$?
# 0 means client is renegotiating & doesn't return an error --> vuln!
# 1 means client tried to renegotiating but the server side errored then. You still see RENEGOTIATING in the output
Expand Down

0 comments on commit 73fa97a

Please sign in to comment.