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&... ============================================================================== --- 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");