summaryrefslogtreecommitdiff
path: root/modules/pam_shells/pam_shells.c
diff options
context:
space:
mode:
Diffstat (limited to 'modules/pam_shells/pam_shells.c')
-rw-r--r--modules/pam_shells/pam_shells.c12
1 files changed, 10 insertions, 2 deletions
diff --git a/modules/pam_shells/pam_shells.c b/modules/pam_shells/pam_shells.c
index abebdd0c..05c09c65 100644
--- a/modules/pam_shells/pam_shells.c
+++ b/modules/pam_shells/pam_shells.c
@@ -61,8 +61,16 @@ static int perform_check(pam_handle_t *pamh)
}
pw = pam_modutil_getpwnam(pamh, userName);
- if (pw == NULL || pw->pw_shell == NULL) {
- return PAM_AUTH_ERR; /* user doesn't exist */
+ if (pw == NULL) {
+ return PAM_USER_UNKNOWN;
+ }
+ if (pw->pw_shell == NULL) {
+ /* TODO: when does this happen? I would join it with
+ * the case userShell[0] == '\0' below.
+ *
+ * For now, keep the existing stricter behaviour
+ */
+ return PAM_AUTH_ERR;
}
userShell = pw->pw_shell;
if (userShell[0] == '\0')