BUG/MINOR: mux-h1: Fix error handling when H1S allocation failed on client side
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.
diff --git a/src/mux_h1.c b/src/mux_h1.c
index 36a55f9..1ab6758 100644
--- a/src/mux_h1.c
+++ b/src/mux_h1.c
@@ -2981,7 +2981,6 @@
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 @@
}
}
+ no_parsing:
h1_send(h1c);
/* H1 connection must be released ASAP if: