changeset 8944:b7284807b4fa quic

QUIC: refactored ngx_quic_validate_path(). The function now accepts path argument, as suggested by the name. Socket is not really needed inside.
author Vladimir Homutov <vl@nginx.com>
date Thu, 16 Dec 2021 11:49:08 +0300
parents 118a34e32121
children e72db9162180
files src/event/quic/ngx_event_quic_migration.c
diffstat 1 files changed, 3 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/src/event/quic/ngx_event_quic_migration.c
+++ b/src/event/quic/ngx_event_quic_migration.c
@@ -13,7 +13,7 @@
 static void ngx_quic_set_connection_path(ngx_connection_t *c,
     ngx_quic_path_t *path);
 static ngx_int_t ngx_quic_validate_path(ngx_connection_t *c,
-    ngx_quic_socket_t *qsock);
+    ngx_quic_path_t *path);
 static ngx_int_t ngx_quic_send_path_challenge(ngx_connection_t *c,
     ngx_quic_path_t *path);
 static ngx_int_t ngx_quic_path_restore(ngx_connection_t *c);
@@ -422,7 +422,7 @@ ngx_quic_handle_migration(ngx_connection
                    ngx_quic_path_state_str(next));
 
     if (next->state == NGX_QUIC_PATH_NEW) {
-        if (ngx_quic_validate_path(c, qsock) != NGX_OK) {
+        if (ngx_quic_validate_path(c, qsock->path) != NGX_OK) {
             return NGX_ERROR;
         }
     }
@@ -478,17 +478,14 @@ ngx_quic_handle_migration(ngx_connection
 
 
 static ngx_int_t
-ngx_quic_validate_path(ngx_connection_t *c, ngx_quic_socket_t *qsock)
+ngx_quic_validate_path(ngx_connection_t *c, ngx_quic_path_t *path)
 {
     ngx_msec_t              pto;
-    ngx_quic_path_t        *path;
     ngx_quic_send_ctx_t    *ctx;
     ngx_quic_connection_t  *qc;
 
     qc = ngx_quic_get_connection(c);
 
-    path = qsock->path;
-
     ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0,
                    "quic initiated validation of new path #%uL",
                    path->seqnum);