https://git.reactos.org/?p=reactos.git;a=commitdiff;h=0483a5a38089217868401fb9fe5e9885cddebcc6
commit 0483a5a38089217868401fb9fe5e9885cddebcc6 Author: Serge Gautherie <[email protected]> AuthorDate: Wed Jan 30 00:43:13 2019 +0100 Commit: Hermès BÉLUSCA - MAÏTO <[email protected]> CommitDate: Wed Jan 30 00:43:13 2019 +0100 [NTOS:OB] Correctly check SeSinglePrivilegeCheck() return value (#1323) SeSinglePrivilegeCheck() returns a BOOLEAN, not a NTSTATUS. CORE-14271 --- ntoskrnl/ob/oblife.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/ntoskrnl/ob/oblife.c b/ntoskrnl/ob/oblife.c index 6cc1403e63..d649b0f049 100644 --- a/ntoskrnl/ob/oblife.c +++ b/ntoskrnl/ob/oblife.c @@ -1427,9 +1427,10 @@ NtMakePermanentObject(IN HANDLE ObjectHandle) PAGED_CODE(); /* Make sure that the caller has SeCreatePermanentPrivilege */ - Status = SeSinglePrivilegeCheck(SeCreatePermanentPrivilege, - PreviousMode); - if (!NT_SUCCESS(Status)) return STATUS_PRIVILEGE_NOT_HELD; + if (!SeSinglePrivilegeCheck(SeCreatePermanentPrivilege, PreviousMode)) + { + return STATUS_PRIVILEGE_NOT_HELD; + } /* Reference the object */ Status = ObReferenceObjectByHandle(ObjectHandle,
