Fix s_server bug

If an async event occurs during a renegotiation in SSL_read then s_server
was looping around, detecting we were in init and calling
init_ssl_connection instead of re-calling SSL_read.

Reviewed-by: Rich Salz <rsalz@openssl.org>

diff --git a/apps/s_server.c b/apps/s_server.c
index 6b6035f0ba14c8682d4cfc49723c6cafcedd48fa..14dd8a69dbcd527dbc4eb2384fbee33c68dc224c 100644 (file)
--- a/apps/s_server.c
+++ b/apps/s_server.c
@@ -2152,7 +2152,8 @@ static int sv_body(char *hostname, int s, int stype, unsigned char *context)
         int read_from_sslcon;
 
         read_from_terminal = 0;
         int read_from_sslcon;
 
         read_from_terminal = 0;

-        read_from_sslcon = SSL_pending(con);
+        read_from_sslcon = SSL_pending(con)
+                           || (async && SSL_waiting_for_async(con));

 
         if (!read_from_sslcon) {
             FD_ZERO(&readfds);
 
         if (!read_from_sslcon) {
             FD_ZERO(&readfds);


@@ -2348,7 +2349,13 @@ static int sv_body(char *hostname, int s, int stype, unsigned char *context)
             }
         }
         if (read_from_sslcon) {
             }
         }
         if (read_from_sslcon) {

-            if (!SSL_is_init_finished(con)) {
+            /*
+             * init_ssl_connection handles all async events itself so if we're
+             * waiting for async then we shouldn't go back into
+             * init_ssl_connection
+             */
+            if ((!async || !SSL_waiting_for_async(con))
+                    && !SSL_is_init_finished(con)) {

                 i = init_ssl_connection(con);
 
                 if (i < 0) {
                 i = init_ssl_connection(con);
 
                 if (i < 0) {