From: Todd C. Miller <Todd.Miller@courtesan.com>
Date: Tue, 17 Jan 2012 16:22:44 +0000 (-0500)
Subject: Back out change that put the command in its own pgrp when not logging
X-Git-Tag: SUDO_1_7_9~23
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bea7925f2d8d20f074cdde529cd602d124058472;p=sudo

Back out change that put the command in its own pgrp when not logging
I/O.  It causes problems with pipelines.

--HG--
branch : 1.7
---

diff --git a/exec.c b/exec.c
index dd936cfa8..8235829cf 100644
--- a/exec.c
+++ b/exec.c
@@ -118,7 +118,6 @@ static int fork_cmnd(path, argv, envp, sv, rbac_enabled)
     int sv[2];
     int rbac_enabled;
 {
-    int ttyfd, foreground = 0;
     struct command_status cstat;
     sigaction_t sa;
     pid_t child;
@@ -129,12 +128,6 @@ static int fork_cmnd(path, argv, envp, sv, rbac_enabled)
     sa.sa_handler = handler;
     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();
     switch (child) {
     case -1:
@@ -148,16 +141,6 @@ static int fork_cmnd(path, argv, envp, sv, rbac_enabled)
 	fcntl(sv[1], F_SETFD, FD_CLOEXEC);
 	restore_signals();
 	if (exec_setup(rbac_enabled, user_ttypath, -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() */
 	    int maxfd = def_closefrom;
 	    dup2(sv[1], maxfd);
@@ -176,19 +159,6 @@ static int fork_cmnd(path, argv, envp, sv, rbac_enabled)
 	send(sv[1], &cstat, sizeof(cstat), 0);
 	_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);
-
     return child;
 }
 
@@ -552,17 +522,17 @@ handle_signals(fd, child, cstat)
 			 */
 #ifdef HAVE_TCSETPGRP
 			pid_t saved_pgrp = (pid_t)-1;
-			int ttyfd = open(_PATH_TTY, O_RDWR|O_NOCTTY, 0);
-			if (ttyfd != -1)
-			    saved_pgrp = tcgetpgrp(ttyfd);
+			int fd = open(_PATH_TTY, O_RDWR|O_NOCTTY, 0);
+			if (fd != -1)
+			    saved_pgrp = tcgetpgrp(fd);
 #endif /* HAVE_TCSETPGRP */
 			if (kill(getpid(), WSTOPSIG(status)) != 0)
 			    warning("kill(%d, %d)", getpid(), WSTOPSIG(status));
 #ifdef HAVE_TCSETPGRP
-			if (ttyfd != -1) {
+			if (fd != -1) {
 			    if (saved_pgrp != (pid_t)-1)
-				(void)tcsetpgrp(ttyfd, saved_pgrp);
-			    close(ttyfd);
+				(void)tcsetpgrp(fd, saved_pgrp);
+			    close(fd);
 			}
 #endif /* HAVE_TCSETPGRP */
 		    } else {