]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: promex/resolvers: Don't dump metrics if no nameserver is defined
authorChristopher Faulet <cfaulet@haproxy.com>
Mon, 6 Jan 2025 07:41:57 +0000 (08:41 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Mon, 6 Jan 2025 08:08:38 +0000 (09:08 +0100)
commit892eb2bb2ceba2c0e80d51eed69dfc9494a56988
treedb8abd85d2212c2fa0fe2312764c17ec89ce9b89
parent801e39e1ccfc7404354d6870e779f83351593df0
BUG/MEDIUM: promex/resolvers: Don't dump metrics if no nameserver is defined

A 'resolvers' section may be defined without any nameserver. In that case,
we must take care to not dump corresponding Prometheus metrics. However
there is an issue that could lead to a crash or a strange infinite loop
because we are looping on an empty list and, at some point, we are
dereferencing an invalid pointer.

There is an issue because the loop on the nameservers of a resolvers section
is performed via callback functions and not the standard list_for_each_entry
macro. So we must take care to properly detect end of the list and empty
lists for nameservers. But the fix is not so simple because resolvers
sections with and without nameservers may be mixed.

To fix the issue, in rslv_promex_start_ts() and rslv_promex_next_ts(), when the
next resolvers section must be evaluated, a loop is now used to properly skip
empty sections.

This patch is related to #2831. Not sure it fixes it. It must be backported
as far as 3.0.
src/resolvers.c