isa/sudo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Back out change that put the command in its own pgrp when not logging

Browse Source 
I/O.  It causes problems with pipelines.
This commit is contained in:
Todd C. Miller
2012-01-17 10:25:44 -05:00
parent 40badb50e6
commit 97cecaf6eb
1 changed files with 6 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files

42
src/exec.c
View File

@@ -109,7 +109,6 @@ my_execve(const char *path, char *const argv[], char *const envp[])
*/ */
static int fork_cmnd(struct command_details *details, int sv[2]) static int fork_cmnd(struct command_details *details, int sv[2])
{ {
int ttyfd, foreground = 0;
struct command_status cstat; struct command_status cstat;
sigaction_t sa; sigaction_t sa;
pid_t child; pid_t child;
@@ -121,12 +120,6 @@ static int fork_cmnd(struct command_details *details, int sv[2])
sa.sa_handler = handler; sa.sa_handler = handler;
sigaction(SIGCONT, &sa, NULL); sigaction(SIGCONT, &sa, NULL);
ttyfd = open(_PATH_TTY, O_RDWR|O_NOCTTY, 0);
if (ttyfd != -1) {
foreground = (tcgetpgrp(ttyfd) == getpgrp());
(void)fcntl(ttyfd, F_SETFD, FD_CLOEXEC);
}
child = fork(); child = fork();
switch (child) { switch (child) {
case -1: case -1:
@@ -140,16 +133,6 @@ static int fork_cmnd(struct command_details *details, int sv[2])
fcntl(sv[1], F_SETFD, FD_CLOEXEC); fcntl(sv[1], F_SETFD, FD_CLOEXEC);
restore_signals(); restore_signals();
if (exec_setup(details, NULL, -1) == true) { if (exec_setup(details, NULL, -1) == true) {
/* Set child process group here too to avoid a race. */
child = getpid();
setpgid(0, child);
/* Wait for parent to grant us the tty if we are foreground. */
if (foreground) {
while (tcgetpgrp(ttyfd) != child)
; /* spin */
}
/* headed for execve() */ /* headed for execve() */
sudo_debug_execve(SUDO_DEBUG_INFO, details->command, sudo_debug_execve(SUDO_DEBUG_INFO, details->command,
details->argv, details->envp); details->argv, details->envp);
@@ -177,19 +160,6 @@ static int fork_cmnd(struct command_details *details, int sv[2])
sudo_debug_exit_int(__func__, __FILE__, __LINE__, sudo_debug_subsys, 1); sudo_debug_exit_int(__func__, __FILE__, __LINE__, sudo_debug_subsys, 1);
_exit(1); _exit(1);
} }
/*
* Put child in its own process group. If we are starting the command
* in the foreground, assign its pgrp to the tty.
*/
setpgid(child, child);
if (foreground) {
while (tcsetpgrp(ttyfd, child) == -1 && errno == EINTR)
continue;
}
if (ttyfd != -1)
close(ttyfd);
debug_return_int(child); debug_return_int(child);
} }
@@ -534,17 +504,17 @@ handle_signals(int fd, pid_t child, int log_io, struct command_status *cstat)
* so we can restore it after we resume. * so we can restore it after we resume.
*/ */
pid_t saved_pgrp = (pid_t)-1; pid_t saved_pgrp = (pid_t)-1;
int ttyfd = open(_PATH_TTY, O_RDWR|O_NOCTTY, 0); int fd = open(_PATH_TTY, O_RDWR|O_NOCTTY, 0);
if (ttyfd != -1) if (fd != -1)
saved_pgrp = tcgetpgrp(ttyfd); saved_pgrp = tcgetpgrp(fd);
if (kill(getpid(), WSTOPSIG(status)) != 0) { if (kill(getpid(), WSTOPSIG(status)) != 0) {
warning("kill(%d, %d)", (int)getpid(), warning("kill(%d, %d)", (int)getpid(),
WSTOPSIG(status)); WSTOPSIG(status));
} }
if (ttyfd != -1) { if (fd != -1) {
if (saved_pgrp != (pid_t)-1) if (saved_pgrp != (pid_t)-1)
(void)tcsetpgrp(ttyfd, saved_pgrp); (void)tcsetpgrp(fd, saved_pgrp);
close(ttyfd); close(fd);
} }
} else { } else {
/* Child has exited, we are done. */ /* Child has exited, we are done. */