as the clang static analyser points out, the | is useless here.

[citadel.git] / citadel / auth.c

diff --git a/citadel/auth.c b/citadel/auth.c
index f1062a84f923b1d9cbb3f48abdde7306ce03888c..89218b60df93d61a12b897ba7e8a6f1d0c2ead7c 100644 (file)
--- a/citadel/auth.c
+++ b/citadel/auth.c
@@ -5,7 +5,7 @@
  *
  * Copyright (c) 1999-2009 by the citadel.org team
  *
- * This program is free software; you can redistribute it and/or modify
+ * This program is open source software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
@@ -110,13 +110,14 @@ int validate_password(uid_t uid, const char *pass)
 #endif
        struct passwd *pw;
        int retval = 0;
-       int flags = 0;
-
-       flags = 0;      /* silences compiler warning */
+       int flags;
 
 #ifdef PAM_DATA_SILENT
-       flags = ( flags | PAM_DATA_SILENT ) ;
+       flags = PAM_DATA_SILENT;
+#else
+       flags = 0;
 #endif /* PAM_DATA_SILENT */
+
        if ((pw = getpwuid(uid)) == NULL) {
                return retval;
        }