BUG/MEDIUM: mworker: does not deinit anymore

Does not use the deinit() function during a reload, it's dangerous and
might be subject to double free, segfault and hazardous behavior if
it's called twice in the case of a execvp fail.

diff --git a/src/haproxy.c b/src/haproxy.c
index b90e2540ba9b48e6e51d8e6d851c6cc06660cdfb..00cc25b165a220916eac201fc12188aa129195d4 100644 (file)
--- a/src/haproxy.c
+++ b/src/haproxy.c
@@ -648,7 +648,6 @@ static void mworker_reload()
                next_argv[next_argc++] = NULL;
        }
 
-       deinit(); /* we don't want to leak FD there */
        Warning("Reexecuting Master process\n");
        execvp(next_argv[0], next_argv);