tarantool / luajit / 6008722489

89%

tarantool/master: 93%

push

github

Fix maxslots when recording BC_VARG, part 3.

Thanks to Peter Cawley.

(cherry-picked from commit abb27c777)

In case, when `BC_VARG` sets the VARG slot to the non-top stack slot,
`maxslot` value was unconditionally set to the destination slot, so some
top slots may be omitted in the snapshot entry. Since these slots are
omitted, they are not restored correctly, when restoring from a snapshot
for this side exit.

This patch adds the check for the aforementioned case, to avoid maxslot
shrinking.

Sergey Kaplun:
* added the description and the test for the problem

Part of tarantool/tarantool#8825

Reviewed-by: Maxim Kokryashkin <m.kokryashkin@tarantool.org>
Reviewed-by: Sergey Bronnikov <sergeyb@tarantool.org>
Signed-off-by: Igor Munkin <imun@tarantool.org>
(cherry picked from commit 9d1ae6e4f)

5289 of 5947 branches covered (0.0%)

Branch coverage included in aggregate %.

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

20311 of 23195 relevant lines covered (87.57%)

677366.64 hits per line