emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

we need to daemonize *before* opening the watchdog device

Browse Source
This commit is contained in:
Mike Frysinger 2009-04-25 06:16:37 +00:00
parent 26cf2831d7
commit 6fb5f01c0b
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
miscutils/watchdog.c
View File

@ -45,6 +45,15 @@ int watchdog_main(int argc, char **argv)
opt_complementary = "=1"; /* must have exactly 1 argument */ opt_complementary = "=1"; /* must have exactly 1 argument */
opts = getopt32(argv, "Ft:T:", &st_arg, &ht_arg); opts = getopt32(argv, "Ft:T:", &st_arg, &ht_arg);
/* We need to daemonize *before* opening the watchdog as many drivers
* will only allow one process at a time to do so. Since daemonizing
* is not perfect (child may run before parent finishes exiting), we
* can't rely on parent exiting before us (let alone *cleanly* releasing
* the watchdog fd -- something else that may not even be allowed).
*/
if (!(opts & OPT_FOREGROUND))
bb_daemonize_or_rexec(DAEMON_CHDIR_ROOT, argv);
if (opts & OPT_HTIMER) if (opts & OPT_HTIMER)
htimer_duration = xatou_sfx(ht_arg, suffixes); htimer_duration = xatou_sfx(ht_arg, suffixes);
stimer_duration = htimer_duration / 2; stimer_duration = htimer_duration / 2;
@ -76,10 +85,6 @@ int watchdog_main(int argc, char **argv)
stimer_duration, htimer_duration * 1000); stimer_duration, htimer_duration * 1000);
#endif #endif
if (!(opts & OPT_FOREGROUND)) {
bb_daemonize_or_rexec(DAEMON_CHDIR_ROOT, argv);
}
while (1) { while (1) {
/* /*
* Make sure we clear the counter before sleeping, * Make sure we clear the counter before sleeping,