Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-06-29 01:12:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

[NTOS:MM] MiDereferenceSession: return as soon as session is destroyed

Browse source 
This leads to some simplifications in the following code.
This commit is contained in:
Hervé Poussineau 2021-05-24 16:38:57 +02:00
parent dd08ae2c0f
commit eac0afc6fe
1 changed files with 6 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
ntoskrnl/mm/ARM3/session.c
View file

@ -364,10 +364,11 @@ MiDereferenceSession(VOID)
{ {
/* No more references left, kill the session completely */ /* No more references left, kill the session completely */
MiDereferenceSessionFinal(); MiDereferenceSessionFinal();
return;
} }
/* Check if tis is the session leader or the last process in the session */ /* Check if this is the session leader */
if ((Process->Vm.Flags.SessionLeader) || (ReferenceCount == 0)) if (Process->Vm.Flags.SessionLeader)
{ {
/* Get the global session address before we kill the session mapping */ /* Get the global session address before we kill the session mapping */
SessionGlobal = MmSessionSpace->GlobalVirtualAddress; SessionGlobal = MmSessionSpace->GlobalVirtualAddress;
@ -377,13 +378,9 @@ MiDereferenceSession(VOID)
BYTES_TO_PAGES(MmSessionSize) * sizeof(MMPDE)); BYTES_TO_PAGES(MmSessionSize) * sizeof(MMPDE));
KeFlushEntireTb(FALSE, FALSE); KeFlushEntireTb(FALSE, FALSE);
/* Is this the session leader? */ /* Clean up the references here. */
if (Process->Vm.Flags.SessionLeader) ASSERT(Process->Session == NULL);
{ MiReleaseProcessReferenceToSessionDataPage(SessionGlobal);
/* Clean up the references here. */
ASSERT(Process->Session == NULL);
MiReleaseProcessReferenceToSessionDataPage(SessionGlobal);
}
} }
/* Reset the current process' session flag */ /* Reset the current process' session flag */