1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

buffer: fix offset checks

Browse Source 
Fixed offset checks in Buffer.readInt32LE() and Buffer.readInt32BE()
functions.
This commit is contained in:
Łukasz Walukiewicz 2013-04-05 16:48:18 +02:00 committed by isaacs
parent c93af860a0
commit 2e28832660
2 changed files with 35 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/buffer.js
View File

@ -737,14 +737,14 @@ function readInt32(buffer, offset, isBigEndian) {
Buffer.prototype.readInt32LE = function(offset, noAssert) {
if (!noAssert)
checkOffset(offset, 2, this.length);
checkOffset(offset, 4, this.length);
return readInt32(this, offset, false);
};
Buffer.prototype.readInt32BE = function(offset, noAssert) {
if (!noAssert)
checkOffset(offset, 2, this.length);
checkOffset(offset, 4, this.length);
return readInt32(this, offset, true);
};

33
test/simple/test-buffer.js
View File

@ -908,6 +908,39 @@ assert.throws(function() {
buf.writeFloatLE(0.0, -1);
}, /offset is not uint/);
// offset checks
var buf = new Buffer(0);
assert.throws(function() { buf.readUInt8(0); }, /beyond buffer length/);
assert.throws(function() { buf.readInt8(0); }, /beyond buffer length/);
[16, 32].forEach(function(bits) {
var buf = new Buffer(bits / 8 - 1);
assert.throws(
function() { buf['readUInt' + bits + 'BE'](0); },
/beyond buffer length/,
'readUInt' + bits + 'BE'
);
assert.throws(
function() { buf['readUInt' + bits + 'LE'](0); },
/beyond buffer length/,
'readUInt' + bits + 'LE'
);
assert.throws(
function() { buf['readInt' + bits + 'BE'](0); },
/beyond buffer length/,
'readInt' + bits + 'BE()'
);
assert.throws(
function() { buf['readInt' + bits + 'LE'](0); },
/beyond buffer length/,
'readInt' + bits + 'LE()'
);
});
// SlowBuffer sanity checks.
assert.throws(function() {