BUG/MEDIUM: promex: Use right context pointers to dump backends extra-counters
authorChristopher Faulet <cfaulet@haproxy.com>
Tue, 14 Jan 2025 06:39:48 +0000 (07:39 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Thu, 23 Jan 2025 10:32:54 +0000 (11:32 +0100)
commit0156edf056f5047375c017963d48464bde4de196
tree1e4e78e1cdb162ce4f905818b0643fdf838412ae
parent202b85321345e2489d3170e4166cfd3794a3a380
BUG/MEDIUM: promex: Use right context pointers to dump backends extra-counters

When backends extra counters are dumped, the wrong pointer was used in the
promex context to retrieve the stats module. p[1] must be used instead of
p[2]. Because of this typo, a infinite loop could be experienced if the
output buffer is full during this stage. But in all cases an overflow is
possible leading to a memory corruption.

This patch may be related to issue #2831. It must be backported as far as
3.0.

(cherry picked from commit 91578212d7cef7405f3631db43a086ac9f9ca162)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit ecd012d853de2ac8996bccd6cb8968d56a86ed84)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
addons/promex/service-prometheus.c