BUG/MINOR: ssl/ckch: always ha_freearray() the previous entry during parsing

The ckch_conf_parse() function is the generic function which parses
crt-store keywords from the crt-store section, and also from a
crt-list.

When having multiple time the same keyword, a leak of the previous
value happens. This patch ensure that the previous value is always
freed before overwriting it.

This is the same problem as the previous "BUG/MINOR: ssl/ckch: always
free() the previous entry during parsing" patch, however this one
applies on PARSE_TYPE_ARRAY_SUBSTR.

No backport needed.

diff --git a/src/ssl_ckch.c b/src/ssl_ckch.c
index cf1357168159b73b5654b79c21c51001067c9d37..a2e96dd36dd63dcd45196583fe1c61b5952dbb27 100644 (file)
--- a/src/ssl_ckch.c
+++ b/src/ssl_ckch.c
@@ -4884,7 +4884,7 @@ int ckch_conf_parse(char **args, int cur_arg, struct ckch_conf *f, int *found, c
                                char *b, *e;
 
                                /* split a string into substring split by colons */
-
+                               ha_freearray(t);
                                e = b = args[cur_arg + 1];
                                do {
                                        while (*e != ',' && *e != '\0')