BUG/MINOR: snapshots: leak of snapshots on deinit()

Free the snapshots on deinit() when they were initialized in a proxy
upon an error.

This was introduced by c55015e ("MEDIUM: snapshots: dynamically allocate
the snapshots").

Should be backported as far as 1.9.

diff --git a/src/haproxy.c b/src/haproxy.c
index c6f7394269c9dd216b1d429a18d1ecdd6e0e9564..e891eb058e65b3ae05a05b2a1758e22825a7a3cc 100644 (file)
--- a/src/haproxy.c
+++ b/src/haproxy.c
@@ -2620,6 +2620,8 @@ void deinit(void)
                free(p->capture_name);
                free(p->monitor_uri);
                free(p->rdp_cookie_name);
+               free(p->invalid_rep);
+               free(p->invalid_req);
                if (p->conf.logformat_string != default_http_log_format &&
                    p->conf.logformat_string != default_tcp_log_format &&
                    p->conf.logformat_string != clf_http_log_format)