Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-08-05 04:33:32 +00:00
Code Issues Projects Releases Packages Wiki Activity

no need to reference the job object pointer in NtIsProcessInJob()

Browse source 
svn path=/trunk/; revision=11543
This commit is contained in:
Thomas Bluemel 2004-11-05 13:30:18 +00:00
parent 8ee82ee7cc
commit 8655dad649
1 changed files with 18 additions and 29 deletions
Download patch file Download diff file Expand all files Collapse all files

47
reactos/ntoskrnl/ps/job.c
View file

@ -127,38 +127,27 @@ NtIsProcessInJob(IN HANDLE ProcessHandle,
if(ProcessJob != NULL) if(ProcessJob != NULL)
{ {
/* reference the object without caring about access rights as it does not necessarily if(JobHandle == NULL)
have to be accessible from the calling process */
Status = ObReferenceObjectByPointer(ProcessJob,
0,
PsJobType,
KernelMode);
if(NT_SUCCESS(Status))
{ {
if(JobHandle == NULL) /* the process is assigned to a job */
{ Status = STATUS_PROCESS_IN_JOB;
/* simply test whether the process is assigned to a job */ }
Status = ((ProcessJob != NULL) ? STATUS_PROCESS_IN_JOB : STATUS_PROCESS_NOT_IN_JOB); else /* JobHandle != NULL */
} {
else /* JobHandle != NULL */ PEJOB JobObject;
{
PEJOB JobObject;
/* get the job object and compare the object pointer with the one assigned to the process */ /* get the job object and compare the object pointer with the one assigned to the process */
Status = ObReferenceObjectByHandle(JobHandle, Status = ObReferenceObjectByHandle(JobHandle,
JOB_OBJECT_QUERY, JOB_OBJECT_QUERY,
PsJobType, PsJobType,
PreviousMode, PreviousMode,
(PVOID*)&JobObject, (PVOID*)&JobObject,
NULL); NULL);
if(NT_SUCCESS(Status)) if(NT_SUCCESS(Status))
{ {
Status = ((ProcessJob == JobObject) ? STATUS_PROCESS_IN_JOB : STATUS_PROCESS_NOT_IN_JOB); Status = ((ProcessJob == JobObject) ? STATUS_PROCESS_IN_JOB : STATUS_PROCESS_NOT_IN_JOB);
ObDereferenceObject(JobObject); ObDereferenceObject(JobObject);
}
} }
ObDereferenceObject(ProcessJob);
} }
} }
else else