Rob Shearman : dplayx: Fix incorrect check in DirectPlay3AImpl_EnumConnections.

Alexandre Julliard julliard at winehq.org
Fri Feb 22 05:49:29 CST 2008


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

Author: Rob Shearman <rob at codeweavers.com>
Date:   Thu Feb 21 16:44:29 2008 +0000

dplayx: Fix incorrect check in DirectPlay3AImpl_EnumConnections.

Dereferencing a function pointer and checking it against NULL doesn't 
make any sense, so remove the check.

---

 dlls/dplayx/dplay.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/dlls/dplayx/dplay.c b/dlls/dplayx/dplay.c
index e3c3562..04044a7 100644
--- a/dlls/dplayx/dplay.c
+++ b/dlls/dplayx/dplay.c
@@ -3669,7 +3669,7 @@ static HRESULT WINAPI DirectPlay3AImpl_EnumConnections
     return DPERR_INVALIDFLAGS;
   }
 
-  if( !lpEnumCallback || !*lpEnumCallback )
+  if( !lpEnumCallback )
   {
      return DPERR_INVALIDPARAMS;
   }




More information about the wine-cvs mailing list