nodejs
diff --git a/Collapse file
‎lib/internal/fs/promises.js‎
Copy file name to clipboardExpand all lines: lib/internal/fs/promises.js
+1-1Lines changed: 1 addition & 1 deletion b/Collapse file
‎lib/internal/fs/promises.js‎
Copy file name to clipboardExpand all lines: lib/internal/fs/promises.js
+1-1Lines changed: 1 addition & 1 deletion
diff --git a/Collapse file
‎test/parallel/test-fs-promises-file-handle-read.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-fs-promises-file-handle-read.js
+19Lines changed: 19 additions & 0 deletions b/Collapse file
‎test/parallel/test-fs-promises-file-handle-read.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-fs-promises-file-handle-read.js
+19Lines changed: 19 additions & 0 deletions
@@ -399,7 +399,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) {
 
@@ -68,6 +68,24 @@ async function validateReadNoParams() {
 }
 
 let useConf = false;
+// 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() {
   for (const value of [false, true]) {
@@ -80,4 +98,5 @@ let useConf = false;
           .then(common.mustCall());
   }
   await validateReadNoParams();
+  await validateReadWithPositionZero();
 })().then(common.mustCall());
Original file line number	Diff line number	Diff line change
`@@ -399,7 +399,7 @@ async function read(handle, bufferOrOptions, offset, length, position) {`
`399`	`399`	`}`
`400`	`400`	`offset = bufferOrOptions.offset \|\| 0;`
`401`	`401`	`length = buffer.byteLength;`
`402`		`- position = bufferOrOptions.position \|\| null;`
	`402`	`+ position = bufferOrOptions.position ?? null;`
`403`	`403`	`}`
`404`	`404`
`405`	`405`	`if (offset == null) {`