Re: [PATCH] user32: Fix error handling in MapWindowPoints, ClientToScreen and ScreenToClient and add tests for them. (try 2)

[Alexandre Julliard]

Christian Costa  writes:
> @@ -236,20 +236,32 @@ BOOL WINAPI GetClientRect( HWND hwnd, LPRECT rect )
>
> /*******************************************************************
> -
> -
> ClientToScreen (USER32.@)
> -
> -
>         ClientToScreen (USER32.@)
> */
>
> BOOL WINAPI ClientToScreen( HWND hwnd, LPPOINT lppnt )
>  {
> - if (!IsWindow( hwnd ))
> - {
> -
>    SetLastError( ERROR_INVALID_WINDOW_HANDLE );
> -
>    return FALSE;
> - }
> -

You should never use IsWindow(). Handles should be checked at the point
where they are used.