From: Karel Zak <kzak@redhat.com>
Date: Tue, 28 Nov 2017 13:49:05 +0000 (+0100)
Subject: login-utils: use free_getlogindefs_data()
X-Git-Tag: v2.32-rc1~153
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f4b03edb737f5d4991d9ec8816dd2b3ac3da7abf;p=thirdparty%2Futil-linux.git

login-utils: use free_getlogindefs_data()

It seems better to deallocate logindefs.conf stuff in long time
running (=waiting) processes like login(1) and su(1).

Signed-off-by: Karel Zak <kzak@redhat.com>
---

diff --git a/login-utils/login.c b/login-utils/login.c
index ae0d66d0a7..cc14002461 100644
--- a/login-utils/login.c
+++ b/login-utils/login.c
@@ -945,6 +945,8 @@ static void fork_session(struct login_context *cxt)
 		close(0);
 		close(1);
 		close(2);
+		free_getlogindefs_data();
+
 		sa.sa_handler = SIG_IGN;
 		sigaction(SIGQUIT, &sa, NULL);
 		sigaction(SIGINT, &sa, NULL);
diff --git a/login-utils/su-common.c b/login-utils/su-common.c
index 0ba6d19534..b1be7c6ef4 100644
--- a/login-utils/su-common.c
+++ b/login-utils/su-common.c
@@ -854,6 +854,8 @@ static void create_watching_parent(struct su_context *su)
 		break;
 	}
 
+	/* free unnecessary stuff */
+	free_getlogindefs_data();
 
 	/* In the parent watch the child.  */