[dgorbachev] 41788: "Fix" MDL PROBE FAILED! bug #4663.

Author: dgorbachev
Date: Mon Jul  6 22:28:11 2009
New Revision: 41788

URL: http://svn.reactos.org/svn/reactos?rev=41788&view=rev
Log:
"Fix" MDL PROBE FAILED! bug #4663.

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

Modified: trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c?r...
==============================================================================
--- trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c [iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c [iso-8859-1] Mon Jul  6 22:28:11 2009
@@ -679,6 +679,7 @@
         // Oops :(
         //
         ProbeStatus = _SEH2_GetExceptionCode();
+        _SEH2_YIELD(break);
     }
     _SEH2_END;

[Alex Ionescu]

I'm confused... I've looked through the code, and I don't get what the  
"Break" does here.

Does the ExRaiseStatus somehow end up handled in this path again? That  
means there's a serious SEH2 bug.
On 6-Jul-09, at 11:28 AM, [e-mail] wrote:
> Author: dgorbachev
> Date: Mon Jul  6 22:28:11 2009
> New Revision: 41788
>
> URL: http://svn.reactos.org/svn/reactos?rev=41788&view=rev
> Log:
> "Fix" MDL PROBE FAILED! bug #4663.
>
> Modified:
>    trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c
>
> Modified: trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c
> URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c?r...
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> = 
> ======================================================================
> --- trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c [iso-8859-1] (original)
> +++ trunk/reactos/ntoskrnl/mm/ARM3/mdlsup.c [iso-8859-1] Mon Jul  6  
> 22:28:11 2009
> @@ -679,6 +679,7 @@
>         // Oops :(
>         //
>         ProbeStatus = _SEH2_GetExceptionCode();
>    _SEH2_YIELD(break);
> }
>   _SEH2_END;

Best regards,
Alex Ionescu