tarantool / luajit / 13256673271 / 1

93%

tarantool/master: 93%

push

github

Fix recording of BC_VARG.

Reported by Bachir Bendrissou.

(cherry picked from commit 62e362afb)

When the trace is started after the stitching, it may not have some
slots from the first one. That slot may be the first slot given to the
`select()` function (if it is determined by the call that caused the
stitch). Before the patch, there is no loading of this slot in the
`rec_varg()` for the trace, and the 0 value from slots is taken instead.
Hence, the following recording of `BC_VARG` is incorrect.

This patch fixes this by using the `getslot()` instead of taking the
value directly.

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

Part of tarantool/tarantool#11055

Reviewed-by: Sergey Bronnikov <sergeyb@tarantool.org>
Signed-off-by: Sergey Kaplun <skaplun@tarantool.org>
(cherry picked from commit 2ce227aa9)

5697 of 6036 branches covered (94.38%)

Branch coverage included in aggregate %.

21710 of 23442 relevant lines covered (92.61%)

2960302.8 hits per line