There are two warning when compiling pam_unix with -Wall on current gcc. The following patch fixes them and corrects a minor bug. It also removes useless macro in support.c. diff --git a/modules/pam_unix/passverify.c b/modules/pam_unix/passverify.c index 078adc6..5d6a148 100644 --- a/modules/pam_unix/passverify.c +++ b/modules/pam_unix/passverify.c @@ -31,7 +31,7 @@    #ifdef WITH_SELINUX  #include <selinux/selinux.h> -#define SELINUX_ENABLED is_selinux_enabled()>0 +#define SELINUX_ENABLED (is_selinux_enabled()>0)  #else  #define SELINUX_ENABLED 0  #endif @@ -958,7 +958,7 @@ PAMH_ARG_DECL(int unix_update_shadow,      fclose(opwfile);        if (!wroteentry && !err) { -       spwdent.sp_namp = forwho; +       spwdent.sp_namp = (char *)forwho;         spwdent.sp_pwdp = towhat;         spwdent.sp_lstchg = time(NULL) / (60 * 60 * 24);         if (spwdent.sp_lstchg == 0) diff --git a/modules/pam_unix/support.c b/modules/pam_unix/support.c index abccd82..0fd1dba 100644 --- a/modules/pam_unix/support.c +++ b/modules/pam_unix/support.c @@ -30,12 +30,6 @@    #include "support.h"  #include "passverify.h" -#ifdef WITH_SELINUX -#include <selinux/selinux.h> -#define SELINUX_ENABLED is_selinux_enabled()>0 -#else -#define SELINUX_ENABLED 0 -#endif    static char *  search_key (const char *key, const char *filename) -- Tomas Mraz No matter how far down the wrong road you've gone, turn back. Turkish proverb (You'll never know whether the road is wrong though.)