From 59b2f220e65fc35ebf26253ea716f4be623b9065 Mon Sep 17 00:00:00 2001
From: Wilfried Goesgens <dothebart@citadel.org>
Date: Fri, 25 May 2012 12:03:40 +0200
Subject: [PATCH] as the clang static analyser points out, the | is useless
 here.

---
 citadel/auth.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/citadel/auth.c b/citadel/auth.c
index 501008633..89218b60d 100644
--- a/citadel/auth.c
+++ b/citadel/auth.c
@@ -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;
 	}
-- 
2.30.2