QUIC TSERVER: Fix probable nondeterminism in some OS network stacks

Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19897)

diff --git a/test/quic_tserver_test.c b/test/quic_tserver_test.c
index c6c5c719e972e06b9c368764762d289b813edfda..e743adbeab41ecd1caa45a645a4bffa144a56f25 100644 (file)
--- a/test/quic_tserver_test.c
+++ b/test/quic_tserver_test.c
@@ -214,11 +214,18 @@ static int test_tserver(void)
             if (!TEST_false(ret))
                 goto err;
 
-            if (!TEST_int_eq(SSL_get_error(c_ssl, ret), SSL_ERROR_ZERO_RETURN))
-                goto err;
+            /*
+             * Allow the implementation to take as long as it wants to finally
+             * notice EOS. Account for varied timings in OS networking stacks.
+             */
+            if (SSL_get_error(c_ssl, ret) != SSL_ERROR_WANT_READ) {
+                if (!TEST_int_eq(SSL_get_error(c_ssl, ret),
+                                 SSL_ERROR_ZERO_RETURN))
+                    goto err;
 
-            /* DONE */
-            break;
+                /* DONE */
+                break;
+            }
         }
 
         /*