mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-09-23 23:01:24 +02:00
Code Issues Projects Releases Wiki Activity

OPTIM: h3: skip buf realign if no trailer to encode

Browse Source 
h3_resp_trailers_send() may be called due to an HTX EOT block present
without preceeding HTX TRAILER block. In this case, no HEADERS frame
will be generated by H3 layer and MUX will emit an empty STREAM frame
with FIN set.

However, before skipping these, some operations are conducted on qcs
buffer to realign it and try to encode the QPACK field section line in a
buffer copy. These operation are thus unneeded if no trailer is
generated. Even worse, the function will fail if there is not enough
space in the buffer for the superfluous QPACK section line.

To improve this situation, this patch adds an early goto statement to
skip most operations in h3_resp_trailers_send() if no HTX trailer block
is found.

This patch is related to github issue #2006.

This should be backported up to 2.7.
This commit is contained in:
Amaury Denoyelle 2023-01-26 17:49:21 +01:00
parent 224ba5cffe
commit 4be5435014
1 changed files with 18 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
src/h3.c
View File

@ -1377,6 +1377,13 @@ static int h3_resp_trailers_send(struct qcs *qcs, struct htx *htx)
} }
} }
if (!hdr) {
/* No headers encoded here so no need to generate a H3 HEADERS
* frame. Mux will send an empty QUIC STREAM frame with FIN.
*/
TRACE_DATA("skipping trailer", H3_EV_TX_HDR, qcs->qcc->conn, qcs);
goto end;
}
list[hdr].n = ist(""); list[hdr].n = ist("");
res = mux_get_buf(qcs); res = mux_get_buf(qcs);
@ -1420,24 +1427,24 @@ static int h3_resp_trailers_send(struct qcs *qcs, struct htx *htx)
} }
} }
/* Now that all headers are encoded, we are certain that res buffer is
* big enough.
*/
/* Check that at least one header was encoded in buffer. */ /* Check that at least one header was encoded in buffer. */
if (b_tail(&headers_buf) != tail) { if (b_tail(&headers_buf) == tail) {
b_putchr(res, 0x01); /* h3 HEADERS frame type */
if (!b_quic_enc_int(res, b_data(&headers_buf), 8))
ABORT_NOW();
b_add(res, b_data(&headers_buf));
}
else {
/* No headers encoded here so no need to generate a H3 HEADERS /* No headers encoded here so no need to generate a H3 HEADERS
* frame. Mux will send an empty QUIC STREAM frame with FIN. * frame. Mux will send an empty QUIC STREAM frame with FIN.
*/ */
TRACE_DATA("skipping trailer", H3_EV_TX_HDR, qcs->qcc->conn, qcs); TRACE_DATA("skipping trailer", H3_EV_TX_HDR, qcs->qcc->conn, qcs);
goto end;
} }
/* Now that all headers are encoded, we are certain that res buffer is
* big enough.
*/
b_putchr(res, 0x01); /* h3 HEADERS frame type */
if (!b_quic_enc_int(res, b_data(&headers_buf), 8))
ABORT_NOW();
b_add(res, b_data(&headers_buf));
end:
ret = 0; ret = 0;
blk = htx_get_head_blk(htx); blk = htx_get_head_blk(htx);
while (blk) { while (blk) {