scriptoriumadmin/featurebase
1
0
Fork 0
mirror of https://github.com/featurebasedb/featurebase.git synced 2026-10-09 20:37:52 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Rearranged if statement and declared maxOpSize value

Browse source
This commit is contained in:
Ashley Svetlik 2019-06-17 08:36:56 -05:00
parent 734daf79ee
commit 413492552c
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
roaring/roaring.go
View file

@ -3945,6 +3945,7 @@ func (op *op) WriteTo(w io.Writer) (n int64, err error) {
}
var minOpSize = 13
var maxOpSize = math.MaxInt64/8 - 13
// UnmarshalBinary decodes data into an op.
func (op *op) UnmarshalBinary(data []byte) error {
@ -3957,15 +3958,14 @@ func (op *op) UnmarshalBinary(data []byte) error {
// op.value will actually contain the length of values for batch ops
op.value = binary.LittleEndian.Uint64(data[1:9])
if math.MaxInt64/8-13 < int(op.value){
return fmt.Errorf("too big")
}
// Verify checksum.
h := fnv.New32a()
_, _ = h.Write(data[0:9])
if op.typ > 1 {
if maxOpSize < int(op.value){
return fmt.Errorf("too big")
}
if len(data) < int(13+op.value*8) {
return fmt.Errorf("op data truncated - expected %d, got %d", 13+op.value*8, len(data))
}