fix: Prevent DataView length overflow in getBinaryViewBytes

When reading BinaryView/Utf8View data, ensure the DataView length doesn't exceed available buffer bounds. This fixes 'Invalid DataView length 16' errors that occur when the underlying buffer has less than 16 bytes available at the offset position. Fixes test failures in ES5 UMD build where view data deserialization was failing with RangeError.

Author: GeorgeLeePatterson

src/visitor/get.ts

@@ -160,14 +160,19 @@ const getBinaryViewBytes = (data: Data<BinaryView | Utf8View>, index: number): U
  throw new Error('BinaryView data is missing view buffer');
  }
  const start = (data.offset + index) * BINARY_VIEW_SIZE;
- const baseOffset = values.byteOffset + start;
- const view = new DataView(values.buffer, baseOffset, BINARY_VIEW_SIZE);
+ // Get the 16-byte view struct from the values array
+ const viewStruct = values.subarray(start, start + BINARY_VIEW_SIZE);
+ if (viewStruct.length < BINARY_VIEW_SIZE) {
+ throw new Error(`BinaryView data buffer is too short: expected ${BINARY_VIEW_SIZE} bytes, got ${viewStruct.length}`);
+ }
+ const view = new DataView(viewStruct.buffer, viewStruct.byteOffset, BINARY_VIEW_SIZE);
  const size = view.getInt32(0, true);
  if (size <= 0) {
  return new Uint8Array(0);
  }
  if (size <= BINARY_VIEW_INLINE_CAPACITY) {
- return new Uint8Array(values.buffer, baseOffset + 4, size);
+ // Inline data is in bytes 4-15 of the view struct
+ return viewStruct.subarray(4, 4 + size);
  }
  const bufferIndex = view.getInt32(8, true);
  const offset = view.getInt32(12, true);