tarantool / luajit / 22917122874

93%

tarantool/master: 93%

Build Type

push

github

Committed by Buristan

Commit Message

FFI: Avoid dangling cts->L.

Reported by ZumiKua.

(cherry picked from commit c94312d34)

When executing cdata finalizer on some lua_State, this state is set to
`cts->L`. If the state will be GC-ed and freed later, this reference
becomes dangling, so any FFI callback will use this invalid reference.

This patch fixes it by setting `cts->L` to the `mainthread` on the
destruction of the referenced one.

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 1e5887d88)

Run Details

5708 of 6047 branches covered (94.39%)

Branch coverage included in aggregate %.

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

3 existing lines in 2 files now uncovered.

21805 of 23517 relevant lines covered (92.72%)

3886413.43 hits per line