watchdog: always close fd on watchdog stop
authorDragan Milenkovic <strpbrk@tesla.rcub.bg.ac.rs>
Mon, 1 May 2023 11:11:02 +0000 (13:11 +0200)
committerDaniel Golle <daniel@makrotopia.org>
Sat, 30 Mar 2024 12:55:27 +0000 (12:55 +0000)
commit254810d16cf18cb19ca5766dc6fe967b7015722f
tree8e820e5f5cf6357b9ee39a4cd3a26fdf71f7b83c
parentca8c30208d5e1aaa2c0e3f732c4c9944735e9850
watchdog: always close fd on watchdog stop

The user may asks for procd watchdog handler to be stopped
with or without disabling it, by specifying the magicclose flag.
If the flag is set, the watchdog will be disabled and the fd closed,
allowing the user to take control over the watchdog.

There is a race in this scenario. If the system fails before
the user re-enables the watchdog, the system might hang
without a proper reset.

To prevent this, the user should ask the procd handler to be stopped
without disabling the watchdog, by specifying magicclose as false.
However, in this case, the procd will only stop refreshing the watchdog,
but will leave the fd open. At least on Raspberry Pi, this prevents
anyone else from opening the watchdog device, resulting in EBUSY.

With this patch, watchdog fd will always be closed, regardless
of the magicclose flag, allowing for the described safe use-case.

For user that previously stopped the watchdog handler
with the magicclose flag, the functionality remains unchanged.

Signed-off-by: Dragan Milenkovic <strpbrk@tesla.rcub.bg.ac.rs>
watchdog.c