[ros-diffs] [tkreuzer] 58434: [WIN32K] - Fix a bug in EngLoadModuleEx - Fix a bug in co_IntLoadSysMenuTemplate - Fix / improve a number of annotations - Improve code in NtGdiExtGetObjectW - Check return value ...

[Hermès BÉLUSCA - MAÏTO]

Hi Timo !!

----------------------------------------------------------------------------
----------------------------------------------------------------------------
-----------------------------

New Revision: 58434
URL: http://svn.reactos.org/svn/reactos?rev=58434&view=rev

Modified: trunk/reactos/win32ss/gdi/ntgdi/text.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/text.c?re
v=58434&r1=58433&r2=58434&view=diff
============================================================================
==
--- trunk/reactos/win32ss/gdi/ntgdi/text.c [iso-8859-1] (original)
+++ trunk/reactos/win32ss/gdi/ntgdi/text.c [iso-8859-1] Tue Mar  5 08:47:51
2013

@@ -540,15 +538,11 @@
             }
             _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
             {
-                Status = _SEH2_GetExceptionCode();
+                SetLastNtError(_SEH2_GetExceptionCode());
+                return FALSE;
             }
             _SEH2_END

-            if (!NT_SUCCESS(Status))
-            {
-                SetLastNtError(Status);
-                return FALSE;
-            }
             return TRUE;
         }
     }

----------------------------------------------------------------------------
----------------------------------------------------------------------------
-----------------------------

Are you sure you want to return "directly" from the _SEH2_EXCEPT block,
instead of doing :

_SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
             {
-                Status = _SEH2_GetExceptionCode();
+                SetLastNtError(_SEH2_GetExceptionCode());
+               _SEH2_YIELD(return FALSE);
             }

Instead ??

Hermès

[Timo Kreuzer]

Oh sorry, I forgot that PSEH2 is broken. Neither PSEH3 nor "the 
original" need that.
I'll fix it.

Am 05.03.2013 14:28, schrieb Hermès BÉLUSCA - MAÏTO:
> Hi Timo !!
>
> New Revision: 58434
> URL: http://svn.reactos.org/svn/reactos?rev=58434&view=rev
>
> Modified: trunk/reactos/win32ss/gdi/ntgdi/text.c
> URL:
> http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/gdi/ntgdi/text.c?re
> v=58434&r1=58433&r2=58434&view=diff
> ============================================================================
> ==
> --- trunk/reactos/win32ss/gdi/ntgdi/text.c [iso-8859-1] (original)
> +++ trunk/reactos/win32ss/gdi/ntgdi/text.c [iso-8859-1] Tue Mar  5 08:47:51
> 2013
>
> @@ -540,15 +538,11 @@
>               }
>               _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
>               {
>            Status = _SEH2_GetExceptionCode();
>            SetLastNtError(_SEH2_GetExceptionCode());
>            return FALSE;
>         }
>         _SEH2_END
>
>        if (!NT_SUCCESS(Status))
>        {
>            SetLastNtError(Status);
>            return FALSE;
>        }
>         return TRUE;
>     }
> }
>
> Are you sure you want to return "directly" from the _SEH2_EXCEPT block,
> instead of doing :
>           _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER)
>           {
>            Status = _SEH2_GetExceptionCode();
>            SetLastNtError(_SEH2_GetExceptionCode());
>           _SEH2_YIELD(return FALSE);
>         }
>
> Instead ??
>
> Hermès
>
> Ros-dev mailing list
> [e-mail]
> http://www.reactos.org/mailman/listinfo/ros-dev