BUG/MINOR: mworker/init: don't reset nb_oldpids in non-mworker cases
authorWilliam Lallemand <wlallemand@haproxy.org>
Wed, 21 Apr 2021 14:42:18 +0000 (16:42 +0200)
committerWilliam Lallemand <wlallemand@haproxy.org>
Wed, 21 Apr 2021 14:42:18 +0000 (16:42 +0200)
commitea6bf83d621ceeaf3f7e7de59b45fa00940b784d
tree0502bb6bd68197a1705b75f74ae65e01387ef043
parenta2944ecf5d3fd28fe1b7405840a068732bfa035c
BUG/MINOR: mworker/init: don't reset nb_oldpids in non-mworker cases

This bug affects the peers synchronisation code which rely on the
nb_oldpids variable to synchronize the peer from the old PID.

In the case the process is not started in master-worker mode and tries
to synchronize using the peers, there is a small chance that won't work
because nb_oldpids equals 0.

Fix the bug by setting the variable to 0 only in the case of the
master-worker when not reloaded.

It could also be a problem when trying to synchronize the peers between
2 masters process which should be fixed in another patch.

Bug exists since commit 8a361b5 ("BUG/MEDIUM: mworker: don't reuse PIDs
passed to the master").

Sould be backported as far as 1.8.
src/haproxy.c