[linux-pam] Fix compile time errors in --enable-static-modules mode

Dmitry V. Levin ldv at fedoraproject.org
Thu Feb 2 14:07:33 UTC 2012


commit aff021e14203373248e376b4ca013e58074dc7a9
Author: Matveychikov Ilya <i.matveychikov at securitycode.ru>
Date:   Tue Jan 17 11:16:49 2012 +0400

    Fix compile time errors in --enable-static-modules mode
    
    * libpam/pam_static_modules.h (_pam_rhosts_auth_modstruct): Remove
    obsolete declaration.
    (static_modules): Remove undefined reference to
    _pam_rhosts_auth_modstruct.
    * modules/pam_pwhistory/opasswd.h: Rename {save,check}_old_password to
    {save,check}_old_pass in order to avoid conflicts with pam_unix.
    * modules/pam_pwhistory/opasswd.c: Likewise.
    * modules/pam_pwhistory/pam_pwhistory.c: Likewise.
    * modules/pam_tally2/pam_tally2.c: Rename _pam_tally_modstruct to
    _pam_tally2_modstruct.
    
    Signed-off-by: Matveychikov Ilya <i.matveychikov at securitycode.ru>

 libpam/pam_static_modules.h           |    2 --
 modules/pam_pwhistory/opasswd.c       |    8 ++++----
 modules/pam_pwhistory/opasswd.h       |   10 +++++-----
 modules/pam_pwhistory/pam_pwhistory.c |   12 ++++++------
 modules/pam_tally2/pam_tally2.c       |    2 +-
 5 files changed, 16 insertions(+), 18 deletions(-)
---
diff --git a/libpam/pam_static_modules.h b/libpam/pam_static_modules.h
index 999adc2..67d8135 100644
--- a/libpam/pam_static_modules.h
+++ b/libpam/pam_static_modules.h
@@ -63,7 +63,6 @@ extern struct pam_module _pam_nologin_modstruct;
 extern struct pam_module _pam_permit_modstruct;
 extern struct pam_module _pam_pwhistory_modstruct;
 extern struct pam_module _pam_rhosts_modstruct;
-extern struct pam_module _pam_rhosts_auth_modstruct;
 extern struct pam_module _pam_rootok_modstruct;
 extern struct pam_module _pam_securetty_modstruct;
 #ifdef WITH_SELINUX
@@ -124,7 +123,6 @@ static struct pam_module *static_modules[] = {
   &_pam_permit_modstruct,
   &_pam_pwhistory_modstruct,
   &_pam_rhosts_modstruct,
-  &_pam_rhosts_auth_modstruct,
   &_pam_rootok_modstruct,
   &_pam_securetty_modstruct,
 #ifdef WITH_SELINUX
diff --git a/modules/pam_pwhistory/opasswd.c b/modules/pam_pwhistory/opasswd.c
index f896119..274fdb9 100644
--- a/modules/pam_pwhistory/opasswd.c
+++ b/modules/pam_pwhistory/opasswd.c
@@ -113,8 +113,8 @@ compare_password(const char *newpass, const char *oldpass)
 
 /* Check, if the new password is already in the opasswd file.  */
 int
-check_old_password (pam_handle_t *pamh, const char *user,
-		    const char *newpass, int debug)
+check_old_pass (pam_handle_t *pamh, const char *user,
+		const char *newpass, int debug)
 {
   int retval = PAM_SUCCESS;
   FILE *oldpf;
@@ -209,8 +209,8 @@ check_old_password (pam_handle_t *pamh, const char *user,
 }
 
 int
-save_old_password (pam_handle_t *pamh, const char *user, uid_t uid,
-		   const char *oldpass, int howmany, int debug UNUSED)
+save_old_pass (pam_handle_t *pamh, const char *user, uid_t uid,
+	       const char *oldpass, int howmany, int debug UNUSED)
 {
   char opasswd_tmp[] = TMP_PASSWORDS_FILE;
   struct stat opasswd_stat;
diff --git a/modules/pam_pwhistory/opasswd.h b/modules/pam_pwhistory/opasswd.h
index e8a2013..db3e656 100644
--- a/modules/pam_pwhistory/opasswd.h
+++ b/modules/pam_pwhistory/opasswd.h
@@ -36,10 +36,10 @@
 #ifndef __OPASSWD_H__
 #define __OPASSWD_H__
 
-extern int check_old_password (pam_handle_t *pamh, const char *user,
-			       const char *newpass, int debug);
-extern int save_old_password (pam_handle_t *pamh, const char *user,
-			      uid_t uid, const char *oldpass,
-			      int howmany, int debug);
+extern int check_old_pass (pam_handle_t *pamh, const char *user,
+			   const char *newpass, int debug);
+extern int save_old_pass (pam_handle_t *pamh, const char *user,
+			  uid_t uid, const char *oldpass,
+			  int howmany, int debug);
 
 #endif /* __OPASSWD_H__ */
diff --git a/modules/pam_pwhistory/pam_pwhistory.c b/modules/pam_pwhistory/pam_pwhistory.c
index 9b58895..4c582bc 100644
--- a/modules/pam_pwhistory/pam_pwhistory.c
+++ b/modules/pam_pwhistory/pam_pwhistory.c
@@ -168,15 +168,15 @@ pam_sm_chauthtok (pam_handle_t *pamh, int flags, int argc, const char **argv)
       if (spw == NULL)
 	return PAM_USER_UNKNOWN;
 
-      retval = save_old_password (pamh, user, pwd->pw_uid, spw->sp_pwdp,
-				  options.remember, options.debug);
+      retval = save_old_pass (pamh, user, pwd->pw_uid, spw->sp_pwdp,
+			      options.remember, options.debug);
       if (retval != PAM_SUCCESS)
 	return retval;
     }
   else
     {
-      retval = save_old_password (pamh, user, pwd->pw_uid, pwd->pw_passwd,
-				  options.remember, options.debug);
+      retval = save_old_pass (pamh, user, pwd->pw_uid, pwd->pw_passwd,
+			      options.remember, options.debug);
       if (retval != PAM_SUCCESS)
 	return retval;
     }
@@ -208,8 +208,8 @@ pam_sm_chauthtok (pam_handle_t *pamh, int flags, int argc, const char **argv)
       if (options.debug)
 	pam_syslog (pamh, LOG_DEBUG, "check against old password file");
 
-      if (check_old_password (pamh, user, newpass,
-			      options.debug) != PAM_SUCCESS)
+      if (check_old_pass (pamh, user, newpass,
+			  options.debug) != PAM_SUCCESS)
 	{
 	  pam_error (pamh,
 		     _("Password has been already used. Choose another."));
diff --git a/modules/pam_tally2/pam_tally2.c b/modules/pam_tally2/pam_tally2.c
index c72d27a..d3d6779 100644
--- a/modules/pam_tally2/pam_tally2.c
+++ b/modules/pam_tally2/pam_tally2.c
@@ -844,7 +844,7 @@ pam_sm_acct_mgmt(pam_handle_t *pamh, int flags UNUSED,
 
 /* static module data */
 
-struct pam_module _pam_tally_modstruct = {
+struct pam_module _pam_tally2_modstruct = {
      MODULE_NAME,
 #ifdef PAM_SM_AUTH
      pam_sm_authenticate,


More information about the linux-pam-commits mailing list