Fix several problems with start-stop-daemon, add -m support

This commit is contained in:
Eric Andersen 2004-04-13 18:28:46 +00:00
parent 3ddff210e6
commit 625da9d61e
2 changed files with 46 additions and 27 deletions

View File

@ -116,18 +116,18 @@ check(int pid)
static void static void
do_pidfile(const char *name) do_pidfile(void)
{ {
FILE *f; FILE *f;
pid_t pid; pid_t pid;
f = fopen(name, "r"); f = fopen(pidfile, "r");
if (f) { if (f) {
if (fscanf(f, "%d", &pid) == 1) if (fscanf(f, "%d", &pid) == 1)
check(pid); check(pid);
fclose(f); fclose(f);
} else if (errno != ENOENT) } else if (errno != ENOENT)
bb_perror_msg_and_die("open pidfile %s", name); bb_perror_msg_and_die("open pidfile %s", pidfile);
} }
@ -138,6 +138,11 @@ do_procinit(void)
struct dirent *entry; struct dirent *entry;
int foundany, pid; int foundany, pid;
if (pidfile) {
do_pidfile();
return;
}
procdir = opendir("/proc"); procdir = opendir("/proc");
if (!procdir) if (!procdir)
bb_perror_msg_and_die ("opendir /proc"); bb_perror_msg_and_die ("opendir /proc");
@ -162,15 +167,14 @@ do_stop(void)
struct pid_list *p; struct pid_list *p;
int killed = 0; int killed = 0;
if (pidfile) do_procinit();
do_pidfile(pidfile);
else
do_procinit();
if (cmdname) if (cmdname)
strcpy(what, cmdname); strcpy(what, cmdname);
else if (execname) else if (execname)
strcpy(what, execname); strcpy(what, execname);
else if (pidfile)
sprintf(what, "process in pidfile `%.200s'", pidfile);
else if (userspec) else if (userspec)
sprintf(what, "process(es) owned by `%s'", userspec); sprintf(what, "process(es) owned by `%s'", userspec);
else else
@ -186,7 +190,7 @@ do_stop(void)
p->pid = -p->pid; p->pid = -p->pid;
killed++; killed++;
} else { } else {
bb_perror_msg("warning: failed to kill %d:", p->pid); bb_perror_msg("warning: failed to kill %d", p->pid);
} }
} }
if (!quiet && killed) { if (!quiet && killed) {
@ -200,23 +204,25 @@ do_stop(void)
static const struct option ssd_long_options[] = { static const struct option ssd_long_options[] = {
{ "stop", 0, NULL, 'K' }, { "stop", 0, NULL, 'K' },
{ "start", 0, NULL, 'S' }, { "start", 0, NULL, 'S' },
{ "background", 0, NULL, 'b' }, { "background", 0, NULL, 'b' },
{ "quiet", 0, NULL, 'q' }, { "quiet", 0, NULL, 'q' },
{ "startas", 1, NULL, 'a' }, { "make-pidfile", 0, NULL, 'm' },
{ "name", 1, NULL, 'n' }, { "startas", 1, NULL, 'a' },
{ "signal", 1, NULL, 's' }, { "name", 1, NULL, 'n' },
{ "user", 1, NULL, 'u' }, { "signal", 1, NULL, 's' },
{ "exec", 1, NULL, 'x' }, { "user", 1, NULL, 'u' },
{ "pidfile", 1, NULL, 'p' }, { "exec", 1, NULL, 'x' },
{ "pidfile", 1, NULL, 'p' },
{ 0, 0, 0, 0 } { 0, 0, 0, 0 }
}; };
#define SSD_CTX_STOP 1 #define SSD_CTX_STOP 1
#define SSD_CTX_START 2 #define SSD_CTX_START 2
#define SSD_OPT_BACKGROUND 4 #define SSD_OPT_BACKGROUND 4
#define SSD_OPT_QUIET 8 #define SSD_OPT_QUIET 8
#define SSD_OPT_MAKEPID 16
int int
start_stop_daemon_main(int argc, char **argv) start_stop_daemon_main(int argc, char **argv)
@ -228,7 +234,7 @@ start_stop_daemon_main(int argc, char **argv)
bb_applet_long_options = ssd_long_options; bb_applet_long_options = ssd_long_options;
bb_opt_complementaly = "K~S:S~K"; bb_opt_complementaly = "K~S:S~K";
opt = bb_getopt_ulflags(argc, argv, "KSbqa:n:s:u:x:p:", opt = bb_getopt_ulflags(argc, argv, "KSbqma:n:s:u:x:p:",
&startas, &cmdname, &signame, &userspec, &execname, &pidfile); &startas, &cmdname, &signame, &userspec, &execname, &pidfile);
/* Check one and only one context option was given */ /* Check one and only one context option was given */
@ -240,8 +246,8 @@ start_stop_daemon_main(int argc, char **argv)
signal_nr = bb_xgetlarg(signame, 10, 0, NSIG); signal_nr = bb_xgetlarg(signame, 10, 0, NSIG);
} }
if (!execname && !userspec) if (!execname && !pidfile && !userspec && !cmdname)
bb_error_msg_and_die ("need at least one of -x or -u"); bb_error_msg_and_die ("need at least one of -x, -p, -u, or -n");
if (!startas) if (!startas)
startas = execname; startas = execname;
@ -249,19 +255,22 @@ start_stop_daemon_main(int argc, char **argv)
if ((opt & SSD_CTX_START) && !startas) if ((opt & SSD_CTX_START) && !startas)
bb_error_msg_and_die ("-S needs -x or -a"); bb_error_msg_and_die ("-S needs -x or -a");
if ((opt & SSD_OPT_MAKEPID) && pidfile == NULL)
bb_error_msg_and_die ("-m needs -p");
argc -= optind; argc -= optind;
argv += optind; argv += optind;
if (userspec && sscanf(userspec, "%d", &user_id) != 1) if (userspec && sscanf(userspec, "%d", &user_id) != 1)
user_id = my_getpwnam(userspec); user_id = my_getpwnam(userspec);
do_procinit();
if (opt & SSD_CTX_STOP) { if (opt & SSD_CTX_STOP) {
do_stop(); do_stop();
return EXIT_SUCCESS; return EXIT_SUCCESS;
} }
do_procinit();
if (found) { if (found) {
if (!quiet) if (!quiet)
printf("%s already running.\n%d\n", execname ,found->pid); printf("%s already running.\n%d\n", execname ,found->pid);
@ -271,8 +280,17 @@ start_stop_daemon_main(int argc, char **argv)
if (opt & SSD_OPT_BACKGROUND) { if (opt & SSD_OPT_BACKGROUND) {
if (daemon(0, 0) == -1) if (daemon(0, 0) == -1)
bb_perror_msg_and_die ("unable to fork"); bb_perror_msg_and_die ("unable to fork");
setsid();
}
if (opt & SSD_OPT_MAKEPID) {
/* user wants _us_ to make the pidfile */
FILE *pidf = fopen(pidfile, "w");
pid_t pidt = getpid();
if (pidf == NULL)
bb_perror_msg_and_die("Unable to write pidfile '%s'", pidfile);
fprintf(pidf, "%d\n", pidt);
fclose(pidf);
} }
setsid();
execv(startas, argv); execv(startas, argv);
bb_perror_msg_and_die ("unable to start %s", startas); bb_perror_msg_and_die ("unable to start %s", startas);
} }

View File

@ -2230,6 +2230,7 @@
"\n\t-b|--background\t\t\tforce process into background"\ "\n\t-b|--background\t\t\tforce process into background"\
"\n\t-u|--user <username>|<uid>\tstop this user's processes"\ "\n\t-u|--user <username>|<uid>\tstop this user's processes"\
"\n\t-x|--exec <executable>\t\tprogram to either start or check"\ "\n\t-x|--exec <executable>\t\tprogram to either start or check"\
"\n\t-m|--make-pidfile <filename>\tcreate the -p file and enter pid in it"\
"\n\t-n|--name <process-name>\tstop processes with this name"\ "\n\t-n|--name <process-name>\tstop processes with this name"\
"\n\t-p|--pidfile <pid-file>\t\tsave or load pid using a pid-file"\ "\n\t-p|--pidfile <pid-file>\t\tsave or load pid using a pid-file"\
"\n\t-q|--quiet\t\t\tbe quiet" \ "\n\t-q|--quiet\t\t\tbe quiet" \