[ros-diffs] [sir_richard] 55991: [NTOS]: If the address space is exhausted, don't ASSERT, simply return failure just as Windows does. Should fix the last of the ASSERTions on TestBot. Next step is to support V...

[sir_richard at svn.reactos.org]

Author: sir_richard
Date: Sun Mar  4 07:04:42 2012
New Revision: 55991

URL: http://svn.reactos.org/svn/reactos?rev=55991&view=rev
Log:
[NTOS]: If the address space is exhausted, don't ASSERT, simply return failure just as Windows does. Should fix the last of the ASSERTions on TestBot.
Next step is to support VirtualProtect/Query correctly.

Modified:
    trunk/reactos/ntoskrnl/mm/ARM3/virtual.c

Modified: trunk/reactos/ntoskrnl/mm/ARM3/virtual.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/virtual.c?rev=55991&r1=55990&r2=55991&view=diff
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/virtual.c [iso-8859-1] Sun Mar  4 07:04:42 2012
@@ -3345,7 +3345,7 @@
                                                    &Process->VadRoot,
                                                    (PMMADDRESS_NODE*)&Process->VadFreeHint,
                                                    &StartingAddress);
-            ASSERT(NT_SUCCESS(Status));
+            if (!NT_SUCCESS(Status)) goto FailPath;
 
             //
             // Now we know where the allocation ends. Make sure it doesn't end up