https://git.reactos.org/?p=reactos.git;a=commitdiff;h=d84022d7fd414921368420...
commit d84022d7fd4149213684207a33f524cd2c00d3b8 Author: Thomas Faber thomas.faber@reactos.org AuthorDate: Sun Nov 21 09:34:07 2021 -0500 Commit: Thomas Faber thomas.faber@reactos.org 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 */