diff --git a/lib/util.js b/lib/util.js index f6f99f82b45722..1935955ad28c32 100644 --- a/lib/util.js +++ b/lib/util.js @@ -195,8 +195,13 @@ function callbackify(original) { } Object.setPrototypeOf(callbackified, Object.getPrototypeOf(original)); - Object.defineProperties(callbackified, - Object.getOwnPropertyDescriptors(original)); + const descriptors = Object.getOwnPropertyDescriptors(original); + // It is possible to manipulate a functions `length` property. This guards + // against the manipulation. + if (typeof descriptors.length.value === 'number') { + descriptors.length.value++; + } + Object.defineProperties(callbackified, descriptors); return callbackified; } diff --git a/test/parallel/test-util-callbackify.js b/test/parallel/test-util-callbackify.js index c381924e1b5f9d..4d7d46658a8edc 100644 --- a/test/parallel/test-util-callbackify.js +++ b/test/parallel/test-util-callbackify.js @@ -74,6 +74,7 @@ const values = [ } const cbAsyncFn = callbackify(asyncFn); + assert.strictEqual(cbAsyncFn.length, 1); cbAsyncFn(common.mustCall((err, ret) => { assert.strictEqual(ret, undefined); if (err instanceof Error) { @@ -146,6 +147,7 @@ const values = [ } const cbAsyncFn = callbackify(asyncFn); + assert.strictEqual(cbAsyncFn.length, 2); cbAsyncFn(value, common.mustCall((err, ret) => { assert.ifError(err); assert.strictEqual(ret, value); @@ -155,8 +157,16 @@ const values = [ assert.strictEqual(arg, value); return Promise.resolve(arg); } + const obj = {}; + Object.defineProperty(promiseFn, 'length', { + value: obj, + writable: false, + enumerable: false, + configurable: true + }); const cbPromiseFn = callbackify(promiseFn); + assert.strictEqual(promiseFn.length, obj); cbPromiseFn(value, common.mustCall((err, ret) => { assert.ifError(err); assert.strictEqual(ret, value);