tree 9a03aaa10b55e8d139c55e05c094236cefb12166
parent 2814afb38f5a4abf57bbdeb63a275b8ee2081cf4
author Willy Tarreau <w@1wt.eu> 1603788697 +0100
committer Christopher Faulet <cfaulet@haproxy.com> 1604420290 +0100

BUG/MINOR: log: fix memory leak on logsrv parse error

In case of parsing error on logsrv, we can leave parse_logsrv() without
releasing logsrv->ring_name or smp_rgs. Let's free them on the error path.
This should fix issue #926 detected by Coverity.

The impact is only a tiny leak just before reporting a fatal error, so it
will essentially annoy valgrind.

This can be backported to 2.0 (just drop the ring part).

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