Hesham/wintun
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

driver: address Code Analysis warning

Browse Source 
In this case, we're referencing objects by handle for closing user space
handles. Should be UserMode, not Irp->RequestorMode as C28126 suggests.

Signed-off-by: Simon Rozman <simon@rozman.si>
This commit is contained in:
Simon Rozman 2021-10-17 13:25:14 +02:00
parent c7b20de96c
commit c20b834674
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
driver/wintun.c
View File

@ -925,6 +925,7 @@ TunDispatchPnp(DEVICE_OBJECT *DeviceObject, IRP *Irp)
continue; continue;
KeStackAttachProcess(Process, &ApcState); KeStackAttachProcess(Process, &ApcState);
if (!VerifierFlags) if (!VerifierFlags)
#pragma warning(suppress : 28126)
Status = ObReferenceObjectByHandle( Status = ObReferenceObjectByHandle(
HandleTable->Handles[Index].HandleValue, 0, NULL, UserMode, &Object, &HandleInfo); HandleTable->Handles[Index].HandleValue, 0, NULL, UserMode, &Object, &HandleInfo);
if (NT_SUCCESS(Status)) if (NT_SUCCESS(Status))