archive/node
1
0
Fork 0
mirror of https://github.com/nodejs/node.git synced 2025-08-16 14:18:44 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

fs: add validateBuffer to improve error

Browse source 
PR-URL: https://github.com/nodejs/node/pull/44769
Reviewed-By: LiviaMedeiros <livia@cirno.name>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
This commit is contained in:
Hirotaka Tagawa / wafuwafu13 2022-09-28 17:44:39 +09:00 • committed by GitHub
parent 8ec7bfd9c5
commit bb4891d8d4
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
lib/fs.js
View file

@ -632,7 +632,7 @@ function read(fd, buffer, offsetOrOptions, length, position, callback) {
} }
({ ({
offset = 0, offset = 0,
length = buffer.byteLength - offset, length = buffer?.byteLength - offset,
position = null, position = null,
} = params ?? kEmptyObject); } = params ?? kEmptyObject);
} }

2
test/parallel/test-fs-read.js
View file

@ -79,7 +79,7 @@ assert.throws(
assert.throws( assert.throws(
() => fs.read(fd, { buffer: null }, common.mustNotCall()), () => fs.read(fd, { buffer: null }, common.mustNotCall()),
/TypeError: Cannot read properties of null \(reading 'byteLength'\)/, { code: 'ERR_INVALID_ARG_TYPE' },
'throws when options.buffer is null' 'throws when options.buffer is null'
); );