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.
This commit is contained in:
William Lallemand 2017-11-15 19:02:57 +01:00 committed by Willy Tarreau
parent 2f8b31c2c6
commit fade49d8fb

View File

@ -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);