From 6616ed2d983c6f27d5cb58d9ef02c15329178542 Mon Sep 17 00:00:00 2001 From: "Andrew G. Morgan" Date: Mon, 12 Nov 2001 06:57:38 +0000 Subject: Relevant BUGIDs: 476961 Purpose of commit: comments Commit summary: --------------- Added some clarification comments to help explain why I'm rejecting bug 476961. --- modules/pam_pwdb/pam_unix_passwd.-c | 2 ++ modules/pam_pwdb/support.-c | 1 + 2 files changed, 3 insertions(+) (limited to 'modules') 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? */ -- cgit v1.2.3