]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MAJOR: connection: Never remove connection from idle lists outside the lock
authorChristopher Faulet <cfaulet@haproxy.com>
Fri, 22 Apr 2022 15:56:24 +0000 (17:56 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Fri, 22 Apr 2022 16:16:06 +0000 (18:16 +0200)
commit7ae48a70d6f2de0a6d2b4426bdb2a129baffc579
tree5d95327673fa476f82b8a7753b1c4088de8363a6
parenteaa703ef062fbf5f215ceeaedac9aa27c037cd81
BUG/MAJOR: connection: Never remove connection from idle lists outside the lock

Since the idle connections management changed to use eb-trees instead of MT
lists, a lock must be acquired to manipulate servers idle/safe/available
connection lists. However, it remains an unprotected use in
connect_server(), when a connection is removed from an idle list if the mux
has no more streams available. Thus it is possible to remove a connection
from an idle list on a thread, while another one is looking for a idle
connection. Of couse, this may lead to a crash.

To fix the bug, we must take care to acquire the idle connections lock
first. The bug was introduced by the commit f232cb3e9 ("MEDIUM: connection:
replace idle conn lists by eb trees").

The patch must be backported as far as 2.4.
src/backend.c