From 39593767de9ca3cd6125a366b0838e739c366fc8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9e=20Kooi?= <renee@kooi.me>
Date: Wed, 1 May 2019 11:56:33 +0200
Subject: [PATCH] Handle Node.js <= 0.12 Buffers not based on ArrayBuffer

---
 support/types.js   | 5 ++++-
 test/node/types.js | 9 ++++++---
 2 files changed, 10 insertions(+), 4 deletions(-)

diff --git a/support/types.js b/support/types.js
index 6a9899d..b1fedf2 100644
--- a/support/types.js
+++ b/support/types.js
@@ -115,7 +115,10 @@ function isUint8Array(value) {
   } else {
     return (
       ObjectToString(value) === '[object Uint8Array]' ||
-      isBuffer(value)
+      // If it's a Buffer instance _and_ has a `.buffer` property,
+      // this is an ArrayBuffer based buffer; thus it's an Uint8Array
+      // (Old Node.js had a custom non-Uint8Array implementation)
+      isBuffer(value) && value.buffer !== undefined
     );
   }
 }
diff --git a/test/node/types.js b/test/node/types.js
index 31be41f..848c7a1 100644
--- a/test/node/types.js
+++ b/test/node/types.js
@@ -175,6 +175,9 @@ if (isBuggyFirefox) {
   console.log('skipping fake typed array tests because they do not work in FF')
 }
 
+// Old Node.js had a fully custom Buffer implementation, newer are based on ArrayBuffer
+// This is important for the ArrayBuffer and typed array tests
+var isBufferBasedOnArrayBuffer = Buffer.alloc(1).buffer !== undefined;
 {
   var primitive = function primitive() { return true; };
   var arrayBuffer = function arrayBuffer() { return new ArrayBuffer(1); };
@@ -394,7 +397,7 @@ if (isBuggyFirefox) {
 
   var expected = {
     isArrayBufferView: [
-      buffer,
+      isBufferBasedOnArrayBuffer ? buffer : undefined,
       dataView,
       stealthyDataView,
       uint8Array,
@@ -421,7 +424,7 @@ if (isBuggyFirefox) {
       stealthyBigUint64Array
     ],
     isTypedArray: [
-      buffer,
+      isBufferBasedOnArrayBuffer ? buffer : undefined,
       uint8Array,
       stealthyUint8Array,
       uint8ClampedArray,
@@ -446,7 +449,7 @@ if (isBuggyFirefox) {
       stealthyBigUint64Array
     ],
     isUint8Array: [
-      buffer,
+      isBufferBasedOnArrayBuffer ? buffer : undefined,
       uint8Array,
       stealthyUint8Array
     ],