* comparing the result of a bitflag operation with 0 is always a good idea.

diff --git a/citadel/modules/newuser/serv_newuser.c b/citadel/modules/newuser/serv_newuser.c
index 06a029a856927a7b292cc44c0d37bbf947729f0a..afa5f47aee40f0a28cdd19d9e8db8443dea61142 100644 (file)
--- a/citadel/modules/newuser/serv_newuser.c
+++ b/citadel/modules/newuser/serv_newuser.c
@@ -89,7 +89,7 @@ void CopyNewUserGreetings(void) {
         * or if it's not private.
         */
        if (CtdlGetRoom(&CC->room, NEWUSERGREETINGS) != 0) return;
-       if (! CC->room.QRflags & QR_PRIVATE ) return;
+       if ((CC->room.QRflags & QR_PRIVATE) == 0) return;
 
        cdbfr = cdb_fetch(CDB_MSGLISTS, &CC->room.QRnumber, sizeof(long));