Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-05-21 18:16:07 +00:00
Code Issues Projects Releases Packages Wiki Activity

[CSRSRV] Fix locking logic when destroying threads & processes

Browse source
This commit is contained in:
Jérôme Gardou 2021-03-23 18:15:23 +01:00 committed by Jérôme Gardou
parent f5bace8536
commit 7054cd10b1
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
subsystems/win32/csrsrv/procsup.c
View file

@ -167,8 +167,9 @@ CsrLockedDereferenceProcess(PCSR_PROCESS CsrProcess)
{ {
/* Call the generic cleanup code */ /* Call the generic cleanup code */
DPRINT1("Should kill process: %p\n", CsrProcess); DPRINT1("Should kill process: %p\n", CsrProcess);
CsrAcquireProcessLock();
CsrProcessRefcountZero(CsrProcess); CsrProcessRefcountZero(CsrProcess);
/* Acquire again, the lock it was released in CsrProcessRefcountZero */
CsrAcquireProcessLock();
} }
} }

3
subsystems/win32/csrsrv/thredsup.c
View file

@ -474,8 +474,9 @@ CsrLockedDereferenceThread(IN PCSR_THREAD CsrThread)
if (LockCount == 0) if (LockCount == 0)
{ {
/* Call the generic cleanup code */ /* Call the generic cleanup code */
CsrAcquireProcessLock();
CsrThreadRefcountZero(CsrThread); CsrThreadRefcountZero(CsrThread);
/* Acquire the lock again, it was released by CsrThreadRefcountZero */
CsrAcquireProcessLock();
} }
} }