alfred: Fix procd process handling for disable state
authorSven Eckelmann <sven@narfation.org>
Mon, 15 Feb 2021 20:03:57 +0000 (21:03 +0100)
committerSven Eckelmann <sven@narfation.org>
Mon, 15 Feb 2021 20:28:09 +0000 (21:28 +0100)
commit97e760095578a8a1491f59423c712c7dc7b75a1f
tree4246efa093ebf97387cec7fb707c32b877a3db92
parent596dc84b65ac2a4c29f6e7ad0ff2389843801abc
alfred: Fix procd process handling for disable state

Alfred can be stopped during a (re)start or reload when:

* disabled = 1
* batman-adv interface is missing
* MAC based EUI64 IPv6 link-local address not set

It is assumed that procd detects that the procd instance definition
disappeared and the currently running instance must be stopped. But all
these checks were triggering an "exit" which then stopped the alfred init
script and the wrapper from /lib/functions/procd.sh. And thus procd was
never informed about the instances which should be changed.

The correct handling is to return the result of the function instead of
directly killing the init scripts.

Fixes: 45db0e60d2ff ("alfred: use procd")
Signed-off-by: Sven Eckelmann <sven@narfation.org>
alfred/Makefile
alfred/files/alfred.init