1berry/haproxy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

BUG/MINOR: mux-h1: Fix error handling when H1S allocation failed on client side

Browse Source 
The goto label is not at the right place. When H1S allocation failed, the
error is immediately handled. Thus, "no_parsing" label must be set just
after h1_send() call to skip the request parsing part.

It is 2-7-specific. No backport needed.
This commit is contained in:
Christopher Faulet 2022-11-17 15:54:12 +01:00
parent ddfb50eec6
commit e6ef4cd747
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/mux_h1.c
View File

@ -2981,7 +2981,6 @@ static int h1_process(struct h1c * h1c)
h1_release_buf(h1c, &h1s->rxbuf);
h1_set_idle_expiration(h1c);
no_parsing:
if (h1s->flags & H1S_F_INTERNAL_ERROR) {
h1_handle_internal_err(h1c);
h1c->flags &= ~H1C_F_WAIT_NEXT_REQ;
@ -3003,6 +3002,7 @@ static int h1_process(struct h1c * h1c)
}
}
no_parsing:
h1_send(h1c);
/* H1 connection must be released ASAP if: