From f9fae94b98005a687935ebd0814471592e2bc5f7 Mon Sep 17 00:00:00 2001
From: cjihrig <cjihrig@gmail.com>
Date: Fri, 23 Sep 2022 20:59:52 -0400
Subject: [PATCH] child_process: remove lookup of undefined property

execSync() passes opts.args to checkExecSyncError(). There are
two problems:

1. opts.args appears to always be undefined.
2. Even if it was defined, checkExecSyncError() would not use
   it since the command input is guaranteed to be defined.

This commit updates the code to pass undefined instead.

PR-URL: https://github.com/nodejs/node/pull/44766
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
---
 lib/child_process.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/child_process.js b/lib/child_process.js
index 364f908158c0c8..a4def6acce9b2c 100644
--- a/lib/child_process.js
+++ b/lib/child_process.js
@@ -929,7 +929,7 @@ function execSync(command, options) {
   if (inheritStderr && ret.stderr)
     process.stderr.write(ret.stderr);
 
-  const err = checkExecSyncError(ret, opts.args, command);
+  const err = checkExecSyncError(ret, undefined, command);
 
   if (err)
     throw err;