tarantool / luajit / 22904063306

93%

tarantool/master: 93%

Build Type

push

github

Committed by Buristan

Commit Message

Fix edge cases when recording string.byte/sub.

Thanks to Sergey Kaplun.

(cherry picked from commit 89f268b3f)

It is possible that the `len` (`end - start`) will underflow and become
positive in the `recff_string_range()` when the `end` is negative. For
`string.sub()` this is not crucial, since the trace will be valid
anyway. But for `string.byte()` it may lead to the assertion failure in
the `rec_check_slots()`.

This patch fixes those underflows by the correct comparison.

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>
(cherry picked from commit b45b65c33)

Run Details

5707 of 6046 branches covered (94.39%)

Branch coverage included in aggregate %.

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

25 existing lines in 2 files now uncovered.

21780 of 23516 relevant lines covered (92.62%)

3885984.1 hits per line