BUG/MINOR: cfgparse-tcp: leak when re-declaring interface from bind line

When interface keyword is used multiple times within the same bind line,
the previous value isn't checked and is rewritten as-is, resulting in a
small memory leak.

Ensuring the interface name is first freed before assigning it to a new
value.

This may be backported to every stable versions.

[Note for 2.2, the fix must be performed in bind_parse_interface() from
proto_tcp.c, directly within the listener's loop, also ha_free() was
not available so free() must be used instead]

(cherry picked from commit c49224a29df65d81399058d2bb6cd36056915239)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 4c6adb04601306152d99c9831030804c599e1fbf)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 3455a03b7bb75ff1bdd88197247b144c4f106f4d)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit e8998a8be8424a905ad13e581b6a08abfa754bec)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
1 file changed