changeset 8995:19251fb83187 quic

QUIC: merged ngx_quic_close_quic() and ngx_quic_close_connection(). The separate ngx_quic_close_quic() doesn't make much sense.
author Vladimir Homutov <vl@nginx.com>
date Tue, 01 Feb 2022 13:05:38 +0300
parents e00295b76395
children 430755fcdb61
files src/event/quic/ngx_event_quic.c
diffstat 1 files changed, 26 insertions(+), 37 deletions(-) [+]
line wrap: on
line diff
--- a/src/event/quic/ngx_event_quic.c
+++ b/src/event/quic/ngx_event_quic.c
@@ -16,7 +16,6 @@ static ngx_int_t ngx_quic_handle_statele
     ngx_quic_header_t *pkt);
 static void ngx_quic_input_handler(ngx_event_t *rev);
 
-static ngx_int_t ngx_quic_close_quic(ngx_connection_t *c, ngx_int_t rc);
 static void ngx_quic_close_timer_handler(ngx_event_t *ev);
 
 static ngx_int_t ngx_quic_handle_datagram(ngx_connection_t *c, ngx_buf_t *b,
@@ -455,7 +454,9 @@ ngx_quic_input_handler(ngx_event_t *rev)
 void
 ngx_quic_close_connection(ngx_connection_t *c, ngx_int_t rc)
 {
+    ngx_uint_t              i;
     ngx_pool_t             *pool;
+    ngx_quic_send_ctx_t    *ctx;
     ngx_quic_connection_t  *qc;
 
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
@@ -469,41 +470,9 @@ ngx_quic_close_connection(ngx_connection
                            "quic close connection early error");
         }
 
-    } else if (ngx_quic_close_quic(c, rc) == NGX_AGAIN) {
-        return;
-    }
-
-    if (c->ssl) {
-        (void) ngx_ssl_shutdown(c);
-    }
-
-    if (c->read->timer_set) {
-        ngx_del_timer(c->read);
+        goto quic_done;
     }
 
-#if (NGX_STAT_STUB)
-    (void) ngx_atomic_fetch_add(ngx_stat_active, -1);
-#endif
-
-    c->destroyed = 1;
-
-    pool = c->pool;
-
-    ngx_close_connection(c);
-
-    ngx_destroy_pool(pool);
-}
-
-
-static ngx_int_t
-ngx_quic_close_quic(ngx_connection_t *c, ngx_int_t rc)
-{
-    ngx_uint_t              i;
-    ngx_quic_send_ctx_t    *ctx;
-    ngx_quic_connection_t  *qc;
-
-    qc = ngx_quic_get_connection(c);
-
     if (!qc->closing) {
 
         /* drop packets from retransmit queues, no ack is expected */
@@ -582,7 +551,7 @@ ngx_quic_close_quic(ngx_connection_t *c,
     }
 
     if (ngx_quic_close_streams(c, qc) == NGX_AGAIN) {
-        return NGX_AGAIN;
+        return;
     }
 
     if (qc->push.timer_set) {
@@ -602,7 +571,7 @@ ngx_quic_close_quic(ngx_connection_t *c,
     }
 
     if (qc->close.timer_set) {
-        return NGX_AGAIN;
+        return;
     }
 
     ngx_quic_close_sockets(c);
@@ -613,7 +582,27 @@ ngx_quic_close_quic(ngx_connection_t *c,
     /* may be tested from SSL callback during SSL shutdown */
     c->udp = NULL;
 
-    return NGX_OK;
+quic_done:
+
+    if (c->ssl) {
+        (void) ngx_ssl_shutdown(c);
+    }
+
+    if (c->read->timer_set) {
+        ngx_del_timer(c->read);
+    }
+
+#if (NGX_STAT_STUB)
+    (void) ngx_atomic_fetch_add(ngx_stat_active, -1);
+#endif
+
+    c->destroyed = 1;
+
+    pool = c->pool;
+
+    ngx_close_connection(c);
+
+    ngx_destroy_pool(pool);
 }