Marcus Meissner : mciqtz32: Fixed boolean condition for null string check ( Coverity 913).

Alexandre Julliard julliard at winehq.org
Mon Apr 6 09:00:32 CDT 2009


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

Author: Marcus Meissner <marcus at jet.franken.de>
Date:   Sat Apr  4 17:43:57 2009 +0200

mciqtz32: Fixed boolean condition for null string check (Coverity 913).

---

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

diff --git a/dlls/mciqtz32/mciqtz.c b/dlls/mciqtz32/mciqtz.c
index 507956c..874b876 100644
--- a/dlls/mciqtz32/mciqtz.c
+++ b/dlls/mciqtz32/mciqtz.c
@@ -172,7 +172,7 @@ static DWORD MCIQTZ_mciOpen(UINT wDevID, DWORD dwFlags,
         goto err;
     }
 
-    if (!lpOpenParms->lpstrElementName && !lstrlenW(lpOpenParms->lpstrElementName)) {
+    if (!lpOpenParms->lpstrElementName || !lpOpenParms->lpstrElementName[0]) {
         TRACE("Invalid filename specified\n");
         goto err;
     }




More information about the wine-cvs mailing list