From 17af7f0c2741f2a8173a0bd091e97b9bd14d5777 Mon Sep 17 00:00:00 2001 From: Serge Gautherie Date: Wed, 30 Jan 2019 01:54:42 +0100 Subject: [PATCH] [UDFS] Correctly check SeSinglePrivilegeCheck() return value SeSinglePrivilegeCheck() returns a BOOLEAN, not a NTSTATUS. CORE-14271 --- drivers/filesystems/udfs/create.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/filesystems/udfs/create.cpp b/drivers/filesystems/udfs/create.cpp index e841a04d03f..75f209f4b5b 100644 --- a/drivers/filesystems/udfs/create.cpp +++ b/drivers/filesystems/udfs/create.cpp @@ -742,9 +742,9 @@ op_vol_accs_dnd: // we should check appropriate privilege if OpenForBackup requested if(OpenForBackup) { - RC = SeSinglePrivilegeCheck(SeExports->SeBackupPrivilege, UserMode); - if(!NT_SUCCESS(RC)) - try_return(RC); + if (!SeSinglePrivilegeCheck(SeExports->SeBackupPrivilege, UserMode)) { + try_return(RC = STATUS_PRIVILEGE_NOT_HELD); + } } // The FSD might wish to implement the open-by-id option. The "id"