shlwapi: Always close find handle (Coverity).

14dab8ab · Nikolay Sivov · Alexandre Julliard · 0efa2331 · 14dab8ab
Commit 14dab8ab authored Apr 10, 2014 by Nikolay Sivov Committed by Alexandre Julliard Apr 10, 2014
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 7 deletions

path.c dlls/shlwapi/path.c +5 -7

No files found.
--- a/dlls/shlwapi/path.c
+++ b/dlls/shlwapi/path.c
@@ -3885,16 +3885,14 @@ BOOL WINAPI PathIsDirectoryEmptyW(LPCWSTR lpszPath)

  strcpyW(szSearch + dwLen, szAllFiles);
  hfind = FindFirstFileW(szSearch, &find_data);
-
-  if (hfind != INVALID_HANDLE_VALUE &&
-      find_data.cFileName[0] == '.' &&
-      find_data.cFileName[1] == '.')
+  if (hfind != INVALID_HANDLE_VALUE)
  {
-    /* The only directory entry should be the parent */
-    if (!FindNextFileW(hfind, &find_data))
-      retVal = TRUE;
+    if (find_data.cFileName[0] == '.' && find_data.cFileName[1] == '.')
+      /* The only directory entry should be the parent */
+      retVal = !FindNextFileW(hfind, &find_data);
    FindClose(hfind);
  }
+
  return retVal;
 }