]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: trace: fix null deref in lockon mechanism since TRACE_ENABLED()
authorWilly Tarreau <w@1wt.eu>
Tue, 6 Aug 2024 09:32:10 +0000 (11:32 +0200)
committerWilly Tarreau <w@1wt.eu>
Wed, 7 Aug 2024 14:02:59 +0000 (16:02 +0200)
commitb5df6b5a31b86b4403f00b7e0230c97883eca0f3
tree6214789bcb685e870b2becb3b7251751538ca277
parent88a752ca789e6a2f863e16a35ddaa4fade5ccecd
BUG/MEDIUM: trace: fix null deref in lockon mechanism since TRACE_ENABLED()

When calling TRACE_ENABLED(), which is called by TRACE_PRINTF(), we pass
a NULL plockptr to __trace_enabled(). This argument is used when lockon
is active, and may update the pointer. This is an overlook which also
broke the lockon mechanism because now for calls from __trace(), it
dereferences a pointer pointing to NULL, and never updates it due to the
broken condition, so that trace() never sets up src->lockon_ptr.

The bug was introduced in 2.8 by commit 8f9a9704bb ("MINOR: trace: add a
TRACE_ENABLED() macro to determine if a trace is active"), so the fix must
be backported there.
src/trace.c