fs: fix fs.read when passing null value

PR-URL: #32479
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Evan Lucas <evanlucas@me.com>

Author: himself65

lib/fs.js

@@ -484,10 +484,12 @@ function read(fd, buffer, offset, length, position, callback) {
       callback = offset;
     }
 
-    buffer = options.buffer || Buffer.alloc(16384);
-    offset = options.offset || 0;
-    length = options.length || buffer.length;
-    position = options.position;
+    ({
+      buffer = Buffer.alloc(16384),
+      offset = 0,
+      length = buffer.length,
+      position
+    } = options);
   }
 
   validateBuffer(buffer);

test/parallel/test-fs-read.js

@@ -80,6 +80,14 @@ assert.throws(
   }
 );
 
+['buffer', 'offset', 'length'].forEach((option) =>
+  assert.throws(
+    () => fs.read(fd, {
+      [option]: null
+    }),
+    `not throws when options.${option} is null`
+  ));
+
 assert.throws(
   () => fs.read(null, Buffer.alloc(1), 0, 1, 0),
   {