Blame SOURCES/0074-Complete-Linux-PAM-compliance-for-forked-child-in-su.patch

e377cc
From 9e7cedda86e5356d1723e6bd0bab5e38c4fe4a34 Mon Sep 17 00:00:00 2001
e377cc
From: "Andrew G. Morgan" <morgan@kernel.org>
e377cc
Date: Sat, 27 Nov 2021 21:00:22 -0800
e377cc
Subject: [PATCH 74/74] Complete Linux-PAM compliance for forked child in su
e377cc
 and login.
e377cc
e377cc
As documented here:
e377cc
e377cc
http://www.linux-pam.org/Linux-PAM-html/adg-interface-by-app-expected.html#adg-pam_end
e377cc
e377cc
The child that is about to exec*() the user shell is supposed to pam_end()
e377cc
with PAM_DATA_SILENT. This gives the modules a last chance to do a minor
e377cc
cleanup of the module state before the user's shell is launched.
e377cc
e377cc
Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1950187
e377cc
Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=2033566
e377cc
Upstream: http://github.com/util-linux/util-linux/commit/4660286e9cdff6d95b49295674b96f83af10ea36
e377cc
Signed-off-by: Andrew G. Morgan <morgan@kernel.org>
e377cc
---
e377cc
 login-utils/login.c     | 3 +++
e377cc
 login-utils/su-common.c | 3 +++
e377cc
 2 files changed, 6 insertions(+)
e377cc
e377cc
diff --git a/login-utils/login.c b/login-utils/login.c
e377cc
index 8c9e43292..9f50fe03b 100644
e377cc
--- a/login-utils/login.c
e377cc
+++ b/login-utils/login.c
e377cc
@@ -1370,6 +1370,9 @@ int main(int argc, char **argv)
e377cc
 
e377cc
 	childArgv[childArgc++] = NULL;
e377cc
 
e377cc
+	/* http://www.linux-pam.org/Linux-PAM-html/adg-interface-by-app-expected.html#adg-pam_end */
e377cc
+	(void) pam_end(cxt.pamh, PAM_SUCCESS|PAM_DATA_SILENT);
e377cc
+
e377cc
 	execvp(childArgv[0], childArgv + 1);
e377cc
 
e377cc
 	if (!strcmp(childArgv[0], "/bin/sh"))
e377cc
diff --git a/login-utils/su-common.c b/login-utils/su-common.c
e377cc
index c1b1a04e4..825ff1d5a 100644
e377cc
--- a/login-utils/su-common.c
e377cc
+++ b/login-utils/su-common.c
e377cc
@@ -1428,6 +1428,9 @@ int su_main(int argc, char **argv, int mode)
e377cc
 	if (su->simulate_login && chdir(su->pwd->pw_dir) != 0)
e377cc
 		warn(_("warning: cannot change directory to %s"), su->pwd->pw_dir);
e377cc
 
e377cc
+	/* http://www.linux-pam.org/Linux-PAM-html/adg-interface-by-app-expected.html#adg-pam_end */
e377cc
+	(void) pam_end(su->pamh, PAM_SUCCESS|PAM_DATA_SILENT);
e377cc
+
e377cc
 	if (shell)
e377cc
 		run_shell(su, shell, command, argv + optind, max(0, argc - optind));
e377cc
 
e377cc
-- 
e377cc
2.31.1
e377cc