BUG/MEDIUM: server: fix potential null-deref after previous fix
authorWilly Tarreau <w@1wt.eu>
Thu, 22 May 2025 16:09:12 +0000 (18:09 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Tue, 27 May 2025 13:15:37 +0000 (15:15 +0200)
commit301898097c003a386f29e3f9b7d3b624b1c91608
treea053eed13927c271f2900ca6e41f387da12fddb2
parentcf571a65c28db518c0220617b60035f73c52ace5
BUG/MEDIUM: server: fix potential null-deref after previous fix

A valid build warning was reported in the CI with latest commit b40ce97ecc
("BUG/MEDIUM: server: fix crash after duplicate GUID insertion"). Indeed,
if the first test in the function fails, we branch to the err label
with guid==NULL and will crash there. Let's just test guid before
dereferencing it for freeing.

This needs to be backported to 3.0 as well since the commit above was
meant to go there.

(cherry picked from commit 28c7a22790a587c6a3ee1652188ad6786d59b687)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit d883c7113fc28d48468c4cc5dc0a27ad3b122497)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
src/guid.c