From cea77caf41ba479aff9d860821ba68629b8df107 Mon Sep 17 00:00:00 2001 From: Moshe Kaplan Date: Mon, 7 Dec 2020 08:04:40 -0500 Subject: [PATCH] cli-main.c: free() argv's memory free() memory allocated for argv to fix Coverity 1108127, which detected it as a resource leak. --- cli_main.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/cli_main.c b/cli_main.c index 61c8c58ae7..a478e3d339 100644 --- a/cli_main.c +++ b/cli_main.c @@ -26,6 +26,7 @@ wmain(int argc, wchar_t *wc_argv[]) { char **argv; int i; + int return_code; argv = (char **)malloc((argc + 1) * sizeof(char *)); if (argv == NULL) { @@ -67,6 +68,11 @@ wmain(int argc, wchar_t *wc_argv[]) * the cli_main.h header file since either "main" or "wmain" can be * defined on Windows, but not both. */ - return real_main(argc, argv); + return_code = real_main(argc, argv); + for (i = 0; i < argc; i++) { + free(argv[i]); + } + free(argv); + return return_code; } #endif