msvcrt: Use free() on msvcrt_wstrdupa() results (Coverity).

Signed-off-by: Nikolay Sivov <nsivov@codeweavers.com> Signed-off-by: Piotr Caban <piotr@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

msvcrt: Use free() on msvcrt_wstrdupa() results (Coverity).
0eaa43f6 · Nikolay Sivov · Alexandre Julliard · 2b99a4eb · 0eaa43f6
Commit 0eaa43f6 authored Sep 14, 2021 by Nikolay Sivov Committed by Alexandre Julliard Sep 15, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 4 deletions

process.c dlls/msvcrt/process.c +4 -4

No files found.
--- a/dlls/msvcrt/process.c
+++ b/dlls/msvcrt/process.c
@@ -1165,14 +1165,14 @@ FILE* CDECL _popen(const char* command, const char* mode)
  if (!(cmdW = msvcrt_wstrdupa(command))) return NULL;
  if (!(modeW = msvcrt_wstrdupa(mode)))
  {
-    HeapFree(GetProcessHeap(), 0, cmdW);
+    free(cmdW);
    return NULL;
  }

  ret = _wpopen(cmdW, modeW);

-  HeapFree(GetProcessHeap(), 0, cmdW);
-  HeapFree(GetProcessHeap(), 0, modeW);
+  free(cmdW);
+  free(modeW);
  return ret;
 }

@@ -1274,7 +1274,7 @@ int CDECL system(const char* cmd)
  if ((cmdW = msvcrt_wstrdupa(cmd)))
  {
    res = _wsystem(cmdW);
-    HeapFree(GetProcessHeap(), 0, cmdW);
+    free(cmdW);
  }
  return res;
 }