https://git.reactos.org/?p=reactos.git;a=commitdiff;h=d84022d7fd4149213684207a33f524cd2c00d3b8

commit d84022d7fd4149213684207a33f524cd2c00d3b8
Author:     Thomas Faber <[email protected]>
AuthorDate: Sun Nov 21 09:34:07 2021 -0500
Commit:     Thomas Faber <[email protected]>
CommitDate: Sun Nov 21 12:57:50 2021 -0500

    [NTOS:SE] Fix always-true assert in SeTokenCanImpersonate.
    
    Courtesy of VS analysis warning C6289:
    Incorrect operator:  mutual exclusion over || is always a non-zero 
constant.  Did you intend to use && instead?
---
 ntoskrnl/se/token.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ntoskrnl/se/token.c b/ntoskrnl/se/token.c
index fb1b0a52159..cd320a8dcdf 100644
--- a/ntoskrnl/se/token.c
+++ b/ntoskrnl/se/token.c
@@ -3585,7 +3585,7 @@ SeTokenCanImpersonate(
      * allow impersonation. If we get such levels from the call
      * then something's seriously wrong.
      */
-    ASSERT(ImpersonationLevel != SecurityAnonymous ||
+    ASSERT(ImpersonationLevel != SecurityAnonymous &&
            ImpersonationLevel != SecurityIdentification);
 
     /* Time to lock our tokens */

Reply via email to