# HG changeset patch # User Sergey Kandaurov # Date 1597782820 -10800 # Node ID 6e1c88f82280e393d888e3d2987f1409f87ef39e # Parent 830680e78b2425857a6f1dff6b68787b8f758955 QUIC: changed ctx->largest_ack initial value to type maximum. In particular, this prevents declaring packet number 0 as lost if there aren't yet any acknowledgements in this packet number space. For example, only Initial packets were acknowledged in handshake. diff --git a/src/event/ngx_event_quic.c b/src/event/ngx_event_quic.c --- a/src/event/ngx_event_quic.c +++ b/src/event/ngx_event_quic.c @@ -690,6 +690,7 @@ ngx_quic_new_connection(ngx_connection_t ngx_queue_init(&qc->send_ctx[i].frames); ngx_queue_init(&qc->send_ctx[i].sent); qc->send_ctx[i].largest_pn = (uint64_t) -1; + qc->send_ctx[i].largest_ack = (uint64_t) -1; } for (i = 0; i < NGX_QUIC_ENCRYPTION_LAST; i++) { @@ -2346,7 +2347,7 @@ ngx_quic_handle_ack_frame(ngx_connection } /* 13.2.3. Receiver Tracking of ACK Frames */ - if (ctx->largest_ack < max) { + if (ctx->largest_ack < max || ctx->largest_ack == (uint64_t) -1) { ctx->largest_ack = max; ngx_log_debug1(NGX_LOG_DEBUG_EVENT, c->log, 0, "quic updated largest received ack: %uL", max);