summaryrefslogtreecommitdiff
path: root/modules
diff options
context:
space:
mode:
authorAndrew G. Morgan <morgan@kernel.org>2001-11-12 06:57:38 +0000
committerAndrew G. Morgan <morgan@kernel.org>2001-11-12 06:57:38 +0000
commit6616ed2d983c6f27d5cb58d9ef02c15329178542 (patch)
tree8da8f1f0aacaeb75c6774fbcb641c73206d1438e /modules
parent0c1e9795ad93a2f801db37d8e71a1b9d7d074ce8 (diff)
Relevant BUGIDs: 476961
Purpose of commit: comments Commit summary: --------------- Added some clarification comments to help explain why I'm rejecting bug 476961.
Diffstat (limited to 'modules')
-rw-r--r--modules/pam_pwdb/pam_unix_passwd.-c2
-rw-r--r--modules/pam_pwdb/support.-c1
2 files changed, 3 insertions, 0 deletions
diff --git a/modules/pam_pwdb/pam_unix_passwd.-c b/modules/pam_pwdb/pam_unix_passwd.-c
index e0ef0917..0949af7f 100644
--- a/modules/pam_pwdb/pam_unix_passwd.-c
+++ b/modules/pam_pwdb/pam_unix_passwd.-c
@@ -236,6 +236,8 @@ static int _unix_chauthtok(pam_handle_t *pamh, unsigned int ctrl)
/*
* By reaching here we have approved the passwords and must now
* rebuild the password database file.
+ *
+ * This includes the fact that the password is _not_ NULL.
*/
/*
diff --git a/modules/pam_pwdb/support.-c b/modules/pam_pwdb/support.-c
index bbaa51ac..e6d5829d 100644
--- a/modules/pam_pwdb/support.-c
+++ b/modules/pam_pwdb/support.-c
@@ -513,6 +513,7 @@ static int _unix_verify_password(pam_handle_t *pamh, const char *name,
} else {
+ /* there is no way that p can be NULL (one can be "") */
pp = _pam_md(p, salt);
/* the moment of truth -- do we agree with the password? */