]> git.ipfire.org Git - thirdparty/haproxy.git/commit
CLEANUP: mux-h1: remove the unneeded test on conn->owner in h1s_finish_detach() master flx04/master
authorWilly Tarreau <w@1wt.eu>
Mon, 20 Apr 2026 14:15:27 +0000 (16:15 +0200)
committerWilly Tarreau <w@1wt.eu>
Tue, 21 Apr 2026 06:45:46 +0000 (08:45 +0200)
commit2f61566b03051bbb74921a38c7b3f031b9f1ee6f
tree7791b262a2c68690b7e8e47796e1f1fa135eeb5c
parent2e26e427a25d19edb91e81bfabfb80b1c8ffc8a3
CLEANUP: mux-h1: remove the unneeded test on conn->owner in h1s_finish_detach()

There was a test below the "release" label on conn->owner to decide
whether to kill the connection or not. But this test is not needed,
because:
  - for frontends, it's always set so the test never matches
  - for backends, it was NULL on the second stream once a request
    was being reused from an idle pool, so it couldn't be used to
    discriminate between connections. In practice, the goal was to
    try to detect certain dead connections but all cases leading to
    such connections are either already handled in the tests before
    (which don't reach this label), or are handled by the other
    conditions.

Thus, let's remove this confusing test.
src/mux_h1.c