Added a test for string plausibility checking

When a string length within a variable header would exceed the overall packet length, the string parsing should return an error
dpatekar · Jan 14, 2016 · 2c5b0cc · 2c5b0cc
1 parent da768a7
commit 2c5b0cc
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/test.js b/test.js
@@ -919,3 +919,16 @@ testParseError('invalid protocol version', new Buffer([
   0, 4, //Client id length
   116, 101, 115, 116 // Client id
 ]))
+
+testParseError('cannot parse protocol id', new Buffer([
+  16, 8,
+  0, 15,
+  77, 81, 73, 115, 100, 112,
+  77, 81, 73, 115, 100, 112,
+  77, 81, 73, 115, 100, 112,
+  77, 81, 73, 115, 100, 112,
+  77, 81, 73, 115, 100, 112,
+  77, 81, 73, 115, 100, 112,
+  77, 81, 73, 115, 100, 112,
+  77, 81, 73, 115, 100, 112
+]))