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

make sure setArray isn't copying mapped data addresses by accident in unionInPlace

Browse source
This commit is contained in:
Seebs 2020-02-07 12:39:31 -06:00 • committed by Matt Jaffee
parent 99d865c2ea
commit 7841a660a8
No known key found for this signature in database
GPG key ID: 08A3DFFF987B11BF
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
roaring/roaring.go
View file

@ -3779,7 +3779,10 @@ func unionArrayArrayInPlace(a, b *Container) *Container {
// for InPlace, we actually want to ensure that
// we update a, as long as it's not frozen.
a = a.Thaw()
a.setArray(b.array())
// ... but we also want to be sure we don't end up
// copying in a mapped object into our not-mapped
// object.
a.setArrayMaybeCopy(b.array(), b.Mapped())
return a.optimize()
}
return a