diff options
author | Darren Tucker <dtucker@zip.com.au> | 2013-12-19 11:31:44 +1100 |
---|---|---|
committer | Darren Tucker <dtucker@zip.com.au> | 2013-12-19 11:31:44 +1100 |
commit | 53f8e784dc431a82d31c9b0e95b144507f9330e9 (patch) | |
tree | b576ea4fa301ebd2f94d831ae2e15f75644e86a9 | |
parent | 1fcec9d4f265e38af248c4c845986ca8c174bd68 (diff) |
- (dtucker) [auth-pam.c] bz#2163: check return value from pam_get_item().
Patch from Loganaden Velvindron.
-rw-r--r-- | ChangeLog | 2 | ||||
-rw-r--r-- | auth-pam.c | 4 |
2 files changed, 5 insertions, 1 deletions
@@ -1,6 +1,8 @@ 20131219 - (dtucker) [configure.ac] bz#2178: Don't try to use BSM on Solaris versions greater than 11 either rather than just 11. Patch from Tomas Kuthan. + - (dtucker) [auth-pam.c] bz#2163: check return value from pam_get_item(). + Patch from Loganaden Velvindron. 20131218 - (djm) OpenBSD CVS Sync @@ -438,8 +438,10 @@ sshpam_thread(void *ctxtp) const char **ptr_pam_user = &pam_user; char *tz = getenv("TZ"); - pam_get_item(sshpam_handle, PAM_USER, + sshpam_err = pam_get_item(sshpam_handle, PAM_USER, (sshpam_const void **)ptr_pam_user); + if (sshpam_err != PAM_SUCCESS) + goto auth_fail; environ[0] = NULL; if (tz != NULL) |