tarantool / tarantool / 13397747737

86%

master: 88%

Build Type

push

github

Committed by sergepetrenko

Commit Message

memtx: fix a crash caused by mhash misusage in MVCC

We use mhash in memtx MVCC to store trackers of reads that have read
nothing, we call them point holes. When handling a write to such hole,
the tracker should be deleted because we can use the new tuple to store
reads instead. Deletion flow in mhash is: we find bucket of the element
with `find`, then we free the bucket with `del`. It can seem that the
element is not needed on `del` because bucket id is used. However, it
can be used on incremental resize of mhash. And, since we delete the
point holes before releasing the bucket in mhash, in the rare case of
incremental resize Tarantool will be aborted by mhash consistency check.
The commit fixes the problem - simply release the bucket before deleting
the object.

Along the way, the commit fixes another misusage that doesn't actually
break something. Method `put_slot` is internal so shouldn't be used
manually - let's use more convenient `get` instead.

Closes #11022

NO_DOC=bugfix

(cherry picked from commit 7c73e24de)

Run Details

63458 of 114790 branches covered (55.28%)

4 of 4 new or added lines in 1 file covered. (100.0%)

18 existing lines in 8 files now uncovered.

94415 of 109719 relevant lines covered (86.05%)

1892614.98 hits per line