Skip to content

Commit

Permalink
fs: nullish coalescing to respect zero positional reads
Browse files Browse the repository at this point in the history
When the file read position is moved passing zero is
not respected and `null` is used instead. PR fixes the
issues by using nullish coalescing which will return
the rhs only when the lhs is `null` or `undefined`;
respecting the zero.

Fixes: nodejs#40715
Refs: nodejs#40716
  • Loading branch information
mihilmy committed Nov 4, 2021
1 parent 8fce09e commit e1d3e9d
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/internal/fs/promises.js
Original file line number Diff line number Diff line change
Expand Up @@ -521,7 +521,7 @@ async function read(handle, bufferOrOptions, offset, length, position) {
}
offset = bufferOrOptions.offset || 0;
length = buffer.byteLength;
position = bufferOrOptions.position || null;
position = bufferOrOptions.position ?? null;
}

if (offset == null) {
Expand Down
22 changes: 22 additions & 0 deletions test/parallel/test-fs-promises-file-handle-read.js
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,27 @@ async function validateReadNoParams() {
await fileHandle.read();
}

/**
* Validates that the zero position is respected after the position has been
* moved. The test iterates over the xyz chars twice making sure that the values
* are read from the correct position.
*/
async function validateReadWithPositionZero() {
const opts = { useConf: true };
const filePath = fixtures.path('x.txt');
const fileHandle = await open(filePath, 'r');
const expectedSequence = ['x', 'y', 'z'];

for (let i = 0; i < expectedSequence.length * 2; i++) {
const len = 1;
const pos = i % 3;
const buf = Buffer.alloc(len);
const { bytesRead } = await read(fileHandle, buf, 0, len, pos, opts);
assert.strictEqual(bytesRead, len);
assert.strictEqual(buf.toString(), expectedSequence[pos]);
}
}


(async function() {
tmpdir.refresh();
Expand All @@ -78,4 +99,5 @@ async function validateReadNoParams() {
await validateLargeRead({ useConf: false });
await validateLargeRead({ useConf: true });
await validateReadNoParams();
await validateReadWithPositionZero();
})().then(common.mustCall());

0 comments on commit e1d3e9d

Please sign in to comment.