feat: fromBase64 never returns pooled Uint8Arrays

Author: ChALkeR

base64.js

@@ -140,7 +140,9 @@ if (Uint8Array.fromBase64) {
   }
 } else if (haveNativeBuffer) {
   fromBase64impl = (str, isBase64url, padding, format) => {
-    const arr = Buffer.from(str, 'base64')
+    const size = Buffer.byteLength(str, 'base64')
+    const arr = Buffer.allocUnsafeSlow(size) // non-pooled
+    if (arr.base64Write(str) !== size) throw new SyntaxError(E_PADDING)
     // Rechecking by re-encoding is cheaper than regexes on Node.js
     const got = isBase64url ? maybeUnpad(str, padding === false) : maybePad(str, padding !== true)
     const valid = isBase64url ? arr.base64urlSlice(0, arr.length) : arr.base64Slice(0, arr.length)

tests/base64.test.js

@@ -171,3 +171,21 @@ describe('fromBase64', () => {
     }
   })
 })
+
+test('fromBase64 returns non-pooled Uint8Array instances', (t) => {
+  for (let i = 0; i < 256; i++) {
+    t.assert.strictEqual(fromBase64('A'.repeat(128 * 4)).buffer.byteLength, 128 * 3)
+  }
+
+  for (let i = 0; i < 256; i++) {
+    t.assert.strictEqual(fromBase64('A'.repeat(64 * 4)).buffer.byteLength, 64 * 3)
+  }
+
+  for (let i = 0; i < 512; i++) {
+    t.assert.strictEqual(fromBase64('A'.repeat(32 * 4)).buffer.byteLength, 32 * 3)
+  }
+
+  for (let i = 0; i < 512; i++) {
+    t.assert.strictEqual(fromBase64('').buffer.byteLength, 0)
+  }
+})