Commit 25e6f48
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: #44766
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>1 parent f01bb58 commit 25e6f48
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -929,7 +929,7 @@ function execSync(command, options) { | |||
| 929 | 929 | if (inheritStderr && ret.stderr) | |
| 930 | 930 | process.stderr.write(ret.stderr); | |
| 931 | 931 | ||
| 932 | - const err = checkExecSyncError(ret, opts.args, command); | ||
| 932 | + const err = checkExecSyncError(ret, undefined, command); | ||
| 933 | 933 | ||
| 934 | 934 | if (err) | |
| 935 | 935 | throw err; | |
0 commit comments