diff options
author | ZYSzys <17367077526@163.com> | 2019-02-10 12:16:53 +0800 |
---|---|---|
committer | Luigi Pinca <luigipinca@gmail.com> | 2019-02-10 09:22:36 +0100 |
commit | 70f4f08a9f0902056ac47271f9d3a761fcee4715 (patch) | |
tree | 18594addd9871c782463f4f52d2d1df74ce2d6ae /test/parallel | |
parent | e154176b7cc397399ccb939b920b308f8d5cb874 (diff) | |
download | android-node-v8-70f4f08a9f0902056ac47271f9d3a761fcee4715.tar.gz android-node-v8-70f4f08a9f0902056ac47271f9d3a761fcee4715.tar.bz2 android-node-v8-70f4f08a9f0902056ac47271f9d3a761fcee4715.zip |
fs: harden validation of start option in createWriteStream
PR-URL: https://github.com/nodejs/node/pull/25579
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Diffstat (limited to 'test/parallel')
-rw-r--r-- | test/parallel/test-file-write-stream3.js | 18 |
1 files changed, 17 insertions, 1 deletions
diff --git a/test/parallel/test-file-write-stream3.js b/test/parallel/test-file-write-stream3.js index db16318a5c..618fbdf884 100644 --- a/test/parallel/test-file-write-stream3.js +++ b/test/parallel/test-file-write-stream3.js @@ -161,6 +161,7 @@ function run_test_3() { assert.strictEqual(fileData, fileDataExpected_3); run_test_4(); + run_test_5(); }); file.on('error', function(err) { @@ -184,7 +185,22 @@ const run_test_4 = common.mustCall(function() { const err = { code: 'ERR_OUT_OF_RANGE', message: 'The value of "start" is out of range. ' + - 'It must be >= 0. Received {start: -5}', + 'It must be >= 0 and <= 2 ** 53 - 1. Received -5', + type: RangeError + }; + common.expectsError(fn, err); +}); + + +const run_test_5 = common.mustCall(function() { + // Error: start must be <= 2 ** 53 - 1 + const fn = () => { + fs.createWriteStream(filepath, { start: 2 ** 53, flags: 'r+' }); + }; + const err = { + code: 'ERR_OUT_OF_RANGE', + message: 'The value of "start" is out of range. ' + + 'It must be >= 0 and <= 2 ** 53 - 1. Received 9007199254740992', type: RangeError }; common.expectsError(fn, err); |