Re: [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, [e-mail] 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...
>
> ==============================================================================
> --- 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;
> }