Piotr Caban : winex11.drv: Only update drop effect if IDropTarget:: DragOver returned S_OK.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: c96589421a5fdf216daed54f20752e63cb31e1a5
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=c96589421a5fdf216daed54f20...

Author: Piotr Caban <piotr(a)codeweavers.com>
Date:   Mon Dec 21 11:21:22 2015 +0100

winex11.drv: Only update drop effect if IDropTarget::DragOver returned S_OK.

Signed-off-by: Piotr Caban <piotr(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 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 d5881ad..841eaff 100644
--- a/dlls/winex11.drv/xdnd.c
+++ b/dlls/winex11.drv/xdnd.c
@@ -325,7 +325,7 @@ void X11DRV_XDND_PositionEvent( HWND hWnd, XClientMessageEvent *event )
         if (dropTarget)
         {
             hr = IDropTarget_DragOver(dropTarget, MK_LBUTTON, pointl, &effect);
-            if (SUCCEEDED(hr))
+            if (hr != S_OK)
                 XDNDDropEffect = effect;
             else
                 WARN("IDropTarget_DragOver failed, error 0x%08X\n", hr);