Marcus Meissner : mciavi32: Removed duplicate strlen/NULL check (Coverity).

[Alexandre Julliard]

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

Author: Marcus Meissner <meissner(a)suse.de>
Date:   Sun Nov  8 10:24:32 2009 +0100

mciavi32: Removed duplicate strlen/NULL check (Coverity).

---

 dlls/mciavi32/mciavi.c |   37 +++++++++++++++++--------------------
 1 files changed, 17 insertions(+), 20 deletions(-)

diff --git a/dlls/mciavi32/mciavi.c b/dlls/mciavi32/mciavi.c
index 4e5faac..d268ee7 100644
--- a/dlls/mciavi32/mciavi.c
+++ b/dlls/mciavi32/mciavi.c
@@ -250,29 +250,26 @@ static	DWORD	MCIAVI_mciOpen(UINT wDevID, DWORD dwFlags,
 	     * contains the hFile value ?
 	     */
 	    dwRet = MCIERR_UNRECOGNIZED_COMMAND;
-	} else if (strlenW(lpOpenParms->lpstrElementName) > 0) {
+	} else if (lpOpenParms->lpstrElementName && lpOpenParms->lpstrElementName[0]) {
 	    /* FIXME : what should be done id wma->hFile is already != 0, or the driver is playin' */
 	    TRACE("MCI_OPEN_ELEMENT %s!\n", debugstr_w(lpOpenParms->lpstrElementName));
 
-            if (lpOpenParms->lpstrElementName && (strlenW(lpOpenParms->lpstrElementName) > 0))
-            {
-                wma->lpFileName = HeapAlloc(GetProcessHeap(), 0, (strlenW(lpOpenParms->lpstrElementName) + 1) * sizeof(WCHAR));
-                strcpyW(wma->lpFileName, lpOpenParms->lpstrElementName);
-
-		wma->hFile = mmioOpenW(lpOpenParms->lpstrElementName, NULL,
-				       MMIO_ALLOCBUF | MMIO_DENYWRITE | MMIO_READ);
-
-		if (wma->hFile == 0) {
-		    WARN("can't find file=%s!\n", debugstr_w(lpOpenParms->lpstrElementName));
-		    dwRet = MCIERR_FILE_NOT_FOUND;
-		} else {
-		    if (!MCIAVI_GetInfo(wma))
-			dwRet = MCIERR_INVALID_FILE;
-		    else if (!MCIAVI_OpenVideo(wma))
-			dwRet = MCIERR_CANNOT_LOAD_DRIVER;
-		    else if (!MCIAVI_CreateWindow(wma, dwFlags, lpOpenParms))
-			dwRet = MCIERR_CREATEWINDOW;
-		}
+            wma->lpFileName = HeapAlloc(GetProcessHeap(), 0, (strlenW(lpOpenParms->lpstrElementName) + 1) * sizeof(WCHAR));
+            strcpyW(wma->lpFileName, lpOpenParms->lpstrElementName);
+
+	    wma->hFile = mmioOpenW(lpOpenParms->lpstrElementName, NULL,
+				   MMIO_ALLOCBUF | MMIO_DENYWRITE | MMIO_READ);
+
+	    if (wma->hFile == 0) {
+		WARN("can't find file=%s!\n", debugstr_w(lpOpenParms->lpstrElementName));
+		dwRet = MCIERR_FILE_NOT_FOUND;
+	    } else {
+		if (!MCIAVI_GetInfo(wma))
+		    dwRet = MCIERR_INVALID_FILE;
+		else if (!MCIAVI_OpenVideo(wma))
+		    dwRet = MCIERR_CANNOT_LOAD_DRIVER;
+		else if (!MCIAVI_CreateWindow(wma, dwFlags, lpOpenParms))
+		    dwRet = MCIERR_CREATEWINDOW;
 	    }
 	} else {
 	    FIXME("Don't record yet\n");