tarantool / luajit / 6808010614

88%

tarantool/master: 93%

Build Type

push

github

Committed by Buristan

Commit Message

FFI: Fix pragma push stack limit check and throw on overflow.

Reported by Sergey Kaplun.

(cherry-picked from commit 433d7e8d8)

`cp->packstack` is the array of size `CPARSE_MAX_PACKSTACK` (7). Before
the patch, `cp->curpack` is checked to be less than
`CPARSE_MAX_PACKSTACK`, but then `cp->packstack` is accessed at
`cp->curpack + 1`, which is out of bounds, so `cp->curpack` value is
overwritten.

This patch fixes a condition and also adds the error throw when counter
is overflow (instead of rewriting a top `cp->packstack` value).

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

Resolves tarantool/tarantool#9339
Part of tarantool/tarantool#9145

Run Details

5359 of 5975 branches covered (0.0%)

Branch coverage included in aggregate %.

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

24 existing lines in 4 files now uncovered.

20535 of 23305 relevant lines covered (88.11%)

2743513.88 hits per line