QUIC: fixed chain returned from ngx_quic_write_chain().

Previously, when input ended on a QUIC buffer boundary, input chain was not
advanced to the next buffer.  As a result, ngx_quic_write_chain() returned
a chain with an empty buffer instead of NULL.  This broke HTTP write filter,
preventing it from closing the HTTP request and eventually timing out.

Now input chain is always advanced to a buffer that has data, before checking
QUIC buffer boundary condition.
This commit is contained in:
Roman Arutyunyan 2022-01-25 09:45:50 +03:00
parent a0aa287d13
commit c403822674

View File

@ -536,14 +536,16 @@ ngx_quic_write_chain(ngx_connection_t *c, ngx_chain_t **chain, ngx_chain_t *in,
continue; continue;
} }
for (p = b->pos + offset; p != b->last && in; /* void */ ) { p = b->pos + offset;
while (in) {
if (!ngx_buf_in_memory(in->buf) || in->buf->pos == in->buf->last) { if (!ngx_buf_in_memory(in->buf) || in->buf->pos == in->buf->last) {
in = in->next; in = in->next;
continue; continue;
} }
if (limit == 0) { if (p == b->last || limit == 0) {
break; break;
} }