]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: server: do not leak default-server in defaults sections
authorWilly Tarreau <w@1wt.eu>
Thu, 23 Nov 2023 13:28:14 +0000 (14:28 +0100)
committerWilly Tarreau <w@1wt.eu>
Thu, 23 Nov 2023 13:32:55 +0000 (14:32 +0100)
commit53da8bfcb6d3f4918a45fe77347317ad885ba25e
tree053a1ed9a11a224cd5f9bfad663e3b0b09820f1e
parent7fc52357cb10e98c0c13358aa6f7babc3d81da95
BUG/MINOR: server: do not leak default-server in defaults sections

When a default-server directive is used in a defaults section, it's never
freed and the "defaults" proxy gets reset without freeing the fields from
that default-server. Normally there are no allocation there, except for
the config file location stored in srv->conf.file form an strdup() since
commit 9394a9444 ("REORG: server: move alert traces in parse_server")
that appeared in 2.4. In addition, if a "default-server" directive
appears multiple times in a defaults section, one more entry will be
leaked per call.

This commit addresses this by checking that we don't overwrite the file
upon multiple calls, and by clearing it when resetting the default proxy.
This should be backported to 2.4.
src/proxy.c
src/server.c