diff options
| author | Steve Langasek <vorlon@debian.org> | 2001-04-28 16:06:19 +0000 |
|---|---|---|
| committer | Steve Langasek <vorlon@debian.org> | 2001-04-28 16:06:19 +0000 |
| commit | 7143eaf024a740627da96c69d960051816c5b593 (patch) | |
| tree | a4d8fe90600452285e64c72119c92181f4c71768 /modules/pam_unix/pam_unix_passwd.c | |
| parent | 377f9e2157d3cebf0de131228a0a525a94a5dc5e (diff) | |
| download | pam-7143eaf024a740627da96c69d960051816c5b593.tar.gz pam-7143eaf024a740627da96c69d960051816c5b593.tar.bz2 pam-7143eaf024a740627da96c69d960051816c5b593.zip | |
Relevant BUGIDs: 419803
Purpose of commit: bugfix
Commit summary:
---------------
Fix pam_unix to not zero out password strings that it doesn't own!
Diffstat (limited to 'modules/pam_unix/pam_unix_passwd.c')
| -rw-r--r-- | modules/pam_unix/pam_unix_passwd.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/modules/pam_unix/pam_unix_passwd.c b/modules/pam_unix/pam_unix_passwd.c index 3fe8a27a..c85305e4 100644 --- a/modules/pam_unix/pam_unix_passwd.c +++ b/modules/pam_unix/pam_unix_passwd.c @@ -891,8 +891,6 @@ PAM_EXTERN int pam_sm_chauthtok(pam_handle_t * pamh, int flags, if (retval != PAM_SUCCESS) { _log_err(LOG_NOTICE, pamh, "new password not acceptable"); - _pam_overwrite(pass_new); - _pam_overwrite(pass_old); pass_new = pass_old = NULL; /* tidy up */ #ifdef USE_LCKPWDF ulckpwdf(); @@ -935,8 +933,6 @@ PAM_EXTERN int pam_sm_chauthtok(pam_handle_t * pamh, int flags, if (temp == NULL) { _log_err(LOG_CRIT, pamh, "out of memory for password"); - _pam_overwrite(pass_new); - _pam_overwrite(pass_old); pass_new = pass_old = NULL; /* tidy up */ #ifdef USE_LCKPWDF ulckpwdf(); @@ -970,8 +966,6 @@ PAM_EXTERN int pam_sm_chauthtok(pam_handle_t * pamh, int flags, retval = _do_setpass(pamh, user, pass_old, tpass, ctrl, remember); - _pam_overwrite(pass_new); - _pam_overwrite(pass_old); _pam_delete(tpass); pass_old = pass_new = NULL; } else { /* something has broken with the module */ |