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 10:35:13 +0000 (11:35 +0100)
commit00f01f5e636351e1d91dbf62c80e1d0fea34172f
tree11deb342b1a55b35647b1237fea42a5ba6e56914
parent85e3bf658163d97fd09ac63f6c997d230d32729b
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.

(cherry picked from commit 892eb2bb2ceba2c0e80d51eed69dfc9494a56988)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
src/resolvers.c