]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MAJOR: server: do not delete srv referenced by session
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Tue, 12 Mar 2024 13:09:55 +0000 (14:09 +0100)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Thu, 14 Mar 2024 14:21:07 +0000 (15:21 +0100)
commit7dae3ceaa08ead56104bae6471b236de536c3f51
tree1ba623a491117fed89dc01fdce2bc4d4aab36f0e
parent5ad801c0586bc07109860d40bb4e7da84c985cb0
BUG/MAJOR: server: do not delete srv referenced by session

A server can only be deleted if there is no elements which reference it.
This is taken care via srv_check_for_deletion(), most notably for active
and idle connections.

A special case occurs for connections directly managed by a session.
This is for so-called private connections, when using http-reuse never
or H2 + http-reuse safe for example. In this case. server does not
account these connections into its idle lists. This caused a bug as the
server is deleted despite the session still being able to access it.

To properly fix this, add a new referencing element into the server for
these session connections. A mt_list has been chosen for this. On
default http-reuse, private connections are typically not used so it
won't make any difference. If using H2 servers, or more generally when
dealing with private connections, insert/delete should typically occur
only once per session lifetime so impact on performance should be
minimal.

This should be backported up to 2.4. Note that srv_check_for_deletion()
was introduced in 3.0 dev tree. On backport, the extra condition in it
should be placed in cli_parse_delete_server() instead.
include/haproxy/server-t.h
include/haproxy/session-t.h
include/haproxy/session.h
src/server.c
src/session.c