Small MSVIDEO fix...

Lionel Ulmer lionel.ulmer at free.fr
Sat Dec 14 10:39:43 CST 2002


Hi all,

I really wonder how this code could work... If it was NULL, it continued and
crashed. If it was valid, it stop and returned an error code to the
application...

                  Lionel

Changelog:
  Fix bad logic in == NULL test

-- 
		 Lionel Ulmer - http://www.bbrox.org/
-------------- next part --------------
Index: dlls/msvideo/drawdib.c
===================================================================
RCS file: /home/wine/wine/dlls/msvideo/drawdib.c,v
retrieving revision 1.18
diff -u -r1.18 drawdib.c
--- dlls/msvideo/drawdib.c	20 Nov 2002 19:47:08 -0000	1.18
+++ dlls/msvideo/drawdib.c	14 Dec 2002 16:38:17 -0000
@@ -299,7 +299,7 @@
           hdd, hdc, xDst, yDst, dxDst, dyDst, lpbi, lpBits, xSrc, ySrc, dxSrc, dySrc, (DWORD)wFlags);
 
     whdd = MSVIDEO_GetHddPtr(hdd);
-    if (whdd) return FALSE;
+    if (!whdd) return FALSE;
 
     if (wFlags & ~(DDF_SAME_HDC | DDF_SAME_DRAW | DDF_NOTKEYFRAME | DDF_UPDATE | DDF_DONTDRAW))
         FIXME("wFlags == 0x%08lx not handled\n", (DWORD)wFlags);


More information about the wine-patches mailing list