[ros-dev] Regarding 48632

[Ros Arm]

Hello,

Many thanks for fixing this idiotic oversight of mine. I apologies for the long hours of debugging I must have caused.

This fix is somewhat incomplete/at the wrong place; could this be marked as such in the code, perhaps with a FIXME under my name, so that I may fix this correctly upon my return to the US?

Much obliged!

-r

Author: mjmartin
Date: Sat Aug 28 00:26:02 2010
New Revision: 48632

URL: http://svn.reactos.org/svn/reactos?rev=48632&view=rev
Log:
[ntoskrnl/ps]
- When deleting a Process remove the Process from the MmProcessList. Fixes random NonPaged Pool corruptions.  Thanks aicom for assistance.

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

Modified: trunk/reactos/ntoskrnl/ps/kill.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ps/kill.c?rev=48632&r1=48631&r2=48632&view=diff
==============================================================================
--- trunk/reactos/ntoskrnl/ps/kill.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/ps/kill.c [iso-8859-1] Sat Aug 28 00:26:02 2010
@@ -300,6 +300,8 @@

        /* Detach */
        KeUnstackDetachProcess(&ApcState);
+
+        RemoveEntryList(&Process->MmProcessLinks);

        /* Completely delete the Address Space */
        MmDeleteProcessAddressSpace(Process);