diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-04-19 20:19:45 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-04-19 20:19:45 +0000 |
commit | daeddee442e1cd7be15da6be472a637b8547fde7 (patch) | |
tree | 68e119fc56ff548e68830747f08989cf2077c8da | |
parent | f3745ea489c5ef454e2ce68926c5f39f5b30f240 (diff) | |
download | busybox-daeddee442e1cd7be15da6be472a637b8547fde7.tar.gz |
start_stop_daemon: do not stop /proc scan prematurely
function old new delta
do_procinit 185 196 +11
-rw-r--r-- | debianutils/start_stop_daemon.c | 17 |
1 files changed, 12 insertions, 5 deletions
diff --git a/debianutils/start_stop_daemon.c b/debianutils/start_stop_daemon.c index 2f8e04a78..b6b30e7cc 100644 --- a/debianutils/start_stop_daemon.c +++ b/debianutils/start_stop_daemon.c @@ -145,11 +145,18 @@ static void do_procinit(void) procdir = xopendir("/proc"); pid = 0; - while ((entry = readdir(procdir)) != NULL) { - pid = bb_strtou(entry->d_name, NULL, 10); - if (errno) - continue; - check(pid); + while(1) { + errno = 0; /* clear any previous error */ + entry = readdir(procdir); +// TODO: check for exact errno(s) which mean that we got stale entry + if (errno) /* Stale entry, process has died after opendir */ + continue; + if (!entry) /* EOF, no more entries */ + break; + pid = bb_strtou(entry->d_name, NULL, 10); + if (errno) /* NaN */ + continue; + check(pid); } closedir(procdir); if (!pid) |