sea: fix memory leak detected by asan

Signed-off-by: Darshan Sen <[email protected]> PR-URL: #47309 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Mohammed Keyvanzadeh <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Jul 6, 2023 · 7306b0f · 7306b0f
1 parent 3924893
commit 7306b0f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 16 deletions.
diff --git a/src/node_sea.cc b/src/node_sea.cc
@@ -91,19 +91,13 @@ std::tuple<int, char**> FixupArgsForSEA(int argc, char** argv) {
   // Repeats argv[0] at position 1 on argv as a replacement for the missing
   // entry point file path.
   if (IsSingleExecutable()) {
-    char** new_argv = new char*[argc + 2];
-    int new_argc = 0;
-    new_argv[new_argc++] = argv[0];
-    new_argv[new_argc++] = argv[0];
-
-    for (int i = 1; i < argc; ++i) {
-      new_argv[new_argc++] = argv[i];
-    }
-
-    new_argv[new_argc] = nullptr;
-
-    argc = new_argc;
-    argv = new_argv;
+    static std::vector<char*> new_argv;
+    new_argv.reserve(argc + 2);
+    new_argv.emplace_back(argv[0]);
+    new_argv.insert(new_argv.end(), argv, argv + argc);
+    new_argv.emplace_back(nullptr);
+    argc = new_argv.size() - 1;
+    argv = new_argv.data();
   }
 
   return {argc, argv};

diff --git a/test/parallel/test-single-executable-application.js b/test/parallel/test-single-executable-application.js
@@ -16,9 +16,6 @@ if (!process.config.variables.single_executable_application)
 if (!['darwin', 'win32', 'linux'].includes(process.platform))
   common.skip(`Unsupported platform ${process.platform}.`);
 
-if (process.platform === 'linux' && process.config.variables.asan)
-  common.skip('Running the resultant binary fails with `Segmentation fault (core dumped)`.');
-
 if (process.platform === 'linux' && process.config.variables.is_debug === 1)
   common.skip('Running the resultant binary fails with `Couldn\'t read target executable"`.');