From 9835bc077a8880ec28c5aa27ca91f35cd480ec10 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Manuel=20P=C3=A9gouri=C3=A9-Gonnard?= Date: Wed, 14 Jan 2015 14:41:58 +0100 Subject: [PATCH] Fix racy test. With exchanges == renego period, sometimes the connection will be closed by the client before the server had time to read the ClientHello, making the test fail. The extra exchange avoids that. --- tests/ssl-opt.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/ssl-opt.sh b/tests/ssl-opt.sh index 51c3fd0c6..8bf7d97bf 100755 --- a/tests/ssl-opt.sh +++ b/tests/ssl-opt.sh @@ -1088,9 +1088,10 @@ run_test "Renegotiation: periodic, just below period" \ -S "SSL - An unexpected message was received from our peer" \ -S "failed" +# one extra exchange to be able to complete renego run_test "Renegotiation: periodic, just above period" \ "$P_SRV debug_level=3 exchanges=9 renegotiation=1 renego_period=3" \ - "$P_CLI debug_level=3 exchanges=3 renegotiation=1" \ + "$P_CLI debug_level=3 exchanges=4 renegotiation=1" \ 0 \ -c "client hello, adding renegotiation extension" \ -s "received TLS_EMPTY_RENEGOTIATION_INFO" \ @@ -1106,7 +1107,7 @@ run_test "Renegotiation: periodic, just above period" \ run_test "Renegotiation: periodic, two times period" \ "$P_SRV debug_level=3 exchanges=9 renegotiation=1 renego_period=3" \ - "$P_CLI debug_level=3 exchanges=6 renegotiation=1" \ + "$P_CLI debug_level=3 exchanges=7 renegotiation=1" \ 0 \ -c "client hello, adding renegotiation extension" \ -s "received TLS_EMPTY_RENEGOTIATION_INFO" \