Fix race condition in TLSProxy
authorMatt Caswell <matt@openssl.org>
Fri, 3 Nov 2017 10:43:06 +0000 (10:43 +0000)
committerMatt Caswell <matt@openssl.org>
Tue, 7 Nov 2017 10:51:17 +0000 (10:51 +0000)
commit018632ae986af33b2dddb2c1496bf91446d35abe
tree1fe61184d8bddf8a2b02b19c8dc99a1f901fe716
parent9f5671c7e9f30dfa53b1a2b553f234c2761ceb66
Fix race condition in TLSProxy

Normally TLSProxy waits for the s_server process to finish before
continuing. However in cases where serverconnects > 1 we need to keep the
s_server process around for a later test so we continue immediately. This
means that TAP test output can end up being printed to stdout at the same
time as s_server is printing stuff. This confuses the test runner and can
cause spurious test failures. This commit introduces a small delay in cases
where serverconnects > 1 in order to give s_server enough time to finish
what it was doing before we continue to the next test.

Fixes #4129

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4660)
util/perl/TLSProxy/Proxy.pm