compact-encoding 2.15.0 → 2.15.1

package/index.js

@@ -39,6 +39,7 @@ const uint8 = exports.uint8 = {
     state.end += 1
   },
   encode (state, n) {
+    validateUint(n)
     state.buffer[state.start++] = n
   },
   decode (state) {
@@ -52,6 +53,7 @@ const uint16 = exports.uint16 = {
     state.end += 2
   },
   encode (state, n) {
+    validateUint(n)
     state.buffer[state.start++] = n
     state.buffer[state.start++] = n >>> 8
   },
@@ -69,6 +71,7 @@ const uint24 = exports.uint24 = {
     state.end += 3
   },
   encode (state, n) {
+    validateUint(n)
     state.buffer[state.start++] = n
     state.buffer[state.start++] = n >>> 8
     state.buffer[state.start++] = n >>> 16
@@ -88,6 +91,7 @@ const uint32 = exports.uint32 = {
     state.end += 4
   },
   encode (state, n) {
+    validateUint(n)
     state.buffer[state.start++] = n
     state.buffer[state.start++] = n >>> 8
     state.buffer[state.start++] = n >>> 16
@@ -109,6 +113,7 @@ const uint40 = exports.uint40 = {
     state.end += 5
   },
   encode (state, n) {
+    validateUint(n)
     const r = Math.floor(n / 0x100)
     uint8.encode(state, n)
     uint32.encode(state, r)
@@ -124,6 +129,7 @@ const uint48 = exports.uint48 = {
     state.end += 6
   },
   encode (state, n) {
+    validateUint(n)
     const r = Math.floor(n / 0x10000)
     uint16.encode(state, n)
     uint32.encode(state, r)
@@ -139,6 +145,7 @@ const uint56 = exports.uint56 = {
     state.end += 7
   },
   encode (state, n) {
+    validateUint(n)
     const r = Math.floor(n / 0x1000000)
     uint24.encode(state, n)
     uint32.encode(state, r)
@@ -154,6 +161,7 @@ const uint64 = exports.uint64 = {
     state.end += 8
   },
   encode (state, n) {
+    validateUint(n)
     const r = Math.floor(n / 0x100000000)
     uint32.encode(state, n)
     uint32.encode(state, r)
@@ -420,6 +428,7 @@ exports.bool = {
 const fixed = exports.fixed = function fixed (n) {
   return {
     preencode (state, s) {
+      if (s.byteLength !== n) throw new Error('Incorrect buffer size')
       state.end += n
     },
     encode (state, s) {
@@ -737,3 +746,7 @@ function zigZagEncodeBigInt (n) {
   // 0, -1, 1, -2, 2, ...
   return n < 0n ? (2n * -n) - 1n : n === 0n ? 0n : 2n * n
 }
+
+function validateUint (n) {
+  if ((n >= 0) === false /* Handles NaN as well */) throw new Error('uint must be positive')
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "compact-encoding",
-  "version": "2.15.0",
+  "version": "2.15.1",
   "description": "A series of compact encoding schemes for building small and fast parsers and serializers",
   "main": "index.js",
   "dependencies": {

package/test.js

@@ -565,6 +565,19 @@ test('fixed n', function (t) {
   t.exception(() => fixed.decode(state))
 })
 
+test('error for incorrect buffer sizes when encoding fixed-length buffers', function (t) {
+  const smallbuf = b4a.from('aa', 'hex')
+  const bigBuf = b4a.from('aa'.repeat(500), 'hex')
+
+  t.exception(() => enc.encode(enc.fixed32, smallbuf), /Incorrect buffer size/)
+  t.exception(() => enc.encode(enc.fixed64, smallbuf), /Incorrect buffer size/)
+  t.exception(() => enc.encode(enc.fixed(100), smallbuf), /Incorrect buffer size/)
+
+  t.exception(() => enc.encode(enc.fixed32, bigBuf), /Incorrect buffer size/)
+  t.exception(() => enc.encode(enc.fixed64, bigBuf), /Incorrect buffer size/)
+  t.exception(() => enc.encode(enc.fixed(100), bigBuf), /Incorrect buffer size/)
+})
+
 test('array', function (t) {
   const state = enc.state()
   const arr = enc.array(enc.bool)