[PATCH 1/1] extrac32: Don't leak argv (Coverity)

[Fabian Maurer]

From: Fabian Maurer

Signed-off-by: Fabian Maurer 
---
 programs/extrac32/extrac32.c | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/programs/extrac32/extrac32.c b/programs/extrac32/extrac32.c
index 3741f16b3cd..d4e7149bdde 100644
--- a/programs/extrac32/extrac32.c
+++ b/programs/extrac32/extrac32.c
@@ -268,29 +268,29 @@ int PASCAL wWinMain(HINSTANCE hInstance, HINSTANCE prev, LPWSTR cmdline, int sho
                 force_mode = TRUE;
                 break;
             case 'L':
-                if ((i + 1) >= argc) return 0;
+                if ((i + 1) >= argc) goto cleanup;
                 if (!GetFullPathNameW(argv[++i], MAX_PATH, path, NULL))
-                    return 0;
+                    goto cleanup;
                 break;
             case 'C':
             case 'E':
             case 'D':
-                if (cmd) return 0;
+                if (cmd) goto cleanup;
                 cmd = check;
                 break;
             default:
-                return 0;
+                goto cleanup;
         }
     }

if (!cabfile)
-        return 0;
+        goto cleanup;

if (cmd == 'C')
     {
-        if ((i + 1) != argc) return 0;
+        if ((i + 1) != argc) goto cleanup;
         if (!GetFullPathNameW(argv[i], MAX_PATH, path, NULL))
-            return 0;
+            goto cleanup;
     }
     else if (!cmd)
         /* Use extraction by default if names of required files presents */
@@ -317,5 +317,7 @@ int PASCAL wWinMain(HINSTANCE hInstance, HINSTANCE prev, LPWSTR cmdline, int sho
             extract(cabfile, path);
             break;
     }
+cleanup:
+    HeapFree(GetProcessHeap(), 0, argv);
     return 0;
 }