From a964714f514bcabd41de17563b2feea43b76888f Mon Sep 17 00:00:00 2001
From: David Xu <davidxu@FreeBSD.org>
Date: Thu, 18 Oct 2007 11:05:30 +0000
Subject: [PATCH] The upper while loop has already recycled child process, so
 the if statement has never executed as expected, fix it.

MFC after: 3 days
---
 usr.bin/su/su.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/usr.bin/su/su.c b/usr.bin/su/su.c
index 8bc54720c2a6..9f5ac44dade8 100644
--- a/usr.bin/su/su.c
+++ b/usr.bin/su/su.c
@@ -469,9 +469,7 @@ main(int argc, char *argv[])
 			}
 			break;
 		}
-		child_pgrp = getpgid(child_pid);
-		if (tcgetpgrp(STDERR_FILENO) == child_pgrp)
-			tcsetpgrp(STDERR_FILENO, getpgrp());
+		tcsetpgrp(STDERR_FILENO, getpgrp());
 		if (pid == -1)
 			err(1, "waitpid");
 		PAM_END();