state: fix reboot causing shutdown inside LXC container
authorPetr Štetiar <ynezz@true.cz>
Wed, 15 Jan 2020 19:28:38 +0000 (20:28 +0100)
committerPetr Štetiar <ynezz@true.cz>
Fri, 24 Jan 2020 07:31:55 +0000 (08:31 +0100)
commit31e4b2dfdbd74f7451abf876dd5d9a674af7adfd
treedddee27fb94d40019675be2615627e375bdd4775
parent557f11b3a20f2cf6897aad067ffde9d92e9fe015
state: fix reboot causing shutdown inside LXC container

Executing `reboot` command in OpenWrt system runing inside LXC container
results in a shutdown of the container instead of rebooting the
container.

This appears to have been caused by commit 832369078d81 ("state: fix
shutdown when running in a container (FS#2425)"), which exits the pid
einz instead of the reboot().

While at it, refactor the halting code into separate function to shorten
the switch/case block and make it clearer, decrease the indentation
level by reversing the container if condition, replace magic 0 with
EXIT_SUCCESS constant in exit() and make it wait 1s for reboot message
delivery in both container/host cases as well.

Ref: FS#2666
Cc: Paul Spooren <mail@aparcar.org>
Fixes: 832369078d81 ("state: fix shutdown when running in a container (FS#2425)")
Tested-by: Baptiste Jonglez <lede@bitsofnetworks.org>
Signed-off-by: Petr Štetiar <ynezz@true.cz>
state.c