diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2008-04-20 01:30:52 +0400 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2008-04-20 01:30:52 +0400 |
commit | 85d788e491a1678f5aa5eb0392445c61b218e19b (patch) | |
tree | 0e39571b59b8e8d4c5f7070565a734870d9e62e0 | |
parent | daeddee442e1cd7be15da6be472a637b8547fde7 (diff) |
start_stop_daemon: optimization - do not pass paramenter, use
existing globals instead
function old new delta
start_stop_daemon_main 792 976 +184
check 1618 1620 +2
do_procinit 196 - -196
------------------------------------------------------------------------------
(add/remove: 0/1 grow/shrink: 2/0 up/down: 186/-196) Total: -10 bytes
-rw-r--r-- | debianutils/start_stop_daemon.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/debianutils/start_stop_daemon.c b/debianutils/start_stop_daemon.c index b6b30e7cc..c5c4059a1 100644 --- a/debianutils/start_stop_daemon.c +++ b/debianutils/start_stop_daemon.c @@ -65,7 +65,7 @@ static int pid_is_exec(pid_t pid) return 0; } -static int pid_is_user(int pid, int uid) +static int pid_is_user(int pid) { struct stat sb; char buf[sizeof("/proc/") + sizeof(int)*3]; @@ -73,10 +73,10 @@ static int pid_is_user(int pid, int uid) sprintf(buf, "/proc/%u", pid); if (stat(buf, &sb) != 0) return 0; - return (sb.st_uid == uid); + return (sb.st_uid == user_id); } -static int pid_is_cmd(pid_t pid, const char *name) +static int pid_is_cmd(pid_t pid) { char fname[sizeof("/proc//stat") + sizeof(int)*3]; char *buf; @@ -90,7 +90,7 @@ static int pid_is_cmd(pid_t pid, const char *name) char *pe = strrchr(++p, ')'); if (pe) { *pe = '\0'; - r = !strcmp(p, name); + r = !strcmp(p, cmdname); } } free(buf); @@ -105,10 +105,10 @@ static void check(int pid) if (execname && !pid_is_exec(pid)) { return; } - if (userspec && !pid_is_user(pid, user_id)) { + if (userspec && !pid_is_user(pid)) { return; } - if (cmdname && !pid_is_cmd(pid, cmdname)) { + if (cmdname && !pid_is_cmd(pid)) { return; } p = xmalloc(sizeof(*p)); @@ -169,8 +169,6 @@ static int do_stop(void) struct pid_list *p; int killed = 0; - do_procinit(); - if (cmdname) { if (ENABLE_FEATURE_CLEAN_UP) what = xstrdup(cmdname); if (!ENABLE_FEATURE_CLEAN_UP) what = cmdname; @@ -308,13 +306,13 @@ int start_stop_daemon_main(int argc ATTRIBUTE_UNUSED, char **argv) if (execname) xstat(execname, &execstat); + do_procinit(); /* Both start and stop needs to know current processes */ + if (opt & CTX_STOP) { int i = do_stop(); return (opt & OPT_OKNODO) ? 0 : (i <= 0); } - do_procinit(); - if (found) { if (!quiet) printf("%s already running\n%d\n", execname, found->pid); |