fix(node/fs): node:fs.read and write should accept typed arrays other than Uint8Array (#25030)

Part of #25028.

Our underlying read/write operations in `io` assume the buffer is a
Uint8Array, but we were passing in other typed arrays (in the case above
it was `Int8Array`).

1 files changed, 17 insertions, 9 deletions

diff --git a/ext/node/polyfills/_fs/_fs_read.ts b/ext/node/polyfills/_fs/_fs_read.ts
index e25f41e76..7b3d9fccd 100644
--- a/ext/node/polyfills/_fs/_fs_read.ts
+++ b/ext/node/polyfills/_fs/_fs_read.ts
@@ -9,6 +9,7 @@ import * as io from "ext:deno_io/12_io.js";
 import * as fs from "ext:deno_fs/30_fs.js";
 import { ReadOptions } from "ext:deno_node/_fs/_fs_common.ts";
 import {
+  arrayBufferViewToUint8Array,
   validateOffsetLengthRead,
   validatePosition,
 } from "ext:deno_node/internal/fs/utils.mjs";
@@ -16,6 +17,7 @@ import {
   validateBuffer,
   validateInteger,
 } from "ext:deno_node/internal/validators.mjs";
+import { isArrayBufferView } from "ext:deno_node/internal/util/types.ts";
 
 type readSyncOptions = {
   offset: number;
@@ -38,7 +40,7 @@ export function read(
 ): void;
 export function read(
   fd: number,
-  buffer: Buffer | Uint8Array,
+  buffer: ArrayBufferView,
   offset: number,
   length: number,
   position: number | null,
@@ -46,7 +48,7 @@ export function read(
 ): void;
 export function read(
   fd: number,
-  optOrBufferOrCb?: Buffer | Uint8Array | ReadOptions | Callback,
+  optOrBufferOrCb?: ArrayBufferView | ReadOptions | Callback,
   offsetOrCallback?: number | Callback,
   length?: number,
   position?: number | null,
@@ -75,9 +77,9 @@ export function read(
   }
 
   if (
-    optOrBufferOrCb instanceof Buffer || optOrBufferOrCb instanceof Uint8Array
+    isArrayBufferView(optOrBufferOrCb)
   ) {
-    buffer = optOrBufferOrCb;
+    buffer = arrayBufferViewToUint8Array(optOrBufferOrCb);
   } else if (typeof optOrBufferOrCb === "function") {
     offset = 0;
     buffer = Buffer.alloc(16384);
@@ -86,7 +88,7 @@ export function read(
   } else {
     const opt = optOrBufferOrCb as ReadOptions;
     if (
-      !(opt.buffer instanceof Buffer) && !(opt.buffer instanceof Uint8Array)
+      !isArrayBufferView(opt.buffer)
     ) {
       throw new ERR_INVALID_ARG_TYPE("buffer", [
         "Buffer",
@@ -94,8 +96,12 @@ export function read(
         "DataView",
       ], optOrBufferOrCb);
     }
+    if (opt.buffer === undefined) {
+      buffer = Buffer.alloc(16384);
+    } else {
+      buffer = arrayBufferViewToUint8Array(opt.buffer);
+    }
     offset = opt.offset ?? 0;
-    buffer = opt.buffer ?? Buffer.alloc(16384);
     length = opt.length ?? buffer.byteLength;
     position = opt.position ?? null;
   }
@@ -131,19 +137,19 @@ export function read(
 
 export function readSync(
   fd: number,
-  buffer: Buffer | Uint8Array,
+  buffer: ArrayBufferView,
   offset: number,
   length: number,
   position: number | null,
 ): number;
 export function readSync(
   fd: number,
-  buffer: Buffer | Uint8Array,
+  buffer: ArrayBufferView,
   opt: readSyncOptions,
 ): number;
 export function readSync(
   fd: number,
-  buffer: Buffer | Uint8Array,
+  buffer: ArrayBufferView,
   offsetOrOpt?: number | readSyncOptions,
   length?: number,
   position?: number | null,
@@ -156,6 +162,8 @@ export function readSync(
 
   validateBuffer(buffer);
 
+  buffer = arrayBufferViewToUint8Array(buffer);
+
   if (length == null) {
     length = 0;
   }