treewide: use stdio_unset_cloexec() function
This commit is contained in:
parent
3b9a1d87cc
commit
913f38e440
|
@ -888,9 +888,7 @@ int make_stdio(int fd) {
|
|||
|
||||
/* Explicitly unset O_CLOEXEC, since if fd was < 3, then
|
||||
* dup2() was a NOP and the bit hence possibly set. */
|
||||
fd_cloexec(STDIN_FILENO, false);
|
||||
fd_cloexec(STDOUT_FILENO, false);
|
||||
fd_cloexec(STDERR_FILENO, false);
|
||||
stdio_unset_cloexec();
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
|
|
@ -125,9 +125,7 @@ int import_fork_tar_x(const char *path, pid_t *ret) {
|
|||
if (null_fd != STDOUT_FILENO)
|
||||
null_fd = safe_close(null_fd);
|
||||
|
||||
fd_cloexec(STDIN_FILENO, false);
|
||||
fd_cloexec(STDOUT_FILENO, false);
|
||||
fd_cloexec(STDERR_FILENO, false);
|
||||
stdio_unset_cloexec();
|
||||
|
||||
if (unshare(CLONE_NEWNET) < 0)
|
||||
log_error_errno(errno, "Failed to lock tar into network namespace, ignoring: %m");
|
||||
|
@ -199,9 +197,7 @@ int import_fork_tar_c(const char *path, pid_t *ret) {
|
|||
if (null_fd != STDIN_FILENO)
|
||||
null_fd = safe_close(null_fd);
|
||||
|
||||
fd_cloexec(STDIN_FILENO, false);
|
||||
fd_cloexec(STDOUT_FILENO, false);
|
||||
fd_cloexec(STDERR_FILENO, false);
|
||||
stdio_unset_cloexec();
|
||||
|
||||
if (unshare(CLONE_NEWNET) < 0)
|
||||
log_error_errno(errno, "Failed to lock tar into network namespace, ignoring: %m");
|
||||
|
|
|
@ -448,9 +448,7 @@ static int transfer_start(Transfer *t) {
|
|||
safe_close(null_fd);
|
||||
}
|
||||
|
||||
fd_cloexec(STDIN_FILENO, false);
|
||||
fd_cloexec(STDOUT_FILENO, false);
|
||||
fd_cloexec(STDERR_FILENO, false);
|
||||
stdio_unset_cloexec();
|
||||
|
||||
setenv("SYSTEMD_LOG_TARGET", "console-prefixed", 1);
|
||||
setenv("NOTIFY_SOCKET", "/run/systemd/import/notify", 1);
|
||||
|
|
|
@ -506,9 +506,7 @@ int pull_verify(PullJob *main_job,
|
|||
cmd[k++] = "-";
|
||||
cmd[k++] = NULL;
|
||||
|
||||
fd_cloexec(STDIN_FILENO, false);
|
||||
fd_cloexec(STDOUT_FILENO, false);
|
||||
fd_cloexec(STDERR_FILENO, false);
|
||||
stdio_unset_cloexec();
|
||||
|
||||
execvp("gpg2", (char * const *) cmd);
|
||||
execvp("gpg", (char * const *) cmd);
|
||||
|
|
Loading…
Reference in a new issue