diff options
| author | Thorsten Kukuk <kukuk@thkukuk.de> | 2006-11-07 11:11:33 +0000 |
|---|---|---|
| committer | Thorsten Kukuk <kukuk@thkukuk.de> | 2006-11-07 11:11:33 +0000 |
| commit | 2dc572392c3c31de2fa5c3f0e30fa621cac4776e (patch) | |
| tree | 3094a59cb4a2cd18be9848d25df30ebd27928f41 | |
| parent | fa6312088d4bee120c28cb4983623a1846f80096 (diff) | |
| download | pam-2dc572392c3c31de2fa5c3f0e30fa621cac4776e.tar.gz pam-2dc572392c3c31de2fa5c3f0e30fa621cac4776e.tar.bz2 pam-2dc572392c3c31de2fa5c3f0e30fa621cac4776e.zip | |
Relevant BUGIDs:
Purpose of commit: bugfix
Commit summary:
---------------
2006-11-07 Thorsten Kukuk <kukuk@thkukuk.de>
* modules/pam_cracklib/pam_cracklib.c: PAM_DEBUG_ARG
is a bit mask and not a boolean value (Reported by
Jochen Voss <voss@seehuhn.de>).
| -rw-r--r-- | ChangeLog | 6 | ||||
| -rw-r--r-- | modules/pam_cracklib/pam_cracklib.c | 10 |
2 files changed, 11 insertions, 5 deletions
@@ -1,3 +1,9 @@ +2006-11-07 Thorsten Kukuk <kukuk@thkukuk.de> + + * modules/pam_cracklib/pam_cracklib.c: PAM_DEBUG_ARG + is a bit mask and not a boolean value (Reported by + Jochen Voss <voss@seehuhn.de>). + 2006-10-26 Thorsten Kukuk <kukuk@thkukuk.de> * doc/man/pam.3.xml: Add pam_get_user function. diff --git a/modules/pam_cracklib/pam_cracklib.c b/modules/pam_cracklib/pam_cracklib.c index 204132d4..9b496202 100644 --- a/modules/pam_cracklib/pam_cracklib.c +++ b/modules/pam_cracklib/pam_cracklib.c @@ -458,7 +458,7 @@ static int _pam_unix_approve_pass(pam_handle_t *pamh, int retval; if (pass_new == NULL || (pass_old && !strcmp(pass_old,pass_new))) { - if (ctrl && PAM_DEBUG_ARG) + if (ctrl & PAM_DEBUG_ARG) pam_syslog(pamh, LOG_DEBUG, "bad authentication token"); pam_error(pamh, "%s", pass_new == NULL ? _("No password supplied"):_("Password unchanged")); @@ -481,7 +481,7 @@ static int _pam_unix_approve_pass(pam_handle_t *pamh, } if (msg) { - if (ctrl && PAM_DEBUG_ARG) + if (ctrl & PAM_DEBUG_ARG) pam_syslog(pamh, LOG_NOTICE, "new passwd fails strength check: %s", msg); pam_error(pamh, _("BAD PASSWORD: %s"), msg); @@ -598,7 +598,7 @@ PAM_EXTERN int pam_sm_chauthtok(pam_handle_t *pamh, int flags, } if (retval != PAM_SUCCESS) { - if (ctrl && PAM_DEBUG_ARG) + if (ctrl & PAM_DEBUG_ARG) pam_syslog(pamh,LOG_DEBUG,"unable to obtain a password"); continue; } @@ -610,7 +610,7 @@ PAM_EXTERN int pam_sm_chauthtok(pam_handle_t *pamh, int flags, D(("against cracklib")); if ((crack_msg = FascistCheck(token1,options.cracklib_dictpath[0] == '\0'?NULL:options.cracklib_dictpath))) { - if (ctrl && PAM_DEBUG_ARG) + if (ctrl & PAM_DEBUG_ARG) pam_syslog(pamh,LOG_DEBUG,"bad password: %s",crack_msg); pam_error(pamh, _("BAD PASSWORD: %s"), crack_msg); if (getuid() || (flags & PAM_CHANGE_EXPIRED_AUTHTOK)) @@ -666,7 +666,7 @@ PAM_EXTERN int pam_sm_chauthtok(pam_handle_t *pamh, int flags, /* No else, the a retval == PAM_SUCCESS path can change retval to a failure code. */ if (retval != PAM_SUCCESS) { - if (ctrl && PAM_DEBUG_ARG) + if (ctrl & PAM_DEBUG_ARG) pam_syslog(pamh,LOG_DEBUG,"unable to obtain retyped password"); continue; } |