[Pam-developers] [PATCH] pam_loginuid: cleanup loginuid buffer initialization

Dmitry V. Levin ldv at altlinux.org
Sun Jan 19 14:54:50 UTC 2014


* modules/pam_loginuid/pam_loginuid.c (set_loginuid): Move loginuid
buffer initialization closer to its first use.
---
 modules/pam_loginuid/pam_loginuid.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/pam_loginuid/pam_loginuid.c b/modules/pam_loginuid/pam_loginuid.c
index d258422..c476f7b 100644
--- a/modules/pam_loginuid/pam_loginuid.c
+++ b/modules/pam_loginuid/pam_loginuid.c
@@ -69,7 +69,6 @@ static int set_loginuid(pam_handle_t *pamh, uid_t uid)
 		close(fd);
 	}
 
-	count = snprintf(loginuid, sizeof(loginuid), "%lu", (unsigned long)uid);
 	fd = open("/proc/self/loginuid", O_NOFOLLOW|O_RDWR);
 	if (fd < 0) {
 		if (errno == ENOENT) {
@@ -82,6 +81,7 @@ static int set_loginuid(pam_handle_t *pamh, uid_t uid)
 		return rc;
 	}
 
+	count = snprintf(loginuid, sizeof(loginuid), "%lu", (unsigned long)uid);
 	if (pam_modutil_read(fd, buf, sizeof(buf)) == count &&
 	    memcmp(buf, loginuid, count) == 0) {
 		rc = PAM_SUCCESS;
-- 
ldv


More information about the Pam-developers mailing list