Kai Blin : dplayx: Fix possible NULL pointer dereference (Coverity).

Alexandre Julliard julliard at winehq.org
Wed Sep 10 06:01:52 CDT 2008


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

Author: Kai Blin <kai.blin at gmail.com>
Date:   Wed Sep 10 11:20:09 2008 +0200

dplayx: Fix possible NULL pointer dereference (Coverity).

---

 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 ce9cdb6..b349462 100644
--- a/dlls/dplayx/dplay.c
+++ b/dlls/dplayx/dplay.c
@@ -3032,7 +3032,7 @@ static HRESULT WINAPI DP_IF_Receive
   }
 
   /* Copy into the provided buffer */
-  CopyMemory( lpData, lpMsg->msg, *lpdwDataSize );
+  if (lpData) CopyMemory( lpData, lpMsg->msg, *lpdwDataSize );
 
   return DP_OK;
 }




More information about the wine-cvs mailing list