diff options
| author | Darren Tucker <dtucker@zip.com.au> | 2013-12-19 01:31:44 +0100 |
|---|---|---|
| committer | Darren Tucker <dtucker@zip.com.au> | 2013-12-19 01:31:44 +0100 |
| commit | 53f8e784dc431a82d31c9b0e95b144507f9330e9 (patch) | |
| tree | b576ea4fa301ebd2f94d831ae2e15f75644e86a9 /auth-pam.c | |
| parent | - (dtucker) [configure.ac] bz#2178: Don't try to use BSM on Solaris versions (diff) | |
| download | openssh-53f8e784dc431a82d31c9b0e95b144507f9330e9.tar.xz openssh-53f8e784dc431a82d31c9b0e95b144507f9330e9.zip | |
- (dtucker) [auth-pam.c] bz#2163: check return value from pam_get_item().
Patch from Loganaden Velvindron.
Diffstat (limited to 'auth-pam.c')
| -rw-r--r-- | auth-pam.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/auth-pam.c b/auth-pam.c index d51318b3a..d789bad7b 100644 --- a/auth-pam.c +++ b/auth-pam.c @@ -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) |