tarantool / luajit / 22936755036

93%

Build Type

push

github

Committed by Buristan

Commit Message

Prevent snapshot purge while recording a function header.

Thanks to Sergey Kaplun.

(cherry picked from commit d459c6ce5)

This patch is a follow-up to the commit
7505e78bd "Handle on-trace OOM errors
from helper functions.". Since this commit, the `pcall()` (and
`xpcall()`) emits an additional snapshot, but the arguments to the
pcall-ed function are momentarily purged from the snapshot since they
are not used. Hence, in several cases `debug.getlocal()` can't find the
corresponding slot for the previous frame.

This patch prevents purging right after `pcall()`, `xpcall()` recording.

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

Part of tarantool/tarantool#12134

Reviewed-by: Sergey Bronnikov <sergeyb@tarantool.org>
Signed-off-by: Sergey Kaplun <skaplun@tarantool.org>

Run Details

5708 of 6047 branches covered (94.39%)

Branch coverage included in aggregate %.

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

13 existing lines in 5 files now uncovered.

21792 of 23517 relevant lines covered (92.66%)

3884961.64 hits per line