[PATCH] winex11: fix HRESULT handling in XdndPosition

Shu Yokoyama yokoyama at graco.c.u-tokyo.ac.jp
Mon Feb 22 16:41:29 CST 2016


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



More information about the wine-patches mailing list