mirror of
https://github.com/reactos/reactos.git
synced 2025-04-21 20:50:29 +00:00
[NTOSKRNL]: Fix the zombie fix. Should fix toolhelp tests.
[NTOSKRNL]: If you're going to fake success when doing access checks, also grant the rights that would normally be refused, for consistency's sake. Should fix a bunch of the kernel32:synch tests (which have nothing to do with synch...). svn path=/trunk/; revision=55737
This commit is contained in:
parent
0e6938d173
commit
e96c88b49b
2 changed files with 5 additions and 3 deletions
|
@ -721,8 +721,6 @@ QSI_DEF(SystemProcessInformation)
|
||||||
SystemProcess = PsIdleProcess;
|
SystemProcess = PsIdleProcess;
|
||||||
Process = SystemProcess;
|
Process = SystemProcess;
|
||||||
Current = (PUCHAR) Spi;
|
Current = (PUCHAR) Spi;
|
||||||
CurrentSize = 0;
|
|
||||||
ImageNameMaximumLength = 0;
|
|
||||||
|
|
||||||
do
|
do
|
||||||
{
|
{
|
||||||
|
@ -733,7 +731,10 @@ QSI_DEF(SystemProcessInformation)
|
||||||
!(Process->ActiveThreads) &&
|
!(Process->ActiveThreads) &&
|
||||||
(IsListEmpty(&Process->Pcb.ThreadListHead)))
|
(IsListEmpty(&Process->Pcb.ThreadListHead)))
|
||||||
{
|
{
|
||||||
DPRINT1("Skipping zombie\n");
|
DPRINT1("Process %p (%s:%lx) is a zombie\n",
|
||||||
|
Process, Process->ImageFileName, Process->UniqueProcessId);
|
||||||
|
CurrentSize = 0;
|
||||||
|
ImageNameMaximumLength = 0;
|
||||||
goto Skip;
|
goto Skip;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -705,6 +705,7 @@ SepAccessCheck(IN PSECURITY_DESCRIPTOR SecurityDescriptor,
|
||||||
*GrantedAccess, DesiredAccess, GenericMapping);
|
*GrantedAccess, DesiredAccess, GenericMapping);
|
||||||
//*AccessStatus = STATUS_ACCESS_DENIED;
|
//*AccessStatus = STATUS_ACCESS_DENIED;
|
||||||
//return FALSE;
|
//return FALSE;
|
||||||
|
*GrantedAccess = DesiredAccess;
|
||||||
*AccessStatus = STATUS_SUCCESS;
|
*AccessStatus = STATUS_SUCCESS;
|
||||||
return TRUE;
|
return TRUE;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue