[NTOS:OB] Correctly check SeSinglePrivilegeCheck() return value (#1323)
authorSerge Gautherie <32623169+SergeGautherie@users.noreply.github.com>
Tue, 29 Jan 2019 23:43:13 +0000 (00:43 +0100)
committerHermès BÉLUSCA - MAÏTO <hermes.belusca-maito@reactos.org>
Tue, 29 Jan 2019 23:43:13 +0000 (00:43 +0100)
SeSinglePrivilegeCheck() returns a BOOLEAN, not a NTSTATUS.
CORE-14271

ntoskrnl/ob/oblife.c

index 6cc1403..d649b0f 100644 (file)
@@ -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,