[PATCH] winex11: fix HRESULT handling in XdndPosition

[Matteo Bruni]

2016-02-23 0:02 GMT+01:00 Shu Yokoyama <yokoyama at graco.c.u-tokyo.ac.jp>:
> https://bugs.winehq.org/show_bug.cgi?id=40194
>
> Signed-off-by: Shu Yokoyama <yokoyama at graco.c.u-tokyo.ac.jp>
> ---
>  dlls/winex11.drv/xdnd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/dlls/winex11.drv/xdnd.c b/dlls/winex11.drv/xdnd.c
> index 8b2c582..c83caea 100644
> --- a/dlls/winex11.drv/xdnd.c
> +++ b/dlls/winex11.drv/xdnd.c
> @@ -328,7 +328,7 @@ void X11DRV_XDND_PositionEvent( HWND hWnd, XClientMessageEvent *event )
>          if (dropTarget)
>          {
>              hr = IDropTarget_DragOver(dropTarget, MK_LBUTTON, pointl, &effect);
> -            if (hr != S_OK)
> +            if (hr == S_OK)
>                  XDNDDropEffect = effect;
>              else
>                  WARN("IDropTarget_DragOver failed, error 0x%08X\n", hr);
> --
> 2.5.0

Signed-off-by: Matteo Bruni <mbruni at codeweavers.com>