diff options
author | Glenn L McGrath <bug1@ihug.co.nz> | 2004-01-22 12:04:58 +0300 |
---|---|---|
committer | Glenn L McGrath <bug1@ihug.co.nz> | 2004-01-22 12:04:58 +0300 |
commit | 8d441783817d909b7ba3c0148d7deb121e9791c8 (patch) | |
tree | 87e739be605dfb6660cb5e25124728c95f37eb63 /debianutils | |
parent | 85c5152cb874fdac2e59072b6d958af18965182a (diff) |
Check one and only one of start, stop are given.
Remove some global variables.
#define some getopt values.
Diffstat (limited to 'debianutils')
-rw-r--r-- | debianutils/start_stop_daemon.c | 34 |
1 files changed, 17 insertions, 17 deletions
diff --git a/debianutils/start_stop_daemon.c b/debianutils/start_stop_daemon.c index c31bba6dd..082fe6070 100644 --- a/debianutils/start_stop_daemon.c +++ b/debianutils/start_stop_daemon.c @@ -21,15 +21,11 @@ #include "busybox.h" #include "pwd_.h" -static int start = 0; -static int stop = 0; -static int fork_before_exec = 0; static int signal_nr = 15; static int user_id = -1; static char *userspec = NULL; static char *cmdname = NULL; static char *execname = NULL; -static char *startas = NULL; typedef struct pid_list { struct pid_list *next; @@ -195,35 +191,40 @@ static const struct option ssd_long_options[] = { { 0, 0, 0, 0 } }; +#define SSD_CTX_STOP 1 +#define SSD_CTX_START 2 +#define SSD_OPT_BACKGROUND 4 + int start_stop_daemon_main(int argc, char **argv) { - int flags; + unsigned long opt; char *signame = NULL; + char *startas = NULL; + bb_applet_long_options = ssd_long_options; - flags = bb_getopt_ulflags(argc, argv, "KSba:n:s:u:x:", + bb_opt_complementaly = "K~S"; + opt = bb_getopt_ulflags(argc, argv, "KSba:n:s:u:x:", &startas, &cmdname, &signame, &userspec, &execname); - /* Be sneaky and avoid branching */ - stop = (flags & 1); - start = (flags & 2); - fork_before_exec = (flags & 4); + /* Check one and only one context option was given */ + if ((opt & 0x80000000UL) || + (opt & (SSD_CTX_STOP | SSD_CTX_START)) == 0) { + bb_show_usage(); + } if (signame) { signal_nr = bb_xgetlarg(signame, 10, 0, NSIG); } - if (start == stop) - bb_error_msg_and_die ("need exactly one of -S or -K"); - if (!execname && !userspec) bb_error_msg_and_die ("need at least one of -x or -u"); if (!startas) startas = execname; - if (start && !startas) + if ((opt & SSD_CTX_START) && !startas) bb_error_msg_and_die ("-S needs -x or -a"); argc -= optind; @@ -234,7 +235,7 @@ start_stop_daemon_main(int argc, char **argv) do_procfs(); - if (stop) { + if (opt & SSD_CTX_STOP) { do_stop(); return EXIT_SUCCESS; } @@ -244,7 +245,7 @@ start_stop_daemon_main(int argc, char **argv) return EXIT_SUCCESS; } *--argv = startas; - if (fork_before_exec) { + if (opt & SSD_OPT_BACKGROUND) { if (daemon(0, 0) == -1) bb_perror_msg_and_die ("unable to fork"); } @@ -252,4 +253,3 @@ start_stop_daemon_main(int argc, char **argv) execv(startas, argv); bb_perror_msg_and_die ("unable to start %s", startas); } - |