chiark
/
gitweb
/
~ianmdlvl
/
elogind.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
service: fix minor memory leak
[elogind.git]
/
src
/
execute.c
diff --git
a/src/execute.c
b/src/execute.c
index 28baeedcfc44c9ca3c359ef28ff26390a57dc198..0c2526b33e1423b35a834db1553d49be468aa0b3 100644
(file)
--- a/
src/execute.c
+++ b/
src/execute.c
@@
-197,7
+197,7
@@
static int connect_logger_as(const ExecContext *context, ExecOutput output, cons
output == EXEC_OUTPUT_KMSG ? "kmsg" : "syslog",
context->syslog_priority,
context->syslog_identifier ? context->syslog_identifier : ident,
output == EXEC_OUTPUT_KMSG ? "kmsg" : "syslog",
context->syslog_priority,
context->syslog_identifier ? context->syslog_identifier : ident,
-
!context->syslog_no
_prefix);
+
context->syslog_level
_prefix);
if (fd != nfd) {
r = dup2(fd, nfd) < 0 ? -errno : nfd;
if (fd != nfd) {
r = dup2(fd, nfd) < 0 ? -errno : nfd;
@@
-232,7
+232,10
@@
static bool is_terminal_input(ExecInput i) {
i == EXEC_INPUT_TTY_FAIL;
}
i == EXEC_INPUT_TTY_FAIL;
}
-static int fixup_input(ExecInput std_input, int socket_fd) {
+static int fixup_input(ExecInput std_input, int socket_fd, bool apply_tty_stdin) {
+
+ if (is_terminal_input(std_input) && !apply_tty_stdin)
+ return EXEC_INPUT_NULL;
if (std_input == EXEC_INPUT_SOCKET && socket_fd < 0)
return EXEC_INPUT_NULL;
if (std_input == EXEC_INPUT_SOCKET && socket_fd < 0)
return EXEC_INPUT_NULL;
@@
-248,12
+251,12
@@
static int fixup_output(ExecOutput std_output, int socket_fd) {
return std_output;
}
return std_output;
}
-static int setup_input(const ExecContext *context, int socket_fd) {
+static int setup_input(const ExecContext *context, int socket_fd
, bool apply_tty_stdin
) {
ExecInput i;
assert(context);
ExecInput i;
assert(context);
- i = fixup_input(context->std_input, socket_fd);
+ i = fixup_input(context->std_input, socket_fd
, apply_tty_stdin
);
switch (i) {
switch (i) {
@@
-289,14
+292,14
@@
static int setup_input(const ExecContext *context, int socket_fd) {
}
}
}
}
-static int setup_output(const ExecContext *context, int socket_fd, const char *ident) {
+static int setup_output(const ExecContext *context, int socket_fd, const char *ident
, bool apply_tty_stdin
) {
ExecOutput o;
ExecInput i;
assert(context);
assert(ident);
ExecOutput o;
ExecInput i;
assert(context);
assert(ident);
- i = fixup_input(context->std_input, socket_fd);
+ i = fixup_input(context->std_input, socket_fd
, apply_tty_stdin
);
o = fixup_output(context->std_output, socket_fd);
/* This expects the input is already set up */
o = fixup_output(context->std_output, socket_fd);
/* This expects the input is already set up */
@@
-305,16
+308,16
@@
static int setup_output(const ExecContext *context, int socket_fd, const char *i
case EXEC_OUTPUT_INHERIT:
case EXEC_OUTPUT_INHERIT:
- /* If the input is connected to a
termina
l, inherit that... */
+ /* If the input is connected to a
nything that's not a /dev/nul
l, inherit that... */
if (i != EXEC_INPUT_NULL)
return dup2(STDIN_FILENO, STDOUT_FILENO) < 0 ? -errno : STDOUT_FILENO;
if (i != EXEC_INPUT_NULL)
return dup2(STDIN_FILENO, STDOUT_FILENO) < 0 ? -errno : STDOUT_FILENO;
- /* For PID 1 stdout is always connected to /dev/null,
- * hence reopen the console if out parent is PID1. */
- if (getppid() == 1)
- return open_terminal_as(tty_path(context), O_WRONLY, STDOUT_FILENO);
+ /* If we are not started from PID 1 we just inherit STDOUT from our parent process. */
+ if (getppid() != 1)
+ return STDOUT_FILENO;
- return STDOUT_FILENO;
+ /* We need to open /dev/null here anew, to get the
+ * right access mode. So we fall through */
case EXEC_OUTPUT_NULL:
return open_null_as(O_WRONLY, STDOUT_FILENO);
case EXEC_OUTPUT_NULL:
return open_null_as(O_WRONLY, STDOUT_FILENO);
@@
-339,14
+342,14
@@
static int setup_output(const ExecContext *context, int socket_fd, const char *i
}
}
}
}
-static int setup_error(const ExecContext *context, int socket_fd, const char *ident) {
+static int setup_error(const ExecContext *context, int socket_fd, const char *ident
, bool apply_tty_stdin
) {
ExecOutput o, e;
ExecInput i;
assert(context);
assert(ident);
ExecOutput o, e;
ExecInput i;
assert(context);
assert(ident);
- i = fixup_input(context->std_input, socket_fd);
+ i = fixup_input(context->std_input, socket_fd
, apply_tty_stdin
);
o = fixup_output(context->std_output, socket_fd);
e = fixup_output(context->std_error, socket_fd);
o = fixup_output(context->std_output, socket_fd);
e = fixup_output(context->std_error, socket_fd);
@@
-356,7
+359,7
@@
static int setup_error(const ExecContext *context, int socket_fd, const char *id
* the way and are not on a tty */
if (e == EXEC_OUTPUT_INHERIT &&
o == EXEC_OUTPUT_INHERIT &&
* the way and are not on a tty */
if (e == EXEC_OUTPUT_INHERIT &&
o == EXEC_OUTPUT_INHERIT &&
- i
!
= EXEC_INPUT_NULL &&
+ i
=
= EXEC_INPUT_NULL &&
getppid () != 1)
return STDERR_FILENO;
getppid () != 1)
return STDERR_FILENO;
@@
-889,6
+892,7
@@
int exec_spawn(ExecCommand *command,
char **environment,
bool apply_permissions,
bool apply_chroot,
char **environment,
bool apply_permissions,
bool apply_chroot,
+ bool apply_tty_stdin,
bool confirm_spawn,
CGroupBonding *cgroup_bondings,
pid_t *ret) {
bool confirm_spawn,
CGroupBonding *cgroup_bondings,
pid_t *ret) {
@@
-939,7
+943,7
@@
int exec_spawn(ExecCommand *command,
const char *username = NULL, *home = NULL;
uid_t uid = (uid_t) -1;
gid_t gid = (gid_t) -1;
const char *username = NULL, *home = NULL;
uid_t uid = (uid_t) -1;
gid_t gid = (gid_t) -1;
- char **our_env = NULL, **pam_env = NULL, **final_env = NULL;
+ char **our_env = NULL, **pam_env = NULL, **final_env = NULL
, **final_argv = NULL
;
unsigned n_env = 0;
int saved_stdout = -1, saved_stdin = -1;
bool keep_stdout = false, keep_stdin = false;
unsigned n_env = 0;
int saved_stdout = -1, saved_stdin = -1;
bool keep_stdout = false, keep_stdin = false;
@@
-964,19
+968,30
@@
int exec_spawn(ExecCommand *command,
goto fail;
}
goto fail;
}
- if (!context->
no_setsid
)
+ if (!context->
same_pgrp
)
if (setsid() < 0) {
r = EXIT_SETSID;
goto fail;
}
if (setsid() < 0) {
r = EXIT_SETSID;
goto fail;
}
- if (socket_fd >= 0 && context->tcpwrap_name)
- if (!socket_tcpwrap(socket_fd, context->tcpwrap_name)) {
- r = EXIT_TCPWRAP;
- goto fail;
+ if (context->tcpwrap_name) {
+ if (socket_fd >= 0)
+ if (!socket_tcpwrap(socket_fd, context->tcpwrap_name)) {
+ r = EXIT_TCPWRAP;
+ goto fail;
+ }
+
+ for (i = 0; i < (int) n_fds; i++) {
+ if (!socket_tcpwrap(fds[i], context->tcpwrap_name)) {
+ r = EXIT_TCPWRAP;
+ goto fail;
+ }
}
}
+ }
- if (confirm_spawn) {
+ /* We skip the confirmation step if we shall not apply the TTY */
+ if (confirm_spawn &&
+ (!is_terminal_input(context->std_input) || apply_tty_stdin)) {
char response;
/* Set up terminal for the question */
char response;
/* Set up terminal for the question */
@@
-1009,18
+1024,18
@@
int exec_spawn(ExecCommand *command,
}
if (!keep_stdin)
}
if (!keep_stdin)
- if (setup_input(context, socket_fd) < 0) {
+ if (setup_input(context, socket_fd
, apply_tty_stdin
) < 0) {
r = EXIT_STDIN;
goto fail;
}
if (!keep_stdout)
r = EXIT_STDIN;
goto fail;
}
if (!keep_stdout)
- if (setup_output(context, socket_fd, file_name_from_path(command->path)) < 0) {
+ if (setup_output(context, socket_fd, file_name_from_path(command->path)
, apply_tty_stdin
) < 0) {
r = EXIT_STDOUT;
goto fail;
}
r = EXIT_STDOUT;
goto fail;
}
- if (setup_error(context, socket_fd, file_name_from_path(command->path)) < 0) {
+ if (setup_error(context, socket_fd, file_name_from_path(command->path)
, apply_tty_stdin
) < 0) {
r = EXIT_STDERR;
goto fail;
}
r = EXIT_STDERR;
goto fail;
}
@@
-1062,8
+1077,8
@@
int exec_spawn(ExecCommand *command,
}
}
}
}
- if (context->cpu
_affinity_
set)
- if (sched_setaffinity(0,
sizeof(context->cpu_affinity), &context->cpu_affinity
) < 0) {
+ if (context->cpuset)
+ if (sched_setaffinity(0,
CPU_ALLOC_SIZE(context->cpuset_ncpus), context->cpuset
) < 0) {
r = EXIT_CPUAFFINITY;
goto fail;
}
r = EXIT_CPUAFFINITY;
goto fail;
}
@@
-1074,8
+1089,8
@@
int exec_spawn(ExecCommand *command,
goto fail;
}
goto fail;
}
- if (context->timer_slack_ns_set)
- if (prctl(PR_SET_TIMERSLACK, context->timer_slack_ns
_set
) < 0) {
+ if (context->timer_slack_ns
ec
_set)
+ if (prctl(PR_SET_TIMERSLACK, context->timer_slack_ns
ec
) < 0) {
r = EXIT_TIMERSLACK;
goto fail;
}
r = EXIT_TIMERSLACK;
goto fail;
}
@@
-1213,7
+1228,7
@@
int exec_spawn(ExecCommand *command,
}
if (n_fds > 0)
}
if (n_fds > 0)
- if (asprintf(our_env + n_env++, "LISTEN_PID=%l
lu", (unsigned long
long) getpid()) < 0 ||
+ if (asprintf(our_env + n_env++, "LISTEN_PID=%l
u", (unsigned
long) getpid()) < 0 ||
asprintf(our_env + n_env++, "LISTEN_FDS=%u", n_fds) < 0) {
r = EXIT_MEMORY;
goto fail;
asprintf(our_env + n_env++, "LISTEN_FDS=%u", n_fds) < 0) {
r = EXIT_MEMORY;
goto fail;
@@
-1245,13
+1260,19
@@
int exec_spawn(ExecCommand *command,
goto fail;
}
goto fail;
}
- execve(command->path, argv, final_env);
+ if (!(final_argv = replace_env_argv(argv, final_env))) {
+ r = EXIT_MEMORY;
+ goto fail;
+ }
+
+ execve(command->path, final_argv, final_env);
r = EXIT_EXEC;
fail:
strv_free(our_env);
strv_free(final_env);
strv_free(pam_env);
r = EXIT_EXEC;
fail:
strv_free(our_env);
strv_free(final_env);
strv_free(pam_env);
+ strv_free(final_argv);
if (saved_stdin >= 0)
close_nointr_nofail(saved_stdin);
if (saved_stdin >= 0)
close_nointr_nofail(saved_stdin);
@@
-1270,10
+1291,9
@@
int exec_spawn(ExecCommand *command,
if (cgroup_bondings)
cgroup_bonding_install_list(cgroup_bondings, pid);
if (cgroup_bondings)
cgroup_bonding_install_list(cgroup_bondings, pid);
- log_debug("Forked %s as %l
lu", command->path, (unsigned long
long) pid);
+ log_debug("Forked %s as %l
u", command->path, (unsigned
long) pid);
- command->exec_status.pid = pid;
- command->exec_status.start_timestamp = now(CLOCK_REALTIME);
+ exec_status_start(&command->exec_status, pid);
*ret = pid;
return 0;
*ret = pid;
return 0;
@@
-1286,7
+1306,9
@@
void exec_context_init(ExecContext *c) {
c->ioprio = IOPRIO_PRIO_VALUE(IOPRIO_CLASS_BE, 0);
c->cpu_sched_policy = SCHED_OTHER;
c->syslog_priority = LOG_DAEMON|LOG_INFO;
c->ioprio = IOPRIO_PRIO_VALUE(IOPRIO_CLASS_BE, 0);
c->cpu_sched_policy = SCHED_OTHER;
c->syslog_priority = LOG_DAEMON|LOG_INFO;
+ c->syslog_level_prefix = true;
c->mount_flags = MS_SHARED;
c->mount_flags = MS_SHARED;
+ c->kill_signal = SIGTERM;
}
void exec_context_done(ExecContext *c) {
}
void exec_context_done(ExecContext *c) {
@@
-1341,6
+1363,9
@@
void exec_context_done(ExecContext *c) {
strv_free(c->inaccessible_dirs);
c->inaccessible_dirs = NULL;
strv_free(c->inaccessible_dirs);
c->inaccessible_dirs = NULL;
+
+ if (c->cpuset)
+ CPU_FREE(c->cpuset);
}
void exec_command_done(ExecCommand *c) {
}
void exec_command_done(ExecCommand *c) {
@@
-1449,16
+1474,16
@@
void exec_context_dump(ExecContext *c, FILE* f, const char *prefix) {
prefix, c->cpu_sched_priority,
prefix, yes_no(c->cpu_sched_reset_on_fork));
prefix, c->cpu_sched_priority,
prefix, yes_no(c->cpu_sched_reset_on_fork));
- if (c->cpu
_affinity_
set) {
+ if (c->cpuset) {
fprintf(f, "%sCPUAffinity:", prefix);
fprintf(f, "%sCPUAffinity:", prefix);
- for (i = 0; i <
CPU_SETSIZE
; i++)
- if (CPU_ISSET
(i, &c->cpu_affinity
))
+ for (i = 0; i <
c->cpuset_ncpus
; i++)
+ if (CPU_ISSET
_S(i, CPU_ALLOC_SIZE(c->cpuset_ncpus), c->cpuset
))
fprintf(f, " %i", i);
fputs("\n", f);
}
fprintf(f, " %i", i);
fputs("\n", f);
}
- if (c->timer_slack_ns_set)
- fprintf(f, "%sTimerSlackNS
: %lu\n", prefix, c->timer_slack_ns
);
+ if (c->timer_slack_ns
ec
_set)
+ fprintf(f, "%sTimerSlackNS
ec: %lu\n", prefix, c->timer_slack_nsec
);
fprintf(f,
"%sStandardInput: %s\n"
fprintf(f,
"%sStandardInput: %s\n"
@@
-1547,13
+1572,31
@@
void exec_context_dump(ExecContext *c, FILE* f, const char *prefix) {
strv_fprintf(f, c->inaccessible_dirs);
fputs("\n", f);
}
strv_fprintf(f, c->inaccessible_dirs);
fputs("\n", f);
}
+
+ fprintf(f,
+ "%sKillMode: %s\n"
+ "%sKillSignal: SIG%s\n",
+ prefix, kill_mode_to_string(c->kill_mode),
+ prefix, signal_to_string(c->kill_signal));
}
}
-void exec_status_
fill(ExecStatus *s, pid_t pid, int code, int status
) {
+void exec_status_
start(ExecStatus *s, pid_t pid
) {
assert(s);
assert(s);
+ zero(*s);
+ s->pid = pid;
+ dual_timestamp_get(&s->start_timestamp);
+}
+
+void exec_status_exit(ExecStatus *s, pid_t pid, int code, int status) {
+ assert(s);
+
+ if ((s->pid && s->pid != pid) ||
+ !s->start_timestamp.realtime <= 0)
+ zero(*s);
+
s->pid = pid;
s->pid = pid;
-
s->exit_timestamp = now(CLOCK_REALTIME
);
+
dual_timestamp_get(&s->exit_timestamp
);
s->code = code;
s->status = status;
s->code = code;
s->status = status;
@@
-1572,20
+1615,20
@@
void exec_status_dump(ExecStatus *s, FILE *f, const char *prefix) {
return;
fprintf(f,
return;
fprintf(f,
- "%sPID: %l
l
u\n",
- prefix, (unsigned long
long
) s->pid);
+ "%sPID: %lu\n",
+ prefix, (unsigned long) s->pid);
- if (s->start_timestamp > 0)
+ if (s->start_timestamp
.realtime
> 0)
fprintf(f,
"%sStart Timestamp: %s\n",
fprintf(f,
"%sStart Timestamp: %s\n",
- prefix, format_timestamp(buf, sizeof(buf), s->start_timestamp));
+ prefix, format_timestamp(buf, sizeof(buf), s->start_timestamp
.realtime
));
- if (s->exit_timestamp > 0)
+ if (s->exit_timestamp
.realtime
> 0)
fprintf(f,
"%sExit Timestamp: %s\n"
"%sExit Code: %s\n"
"%sExit Status: %i\n",
fprintf(f,
"%sExit Timestamp: %s\n"
"%sExit Code: %s\n"
"%sExit Status: %i\n",
- prefix, format_timestamp(buf, sizeof(buf), s->exit_timestamp),
+ prefix, format_timestamp(buf, sizeof(buf), s->exit_timestamp
.realtime
),
prefix, sigchld_code_to_string(s->code),
prefix, s->status);
}
prefix, sigchld_code_to_string(s->code),
prefix, s->status);
}