BUG/MEDIUM: server: fix crash after duplicate GUID insertion
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Thu, 22 May 2025 15:48:58 +0000 (17:48 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Tue, 27 May 2025 12:08:28 +0000 (14:08 +0200)
commitdfde7bd6b0b6f5e87e4ab65059f9a86219c55c95
tree96b962d42ee1f7d905df97c275f99685a0b7a138
parente31f447c9d6681a4f3f98bfdd7298082adadc7a1
BUG/MEDIUM: server: fix crash after duplicate GUID insertion

On "add server", if a GUID is defined, guid_insert() is used to add the
entry into the global GUID tree. If a similar entry already exists, GUID
insertion fails and the server creation is eventually aborted.

A crash could occur in this case because of an invalid memory access via
guid_remove(). The latter is caused via free_server() as the server
insertion is rejected. The invalid occurs on GUID key.

The issue occurs because of guid_insert(). The function properly
deallocates the GUID key on duplicate insertion, but it failed to reset
<guid.node.key> to NULL. This caused the invalid memory access on
guid_remove(). To fix this, ensure that key member is properly resetted
on guid_insert() error path.

This must be backported up to 3.0.

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