tarantool / luajit / 23194735910

70%

tarantool/master: 93%

Build Type

push

github

Committed by Buristan

Commit Message

Handle OOM error on stack resize in coroutine.resume and lua_checkstack.

Thanks to Peter Cawley.

(cherry picked from commit a5d2f70c7)

If the OOM or stack overflow error is obtained during
`lj_state_growstack()` in the `coroutine.resume()` it may raise an error
leading to the incorrect error message and unexpected behaviour for
this corner case. Also, `lua_checkstack()` may raise an OOM error
leading to panic on an unprotected lua_State, for example.

This patch handles these cases by protecting `lj_state_growstack()`.

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 357a260d8)

Run Details

5698 of 6034 branches covered (94.43%)

Branch coverage included in aggregate %.

14 of 14 new or added lines in 3 files covered. (100.0%)

10 existing lines in 2 files now uncovered.

21724 of 23448 relevant lines covered (92.65%)

1832241.86 hits per line