tarantool / tarantool / 16992294404

88%

master: 88%

push

github

box: fix key corruption when pagination by tuple is used

Function `iterator_pos_set` works completely incorrectly when tuple
is passed to the option `after`.

Firstly, before extracting position, we call `ibuf:consume()` for
the memory allocated before position (this method is actually ASAN
poisoning) - that's not correct since consumed memory contains query key
that will be used later. Let's remove this call.

Secondly, new allocation on ibuf can lead to its reallocation, so
pointers to old allocations can be invalidated. Let's set the pointers
to the key after all allocations are done, just like in `index:quantile()`.

What's about performance, `perf/lua/box_select.lua` shows no degradation.

Closes #11221
Closes #11648

NO_DOC=bugfix

(cherry picked from commit 7426f5912)

70566 of 124329 branches covered (56.76%)

103767 of 118510 relevant lines covered (87.56%)

1363039.14 hits per line