child_process: fix bad abort signal leak

Move abort signal validation to before spawn is executed so that the file is not leaked. PR-URL: #37257 Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
nodejs · Sep 1, 2021 · 8010c83 · 8010c83
1 parent 9f6bc58
commit 8010c83
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/child_process.js b/lib/child_process.js
@@ -870,11 +870,14 @@ function spawnWithSignal(file, args, options) {
   const opts = options && typeof options === 'object' && ('signal' in options) ?
     { ...options, signal: undefined } :
     options;
-  const child = spawn(file, args, opts);
 
-  if (options && options.signal) {
+  if (options?.signal) {
     // Validate signal, if present
     validateAbortSignal(options.signal, 'options.signal');
+  }
+  const child = spawn(file, args, opts);
+
+  if (options && options.signal) {
     function kill() {
       if (child._handle) {
         child._handle.kill(options.killSignal || 'SIGTERM');