From: djm@openbsd.org <djm@openbsd.org>
Date: Mon, 21 Sep 2020 07:29:09 +0000 (+0000)
Subject: upstream: close stdin when forking after authentication too; ok markus
X-Git-Tag: V_8_4_P1~3
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0a4a5571ada76b1b012bec9cf6ad1203fc19ec8d;p=thirdparty%2Fopenssh-portable.git

upstream: close stdin when forking after authentication too; ok markus

OpenBSD-Commit-ID: 43db17e4abc3e6b4a7b033aa8cdab326a7cb6c24
---

diff --git a/ssh.c b/ssh.c
index 6202e3c09..f34ca0d71 100644
--- a/ssh.c
+++ b/ssh.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh.c,v 1.535 2020/09/20 23:31:46 djm Exp $ */
+/* $OpenBSD: ssh.c,v 1.536 2020/09/21 07:29:09 djm Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -1758,7 +1758,8 @@ fork_postauth(void)
 		    _PATH_DEVNULL, strerror(errno));
 	else {
 		keep_stderr = log_is_on_stderr() && debug_flag;
-		if (dup2(devnull, STDOUT_FILENO) == -1 ||
+		if (dup2(devnull, STDIN_FILENO) == -1 ||
+		    dup2(devnull, STDOUT_FILENO) == -1 ||
 		    (!keep_stderr && dup2(devnull, STDOUT_FILENO) == -1))
 			fatal("%s: dup2() stdio failed", __func__);
 		if (devnull > STDERR_FILENO)