From 785481149d59fddead9007d469e2578204f24cfb Mon Sep 17 00:00:00 2001 From: cjihrig Date: Fri, 23 Jan 2015 17:18:55 -0500 Subject: [PATCH] child_process: clone spawn options argument spawnSync() modifies the options argument. This commit makes a copy of options before any modifications occur. Fixes: https://github.com/iojs/io.js/issues/576 PR-URL: https://github.com/iojs/io.js/pull/579 Reviewed-By: Bert Belder --- lib/child_process.js | 1 + test/common.js | 11 +++++++++++ test/parallel/test-child-process-stdio.js | 4 ++++ 3 files changed, 16 insertions(+) diff --git a/lib/child_process.js b/lib/child_process.js index 0d73beb..9226d12 100644 --- a/lib/child_process.js +++ b/lib/child_process.js @@ -931,6 +931,7 @@ function normalizeSpawnArguments(file /*, args, options*/) { else if (!util.isObject(options)) throw new TypeError('options argument must be an object'); + options = util._extend({}, options); args.unshift(file); var env = options.env || process.env; diff --git a/test/common.js b/test/common.js index b9b03c7..0cd9912 100644 --- a/test/common.js +++ b/test/common.js @@ -97,6 +97,17 @@ exports.spawnCat = function(options) { }; +exports.spawnSyncCat = function(options) { + var spawnSync = require('child_process').spawnSync; + + if (process.platform === 'win32') { + return spawnSync('more', [], options); + } else { + return spawnSync('cat', [], options); + } +}; + + exports.spawnPwd = function(options) { var spawn = require('child_process').spawn; diff --git a/test/parallel/test-child-process-stdio.js b/test/parallel/test-child-process-stdio.js index 2e1875a..7292007 100644 --- a/test/parallel/test-child-process-stdio.js +++ b/test/parallel/test-child-process-stdio.js @@ -13,3 +13,7 @@ child = common.spawnPwd(options); assert.equal(child.stdout, null); assert.equal(child.stderr, null); + +options = {stdio: 'ignore'}; +child = common.spawnSyncCat(options); +assert.deepEqual(options, {stdio: 'ignore'}); -- 2.7.4