[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?…
==============================================================================
--- 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, dgorbachev(a)svn.reactos.org 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?…
>  = 
>  = 
>  = 
>  = 
>  = 
>  = 
>  = 
>  = 
>  ======================================================================
>  --- 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