buffer: fix typo in `SlowBuffer`
authorVladimir Kurchatkin <vladimir.kurchatkin@gmail.com>
Sat, 8 Mar 2014 14:52:51 +0000 (18:52 +0400)
committerTimothy J Fontaine <tjfontaine@gmail.com>
Mon, 10 Mar 2014 20:23:56 +0000 (13:23 -0700)
lib/buffer.js
test/simple/test-buffer.js

index 084d96f..cd9592d 100644 (file)
@@ -93,7 +93,7 @@ function Buffer(subject, encoding) {
 
 function SlowBuffer(length) {
   length = length >>> 0;
-  if (this.length > kMaxLength) {
+  if (length > kMaxLength) {
     throw new RangeError('Attempt to allocate Buffer larger than maximum ' +
                          'size: 0x' + kMaxLength.toString(16) + ' bytes');
   }
index 55a5ca7..a13bb22 100644 (file)
@@ -24,6 +24,7 @@ var assert = require('assert');
 
 var Buffer = require('buffer').Buffer;
 var SlowBuffer = require('buffer').SlowBuffer;
+var smalloc = require('smalloc');
 
 // counter to ensure unique value is always copied
 var cntr = 0;
@@ -993,3 +994,12 @@ for (var i = 0; i < 5; i++)
 b.fill('ghijk');
 for (var i = 0; i < 5; i++)
   assert.notEqual(d[i], b[i]);
+
+
+assert.throws(function () {
+  new Buffer(smalloc.kMaxLength + 1);
+}, RangeError);
+
+assert.throws(function () {
+  new SlowBuffer(smalloc.kMaxLength + 1);
+}, RangeError);