BUG/MINOR: mworker: don't use oldpids[] anymore for reload

Since commit 3f12887 ("MINOR: mworker: don't use children variable
anymore"), the oldpids array is not used anymore to generate the new -sf
parameters. So we don't need to set nb_oldpids to 0 during the first
start of the master process.

This patch fixes a bug when 2 masters process tries to synchronize their
peers, there is a small chances that it won't work because nb_oldpids
equals 0.

Should be backported as far as 2.0.

(cherry picked from commit aba7f8b3132de3b29efe634cad8fb574f259549b)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
diff --git a/src/haproxy.c b/src/haproxy.c
index 55ef7b6..62da462 100644
--- a/src/haproxy.c
+++ b/src/haproxy.c
@@ -802,7 +802,7 @@
 		old_argc++;
 
 	/* 1 for haproxy -sf, 2 for -x /socket */
-	next_argv = calloc(old_argc + 1 + 2 + mworker_child_nb() + nb_oldpids + 1,
+	next_argv = calloc(old_argc + 1 + 2 + mworker_child_nb() + 1,
 			   sizeof(*next_argv));
 	if (next_argv == NULL)
 		goto alloc_error;
@@ -3317,13 +3317,6 @@
 	if ((global.mode & MODE_MWORKER) && !(global.mode & MODE_MWORKER_WAIT))
 		mworker_kill_max_reloads(SIGTERM);
 
-	if ((global.mode & MODE_MWORKER) && (getenv("HAPROXY_MWORKER_REEXEC") == NULL)) {
-		nb_oldpids = 0;
-		free(oldpids);
-		oldpids = NULL;
-	}
-
-
 	/* Note that any error at this stage will be fatal because we will not
 	 * be able to restart the old pids.
 	 */