531551
From 9b5dc4cb8d5d82c31c0cda898832998c21afc303 Mon Sep 17 00:00:00 2001
531551
From: Karel Zak <kzak@redhat.com>
531551
Date: Mon, 9 Sep 2013 12:24:01 +0200
531551
Subject: [PATCH] su: fix lastlog and btmp logging
531551
531551
The su(1) logging code mix ups "old" and "new" passwd structs. The
531551
result is things like
531551
531551
	Sep  9 11:50:45 x2 su: (to kzak) kzak on none
531551
531551
in /var/log/messages. The right log entry is
531551
531551
	Sep  9 11:50:45 x2 su: (to root) kzak on pts/3
531551
531551
The bug has been introduced by commit c74a7af17c7a176c358dfaa8e1814786c89ebc14.
531551
531551
References: https://bugzilla.redhat.com/show_bug.cgi?id=1005194
531551
Signed-off-by: Karel Zak <kzak@redhat.com>
531551
---
531551
 login-utils/su-common.c | 5 +----
531551
 1 file changed, 1 insertion(+), 4 deletions(-)
531551
531551
diff --git a/login-utils/su-common.c b/login-utils/su-common.c
531551
index ade5c92..858af01 100644
531551
--- a/login-utils/su-common.c
531551
+++ b/login-utils/su-common.c
531551
@@ -161,7 +161,7 @@ log_syslog(struct passwd const *pw, bool successful)
531551
       old_user = pwd ? pwd->pw_name : "";
531551
     }
531551
 
531551
-  if (get_terminal_name(STDERR_FILENO, NULL, &tty, NULL) == 0 && tty)
531551
+  if (get_terminal_name(STDERR_FILENO, NULL, &tty, NULL) != 0 || !tty)
531551
     tty = "none";
531551
 
531551
   openlog (program_invocation_short_name, 0 , LOG_AUTH);
531551
@@ -483,9 +483,6 @@ authenticate (const struct passwd *pw)
531551
 
531551
 done:
531551
 
531551
-  if (lpw && lpw->pw_name)
531551
-     pw = lpw;
531551
-
531551
   log_syslog(pw, !is_pam_failure(retval));
531551
 
531551
   if (is_pam_failure(retval))
531551
-- 
531551
1.8.1.4
531551