Kai Blin : dplayx: Fix possible NULL pointer deference in error path ( Coverity).

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


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

Author: Kai Blin <kai.blin at gmail.com>
Date:   Wed Sep 10 12:03:29 2008 +0200

dplayx: Fix possible NULL pointer deference in error path (Coverity).

---

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

diff --git a/dlls/dplayx/dplayx_global.c b/dlls/dplayx/dplayx_global.c
index f70335b..fc72dce 100644
--- a/dlls/dplayx/dplayx_global.c
+++ b/dlls/dplayx/dplayx_global.c
@@ -889,7 +889,8 @@ HRESULT DPLAYX_SetConnectionSettingsA
   {
     DPLAYX_ReleaseSemaphore();
 
-    ERR("DPSESSIONDESC passed in? Size=%u\n", lpConn->lpSessionDesc->dwSize );
+    ERR("DPSESSIONDESC passed in? Size=%u\n",
+        lpConn->lpSessionDesc?lpConn->lpSessionDesc->dwSize:0 );
 
     return DPERR_INVALIDPARAMS;
   }




More information about the wine-cvs mailing list