Hi

Am 08.11.18 um 00:32 schrieb Tim Duesterhus:
>       if (!str2listener(path, global.stats_fe, bind_conf, "master-socket", 0, 
> &err)) {

I just notice that `err` in `mworker_cli_sockpair_new` should probably
be freed as well. Valgrind did not report this, because I did not have
any errors. Can you make the necessary adjustments, please? There's
possibly even more strings that leak.

I suggest to run haproxy with valgrind yourself. There's a bit of
"possibly lost" memory as well. I used:

valgrind --leak-check=full ./haproxy -d -Sa /scratch/haproxy/cli.sock
-Ws -f ./haproxy.cfg

with an empty configuration file to find the issues my patch fixes.

Best regards
Tim Düsterhus

Reply via email to