mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-09-22 06:11:32 +02:00
Code Issues Projects Releases Wiki Activity

BUG/MEDIUM: mux-h1: Properly ignore trailers when a content-length is announced

Browse Source 
It is not possible in H1, but in H2 (and probably H3) it is possible to have
trailers at the end of a message while a Content-Length was announced.

However, depending if the trailers are received with the last HTX DATA block
or the zero-copy forwarding is used or not, an processing error may be
triggered, leading to a 500-internal-error.

To fix the issue, when a content-length is announced and all the payload was
processed, we switch the message to H1_MSG_DONE state only if the
end-of-message was also reported (HTX_FL_EOM flag set). Otherwise, it is
switched to H1_MSG_TRAILERS state to be able to properly ignored the
trailers, if so.

The patch must be backported as far as 2.4. Be careful, this part was highly
refactored. The patch will have to be adapted to be backported.
This commit is contained in:
Christopher Faulet 2023-11-27 08:23:45 +01:00
parent 3dd55fa132
commit b4eaadae84
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/mux_h1.c
View File

@ -2485,7 +2485,7 @@ static size_t h1_make_eoh(struct h1s *h1s, struct h1m *h1m, struct htx *htx, siz
} }
else if (!chunk_memcat(&outbuf, "\r\n", 2)) else if (!chunk_memcat(&outbuf, "\r\n", 2))
goto full; goto full;
h1m->state = H1_MSG_DONE; h1m->state = ((htx->flags & HTX_FL_EOM) ? H1_MSG_DONE : H1_MSG_TRAILERS);
} }
else { else {
if (!chunk_memcat(&outbuf, "\r\n", 2)) if (!chunk_memcat(&outbuf, "\r\n", 2))
@ -2577,7 +2577,7 @@ static size_t h1_make_data(struct h1s *h1s, struct h1m *h1m, struct buffer *buf,
} }
h1m->curr_len -= count; h1m->curr_len -= count;
if (!h1m->curr_len) if (!h1m->curr_len)
h1m->state = H1_MSG_DONE; h1m->state = (eom ? H1_MSG_DONE : H1_MSG_TRAILERS);
} }
else if (h1m->flags & H1_MF_CHNK) { else if (h1m->flags & H1_MF_CHNK) {
/* The message is chunked. We need to check if we must /* The message is chunked. We need to check if we must
@ -4530,7 +4530,7 @@ static size_t h1_done_ff(struct stconn *sc)
h1m->curr_len -= total; h1m->curr_len -= total;
if (!h1m->curr_len && (h1m->flags & H1_MF_CLEN)) if (!h1m->curr_len && (h1m->flags & H1_MF_CLEN))
h1m->state = H1_MSG_DONE; h1m->state = ((sd->iobuf.flags & IOBUF_FL_EOI) ? H1_MSG_DONE : H1_MSG_TRAILERS);
else if (!h1m->curr_len && (h1m->flags & H1_MF_CHNK)) { else if (!h1m->curr_len && (h1m->flags & H1_MF_CHNK)) {
if (h1m->state == H1_MSG_DATA) if (h1m->state == H1_MSG_DATA)
h1m->state = H1_MSG_CHUNK_CRLF; h1m->state = H1_MSG_CHUNK_CRLF;