[ros-diffs] [ekohl] 59076: [RTL] Comparison is always true due to limited range of data type. Patch by Samuel Serapion. CORE-7177 #resolve #comment Committed in r59076.

Author: ekohl
Date: Fri May 24 09:50:49 2013
New Revision: 59076

URL: http://svn.reactos.org/svn/reactos?rev=59076&view=rev
Log:
[RTL]
Comparison is always true due to limited range of data type. Patch by Samuel Serapion.

CORE-7177 #resolve #comment Committed in r59076.

Modified:
    trunk/reactos/lib/rtl/acl.c

Modified: trunk/reactos/lib/rtl/acl.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/acl.c?rev=59076&am…
==============================================================================
--- trunk/reactos/lib/rtl/acl.c	[iso-8859-1] (original)
+++ trunk/reactos/lib/rtl/acl.c	[iso-8859-1] Fri May 24 09:50:49 2013
@@ -889,8 +889,7 @@
             }
 
             /* Check what kind of ACE this is */
-            if ((Ace->AceType >= ACCESS_MIN_MS_ACE_TYPE) ||
-                (Ace->AceType <= ACCESS_MAX_MS_V2_ACE_TYPE))
+            if (Ace->AceType <= ACCESS_MAX_MS_V2_ACE_TYPE)
             {
                 /* Validate the length of this ACE */
                 if (ROUND_DOWN(Ace->AceSize, sizeof(ULONG)) != Ace->AceSize)
@@ -932,7 +931,7 @@
             {
                 DPRINT1("Unsupported ACE in ReactOS, assuming valid\n");
             }
-            else if ((Ace->AceType >= ACCESS_MIN_MS_OBJECT_ACE_TYPE) ||
+            else if ((Ace->AceType >= ACCESS_MIN_MS_OBJECT_ACE_TYPE) &&
                      (Ace->AceType <= ACCESS_MAX_MS_OBJECT_ACE_TYPE))
             {
                 DPRINT1("Unsupported ACE in ReactOS, assuming valid\n");