]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: chunk: make sure to flush the trash pool before resizing
authorWilly Tarreau <w@1wt.eu>
Wed, 29 Jan 2025 16:51:12 +0000 (17:51 +0100)
committerWilly Tarreau <w@1wt.eu>
Wed, 29 Jan 2025 16:55:18 +0000 (17:55 +0100)
commitbd7a688b8b38901037cd2f9856533a788ebdbdf7
tree72b0bc58fb6066e40db2ce1b183ae68ff30ed8fe
parentb43e5d8c1692a0f15db4e621e3cff41158a47167
BUG/MEDIUM: chunk: make sure to flush the trash pool before resizing

Late in 3.1 we've added an integrity check to make sure we didn't keep
trash objects allocated before resizing the trash with commit 0bfd36e7b8
("MINOR: chunk: add a BUG_ON upon the next init_trash_buffer()"), but
it turns out that the counter that is being checked includes the number
of objects left in local thread caches. As such it can trigger despite
no object being allocated. This precisely happens when setting
tune.memory.hot-size to a few megabytes because some temporarily used
trash objects will remain in cache.

In order to address this, let's first flush the pool before running
the check. That was previously done by pool_destroy() but the check
had to be inserted before it. So now we first flush the trash pool,
then verify it's no longer used, and finally we can destroy it.

This needs to be backported to 3.1. Thanks to Christian Ruppert for
reporting this bug.
src/chunk.c