Skip to content

Commit 67f070d

Browse files
committed
Ensure unused bits being zero in writeVarint32, see #13
1 parent 31c9c89 commit 67f070d

File tree

7 files changed

+7
-7
lines changed

7 files changed

+7
-7
lines changed

dist/ByteBufferAB.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1063,7 +1063,7 @@
10631063
b = (value >> 21) | 0x80;
10641064
this.view.setUint8(offset+3, b);
10651065
if (value >= 1 << 28) {
1066-
this.view.setUint8(offset+4, (value >> 28) & 0x7F);
1066+
this.view.setUint8(offset+4, (value >> 28) & 0x0F);
10671067
size = 5;
10681068
} else {
10691069
this.view.setUint8(offset+3, b & 0x7F);

dist/ByteBufferAB.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/ByteBufferAB.min.js.gz

2 Bytes
Binary file not shown.

dist/ByteBufferAB.min.map

Lines changed: 1 addition & 1 deletion
Large diffs are not rendered by default.

dist/ByteBufferNB.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1219,7 +1219,7 @@ module.exports = (function() {
12191219
b = (value >> 21) | 0x80;
12201220
this.buffer[offset+3] = b;
12211221
if (value >= 1 << 28) {
1222-
this.buffer[offset+4] = (value >> 28) & 0x7F;
1222+
this.buffer[offset+4] = (value >> 28) & 0x0F;
12231223
size = 5;
12241224
} else {
12251225
this.buffer[offset+3] = b & 0x7F;

src/types/varints/varint32.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,9 +88,9 @@ ByteBuffer.prototype.writeVarint32 = function(value, offset) {
8888
this.view.setUint8(offset+3, b);
8989
if (value >= 1 << 28) {
9090
//? if (NODE)
91-
this.buffer[offset+4] = (value >> 28) & 0x7F;
91+
this.buffer[offset+4] = (value >> 28) & 0x0F;
9292
//? else
93-
this.view.setUint8(offset+4, (value >> 28) & 0x7F);
93+
this.view.setUint8(offset+4, (value >> 28) & 0x0F);
9494
size = 5;
9595
} else {
9696
//? if (NODE)

tests/suite.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -525,7 +525,7 @@ function makeSuite(ByteBuffer) {
525525
["Uint64" , null , 8 , new Long(0xFFFFFFFE, 0xFFFFFFFF, false) , new Long(0xFFFFFFFE, 0xFFFFFFFF, true) , "fffffffffffffffe" ],
526526

527527
// name | alias | size | input | output | representation
528-
["Varint32", null , 5 , 0xFFFFFFFE , -2 , "feffffff7f" ],
528+
["Varint32", null , 5 , 0xFFFFFFFE , -2 , "feffffff0f" ],
529529
["Varint64", null , 10 , new Long(0xFFFFFFFE, 0xFFFFFFFF, true) , new Long(0xFFFFFFFE, 0xFFFFFFFF, false) , "feffffffffffffffff01"]
530530
];
531531

0 commit comments

Comments
 (0)