[ros-diffs] [sir_richard] 48758: [NTOS]: Move the fix for pool corruption due to dangling MmProcessLinks pointer into its correct location. Thanks again to Martin for the fix.

Author: sir_richard
Date: Sun Sep 12 19:02:39 2010
New Revision: 48758

URL: http://svn.reactos.org/svn/reactos?rev=48758&view=rev
Log:
[NTOS]: Move the fix for pool corruption due to dangling MmProcessLinks pointer into its
correct location. Thanks again to Martin for the fix.

Modified:
    trunk/reactos/ntoskrnl/mm/procsup.c
    trunk/reactos/ntoskrnl/ps/kill.c

Modified: trunk/reactos/ntoskrnl/mm/procsup.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/procsup.c?rev=…
==============================================================================
--- trunk/reactos/ntoskrnl/mm/procsup.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/procsup.c [iso-8859-1] Sun Sep 12 19:02:39 2010
@@ -24,6 +24,8 @@
 
    DPRINT("MmDeleteProcessAddressSpace(Process %x (%s))\n", Process,
           Process->ImageFileName);
+
+   RemoveEntryList(&Process->MmProcessLinks);
 
    MmLockAddressSpace(&Process->Vm);
 

Modified: trunk/reactos/ntoskrnl/ps/kill.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ps/kill.c?rev=487…
==============================================================================
--- trunk/reactos/ntoskrnl/ps/kill.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/ps/kill.c [iso-8859-1] Sun Sep 12 19:02:39 2010
@@ -300,8 +300,6 @@
 
         /* Detach */
         KeUnstackDetachProcess(&ApcState);
-
-        RemoveEntryList(&Process->MmProcessLinks);
 
         /* Completely delete the Address Space */
         MmDeleteProcessAddressSpace(Process);