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