diff options
author | Richard Levitte <levitte@openssl.org> | 2018-01-15 10:40:24 +0100 |
---|---|---|
committer | Richard Levitte <levitte@openssl.org> | 2018-01-15 18:55:32 +0100 |
commit | 9556a442188421b109d8c4ff51d34f3981aa25f0 (patch) | |
tree | e0ee806d2dfdcae7be1e5e8503cceb9a2fe77932 /apps | |
parent | 44e688bca36ab4e1b4cd1cf6710871b7906e7c96 (diff) |
Fix intermittent Windows and Cygwin failures in s_server
The same kind of failure that has already been observed on the
s_client can sometimes also be observed on s_server, so we need to add
the same kind of 50ms delay as was previously added on s_client.
Ref: git commit cb2e10f257a464c6b475b321dd9e4769df84dbf6:
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Bernd Edlinger <bernd.edlinger@hotmail.de>
(Merged from https://github.com/openssl/openssl/pull/5079)
(cherry picked from commit d17bdfc26d87097c2c80854fd521b6ecebdf5ebb)
Diffstat (limited to 'apps')
-rw-r--r-- | apps/s_socket.c | 14 |
1 files changed, 14 insertions, 0 deletions
diff --git a/apps/s_socket.c b/apps/s_socket.c index 9988626d33..e8fc37f97c 100644 --- a/apps/s_socket.c +++ b/apps/s_socket.c @@ -178,6 +178,20 @@ int do_server(int *accept_sock, const char *host, const char *port, break; } i = (*cb)(sock, type, context); + + /* + * Give the socket time to send its last data before we close it. + * No amount of setting SO_LINGER etc on the socket seems to + * persuade Windows to send the data before closing the socket... + * but sleeping for a short time seems to do it (units in ms) + * TODO: Find a better way to do this + */ +#if defined(OPENSSL_SYS_WINDOWS) + Sleep(50); +#elif defined(OPENSSL_SYS_CYGWIN) + usleep(50000); +#endif + /* * If we ended with an alert being sent, but still with data in the * network buffer to be read, then calling BIO_closesocket() will |