tree 3591818b58e2f91928c3864b1fd3237fda9c3897
parent 2b8f7ab9c59ced8d40d9f9f7ac5adea9499a6fc0
author Aurelien DARRAGON <adarragon@haproxy.com> 1689060666 +0200
committer Christopher Faulet <cfaulet@haproxy.com> 1689945608 +0200

BUG/MINOR: sink/log: properly deinit srv in sink_new_from_logsrv()

When errors are encountered in sink_new_from_logsrv() function,
incompetely allocated ressources are freed to prevent memory leaks.

For instance: logsrv implicit server is manually cleaned up on error prior
to returning from the function.

However, since 198e92a8e5 ("MINOR: server: add a global list of all known
servers") every server created using new_server() is registered to the
global list, but unfortunately the manual srv cleanup in
sink_new_from_logsrv() doesn't remove the srv from the global list, so the
freed server will still be referenced there, which can result in invalid
reads later.

Moreover, server API has evolved since, and now the srv_drop() function is
available for that purpose, so let's use it, but make sure that srv is
freed before the proxy because on older versions srv_drop() expects the
srv to be linked to a valid proxy pointer.

This must be backported up to 2.4.

[For 2.4 version, free_server() must be used instead of srv_drop()]

(cherry picked from commit c38cf3cf98d30d6d069c36d3d475511d892201b9)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>