]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: pools: fix a possible memory leak in the lockless pool_flush()
authorWilly Tarreau <w@1wt.eu>
Thu, 10 Jun 2021 04:54:22 +0000 (06:54 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 10 Jun 2021 15:46:50 +0000 (17:46 +0200)
commitc239cde26fbf0b07abdf590b77c31154bd65c566
tree89205ef8ed8a4cdf6452b209b659b6b78972af30
parentefbf35caf9a3dcf9ead7bc07d140447e483a6bab
BUG/MINOR: pools: fix a possible memory leak in the lockless pool_flush()

The lockless version of pool_flush() had a leftover of the original
version causing the pool's first entry to be set to NULL at the end.
The problem is that it does this outside of any lock and in a non-
atomic way, so that any concurrent alloc+free would result in a lost
object.

The risk is low and the consequence even lower, given that pool_flush()
is only used in pool_destroy() (hence single-threaded) or by stream_free()
during a soft-stop (not the place where most allocations happen), so in
the worst case it could result in valgrind complaining on soft-stop.

The bug was introduced with the first version of the code, in 1.9, so
the fix can be backported to all stable versions.
src/pool.c