[ros-dev] [ros-diffs] [jgardou] 64537: [NTOS/MM] - Do not assert in case of stack overflow, just let the page fault handler raise STATUS_STACK_OVERFLOW

[Alex Ionescu]

The ASSERT is there because of the missing functionality. Please see the
comment just above.

Best regards,
Alex Ionescu

On Sun, Oct 5, 2014 at 2:57 AM, <jgardou at svn.reactos.org> wrote:

> Author: jgardou
> Date: Sun Oct  5 09:57:02 2014
> New Revision: 64537
>
> URL: http://svn.reactos.org/svn/reactos?rev=64537&view=rev
> Log:
> [NTOS/MM]
>  - Do not assert in case of stack overflow, just let the page fault
> handler raise STATUS_STACK_OVERFLOW
>
> Modified:
>     trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c
>
> Modified: trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c
> URL:
> http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c?rev=64537&r1=64536&r2=64537&view=diff
>
> ==============================================================================
> --- trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c   [iso-8859-1] (original)
> +++ trunk/reactos/ntoskrnl/mm/ARM3/pagfault.c   [iso-8859-1] Sun Oct  5
> 09:57:02 2014
> @@ -82,7 +82,6 @@
>      {
>          /* We don't -- Windows would try to make this guard page valid
> now */
>          DPRINT1("Close to our death...\n");
> -        ASSERT(FALSE);
>          return STATUS_STACK_OVERFLOW;
>      }
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.reactos.org/pipermail/ros-dev/attachments/20141006/8d3a58c1/attachment.html>