aboutsummaryrefslogtreecommitdiff
path: root/debianutils
diff options
context:
space:
mode:
authorGravatar Denis Vlasenko <vda.linux@googlemail.com>2008-05-18 21:17:52 +0000
committerGravatar Denis Vlasenko <vda.linux@googlemail.com>2008-05-18 21:17:52 +0000
commite125a683a77d14401644d15f38b7f578db723924 (patch)
tree2beb60ca4b40bb96c017c02d4d9cefff85df1db9 /debianutils
parentfaf334aeb87ccc6680987bbd1fae8faf81969fed (diff)
downloadbusybox-e125a683a77d14401644d15f38b7f578db723924.tar.gz
busybox-e125a683a77d14401644d15f38b7f578db723924.tar.bz2
start_stop_daemon: add -test, fix -x to not match by inode,
vastly improve hext text. (mostly by Roy Marples <roy AT marples.name>) function old new delta packed_usage 24124 24190 +66 start_stop_daemon_main 959 991 +32 start_stop_daemon_longopts 149 156 +7 check 1632 1589 -43 ------------------------------------------------------------------------------ (add/remove: 0/0 grow/shrink: 3/1 up/down: 105/-43) Total: 62 bytes
Diffstat (limited to 'debianutils')
-rw-r--r--debianutils/start_stop_daemon.c134
1 files changed, 86 insertions, 48 deletions
diff --git a/debianutils/start_stop_daemon.c b/debianutils/start_stop_daemon.c
index 094b3b18a..524259979 100644
--- a/debianutils/start_stop_daemon.c
+++ b/debianutils/start_stop_daemon.c
@@ -23,15 +23,18 @@ If --start if given, start a new process unless a matching process was found.
Options controlling process matching:
-u,--user USERNAME|UID Only consider this user's processes
- -n,--name PROCESS_NAME Look for processes with matching argv[0]
- or /proc/$PID/exe or /proc/$PID/stat (comm field).
+ -n,--name PROCESS_NAME Look for processes by matching PROCESS_NAME
+ with comm field in /proc/$PID/stat.
Only basename is compared:
"ntpd" == "./ntpd" == "/path/to/ntpd".
[TODO: can PROCESS_NAME be a full pathname? Should we require full match then
with /proc/$PID/exe or argv[0] (comm can't be matched, it never contains path)]
- -x,--exec EXECUTABLE Look for processes with matching /proc/$PID/exe.
- Match is performed using device+inode.
+ -x,--exec EXECUTABLE Look for processes that were started with this
+ command in /proc/$PID/cmdline.
+ Unlike -n, we match against the full path:
+ "ntpd" != "./ntpd" != "/path/to/ntpd"
-p,--pidfile PID_FILE Look for processes with PID from this file
+If multiple conditions are specified, all must match.
Options which are valid for --start only:
-x,--exec EXECUTABLE Program to run (1st arg of execvp). Mandatory.
@@ -41,8 +44,13 @@ Options which are valid for --start only:
-c,--chuid USER[:[GRP]] Change to specified user [and group]
-m,--make-pidfile Write PID to the pidfile
(both -m and -p must be given!)
-Misc options:
+
+Options which are valid for --stop only:
-s,--signal SIG Signal to send (default:TERM)
+ -t,--test Exit with status 0 if process is found
+ (we don't actually start or stop daemons)
+
+Misc options:
-o,--oknodo Exit with status 0 if nothing is done
-q,--quiet Quiet
-v,--verbose Verbose
@@ -64,19 +72,21 @@ enum {
CTX_START = (1 << 1),
OPT_BACKGROUND = (1 << 2), // -b
OPT_QUIET = (1 << 3), // -q
- OPT_MAKEPID = (1 << 4), // -m
- OPT_a = (1 << 5), // -a
- OPT_n = (1 << 6), // -n
- OPT_s = (1 << 7), // -s
- OPT_u = (1 << 8), // -u
- OPT_c = (1 << 9), // -c
- OPT_x = (1 << 10), // -x
- OPT_p = (1 << 11), // -p
- OPT_OKNODO = (1 << 12) * ENABLE_FEATURE_START_STOP_DAEMON_FANCY, // -o
- OPT_VERBOSE = (1 << 13) * ENABLE_FEATURE_START_STOP_DAEMON_FANCY, // -v
- OPT_NICELEVEL = (1 << 14) * ENABLE_FEATURE_START_STOP_DAEMON_FANCY, // -N
+ OPT_TEST = (1 << 4), // -t
+ OPT_MAKEPID = (1 << 5), // -m
+ OPT_a = (1 << 6), // -a
+ OPT_n = (1 << 7), // -n
+ OPT_s = (1 << 8), // -s
+ OPT_u = (1 << 9), // -u
+ OPT_c = (1 << 10), // -c
+ OPT_x = (1 << 11), // -x
+ OPT_p = (1 << 12), // -p
+ OPT_OKNODO = (1 << 13) * ENABLE_FEATURE_START_STOP_DAEMON_FANCY, // -o
+ OPT_VERBOSE = (1 << 14) * ENABLE_FEATURE_START_STOP_DAEMON_FANCY, // -v
+ OPT_NICELEVEL = (1 << 15) * ENABLE_FEATURE_START_STOP_DAEMON_FANCY, // -N
};
#define QUIET (option_mask32 & OPT_QUIET)
+#define TEST (option_mask32 & OPT_TEST)
struct globals {
struct pid_list *found;
@@ -86,7 +96,6 @@ struct globals {
char *pidfile;
int user_id;
smallint signal_nr;
- struct stat execstat;
};
#define G (*(struct globals*)&bb_common_bufsiz1)
#define found (G.found )
@@ -96,20 +105,23 @@ struct globals {
#define pidfile (G.pidfile )
#define user_id (G.user_id )
#define signal_nr (G.signal_nr )
-#define execstat (G.execstat )
#define INIT_G() \
do { \
user_id = -1; \
signal_nr = 15; \
} while (0)
-
+#ifdef OLDER_VERSION_OF_X
+/* -x,--exec EXECUTABLE
+ * Look for processes with matching /proc/$PID/exe.
+ * Match is performed using device+inode.
+ */
static int pid_is_exec(pid_t pid)
{
struct stat st;
char buf[sizeof("/proc//exe") + sizeof(int)*3];
- sprintf(buf, "/proc/%u/exe", pid);
+ sprintf(buf, "/proc/%u/exe", (unsigned)pid);
if (stat(buf, &st) < 0)
return 0;
if (st.st_dev == execstat.st_dev
@@ -117,24 +129,29 @@ static int pid_is_exec(pid_t pid)
return 1;
return 0;
}
+#endif
-static int pid_is_user(int pid)
+static int pid_is_exec(pid_t pid)
{
- struct stat sb;
- char buf[sizeof("/proc/") + sizeof(int)*3];
-
- sprintf(buf, "/proc/%u", pid);
- if (stat(buf, &sb) != 0)
- return 0;
- return (sb.st_uid == (uid_t)user_id);
+ ssize_t bytes;
+ char buf[PATH_MAX];
+
+ sprintf(buf, "/proc/%u/cmdline", (unsigned)pid);
+ bytes = open_read_close(buf, buf, sizeof(buf) - 1);
+ if (bytes > 0) {
+ buf[bytes] = '\0';
+ return strcmp(buf, execname) == 0;
+ }
+ return 0;
}
-static int pid_is_cmd(pid_t pid)
+static int pid_is_name(pid_t pid)
{
- char buf[256]; /* is it big enough? */
+ /* /proc/PID/stat is "PID (comm_15_bytes_max) ..." */
+ char buf[32]; /* should be enough */
char *p, *pe;
- sprintf(buf, "/proc/%u/stat", pid);
+ sprintf(buf, "/proc/%u/stat", (unsigned)pid);
if (open_read_close(buf, buf, sizeof(buf) - 1) < 0)
return 0;
buf[sizeof(buf) - 1] = '\0'; /* paranoia */
@@ -145,7 +162,23 @@ static int pid_is_cmd(pid_t pid)
if (!pe)
return 0;
*pe = '\0';
- return !strcmp(p, cmdname);
+ /* we require comm to match and to not be truncated */
+ /* in Linux, if comm is 15 chars, it may be a truncated
+ * name, so we don't allow that to match */
+ if (strlen(p) >= COMM_LEN - 1) /* COMM_LEN is 16 */
+ return 0;
+ return strcmp(p, cmdname) == 0;
+}
+
+static int pid_is_user(int pid)
+{
+ struct stat sb;
+ char buf[sizeof("/proc/") + sizeof(int)*3];
+
+ sprintf(buf, "/proc/%u", (unsigned)pid);
+ if (stat(buf, &sb) != 0)
+ return 0;
+ return (sb.st_uid == (uid_t)user_id);
}
static void check(int pid)
@@ -155,10 +188,10 @@ static void check(int pid)
if (execname && !pid_is_exec(pid)) {
return;
}
- if (userspec && !pid_is_user(pid)) {
+ if (cmdname && !pid_is_name(pid)) {
return;
}
- if (cmdname && !pid_is_cmd(pid)) {
+ if (userspec && !pid_is_user(pid)) {
return;
}
p = xmalloc(sizeof(*p));
@@ -195,10 +228,11 @@ static void do_procinit(void)
procdir = xopendir("/proc");
pid = 0;
- while(1) {
+ while (1) {
errno = 0; /* clear any previous error */
entry = readdir(procdir);
-// TODO: check for exact errno(s) which mean that we got stale entry
+// TODO: this check is too generic, it's better
+// to 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 */
@@ -225,12 +259,13 @@ static int do_stop(void)
} else if (execname) {
if (ENABLE_FEATURE_CLEAN_UP) what = xstrdup(execname);
if (!ENABLE_FEATURE_CLEAN_UP) what = execname;
- } else if (pidfile)
+ } else if (pidfile) {
what = xasprintf("process in pidfile '%s'", pidfile);
- else if (userspec)
+ } else if (userspec) {
what = xasprintf("process(es) owned by '%s'", userspec);
- else
+ } else {
bb_error_msg_and_die("internal error, please report");
+ }
if (!found) {
if (!QUIET)
@@ -239,18 +274,18 @@ static int do_stop(void)
goto ret;
}
for (p = found; p; p = p->next) {
- if (kill(p->pid, signal_nr) == 0) {
- p->pid = - p->pid;
+ if (TEST || kill(p->pid, signal_nr) == 0) {
killed++;
} else {
- bb_perror_msg("warning: killing process %u", p->pid);
+ p->pid = 0;
+ bb_perror_msg("warning: killing process %u", (unsigned)p->pid);
}
}
if (!QUIET && killed) {
printf("stopped %s (pid", what);
for (p = found; p; p = p->next)
- if (p->pid < 0)
- printf(" %u", - p->pid);
+ if (p->pid)
+ printf(" %u", (unsigned)p->pid);
puts(")");
}
ret:
@@ -265,6 +300,7 @@ static const char start_stop_daemon_longopts[] ALIGN1 =
"start\0" No_argument "S"
"background\0" No_argument "b"
"quiet\0" No_argument "q"
+ "test\0" No_argument "t"
"make-pidfile\0" No_argument "m"
#if ENABLE_FEATURE_START_STOP_DAEMON_FANCY
"oknodo\0" No_argument "o"
@@ -291,6 +327,7 @@ int start_stop_daemon_main(int argc ATTRIBUTE_UNUSED, char **argv)
char *signame;
char *startas;
char *chuid;
+ struct stat execstat;
#if ENABLE_FEATURE_START_STOP_DAEMON_FANCY
// char *retry_arg = NULL;
// int retries = -1;
@@ -310,7 +347,7 @@ int start_stop_daemon_main(int argc ATTRIBUTE_UNUSED, char **argv)
/* -q turns off -v */
opt_complementary = "K:S:K--S:S--K:m?p:K?xpun:S?xa"
USE_FEATURE_START_STOP_DAEMON_FANCY("q-v");
- opt = getopt32(argv, "KSbqma:n:s:u:c:x:p:"
+ opt = getopt32(argv, "KSbqtma:n:s:u:c:x:p:"
USE_FEATURE_START_STOP_DAEMON_FANCY("ovN:"),
// USE_FEATURE_START_STOP_DAEMON_FANCY("ovN:R:"),
&startas, &cmdname, &signame, &userspec, &chuid, &execname, &pidfile
@@ -338,9 +375,6 @@ int start_stop_daemon_main(int argc ATTRIBUTE_UNUSED, char **argv)
if (errno)
user_id = xuname2uid(userspec);
}
- if (execname)
- xstat(execname, &execstat);
-
do_procinit(); /* Both start and stop needs to know current processes */
if (opt & CTX_STOP) {
@@ -353,6 +387,10 @@ int start_stop_daemon_main(int argc ATTRIBUTE_UNUSED, char **argv)
printf("%s already running\n%d\n", execname, found->pid);
return !(opt & OPT_OKNODO);
}
+
+ if (execname)
+ xstat(execname, &execstat);
+
*--argv = startas;
if (opt & OPT_BACKGROUND) {
#if BB_MMU