tarantool / luajit / 8639131575

93%

tarantool/master: 93%

Build Type

push

github

Committed by mkokryashkin

Commit Message

Fix command-line argv handling.

(cherry-picked from commit 9ebebc9b5)

Before the patch, there was a situation where `luaL_newstate`
could fail in main and the `argv[0]` could be used as a progname
in `l_message`. However, `argv[0]` is not guaranteed to be
non-NULL, so segmentation fault could occur. This patch fixes the
issue by using the predefined name in that case. Moreover, it
refactors the `l_message`, so now there is no need to pass
`pname` everywhere.

The patch is tested with the help of the mocking of
`luaL_newstate` by providing an error-injected implementation
of it and preloading it. For preload to work, the LuaJIT must
be built with dynamic build mode enabled. Corresponding flavor
is added to the CI.

The tarantool-c-tests target cannot be linked with the LuaJIT
library when it is built as shared. The test suite is disabled
for the dynamic build mode.

Part of tarantool/tarantool#9924

Run Details

5660 of 6018 branches covered (94.05%)

Branch coverage included in aggregate %.

7 of 10 new or added lines in 1 file covered. (70.0%)

8 existing lines in 4 files now uncovered.

21600 of 23418 relevant lines covered (92.24%)

2946893.69 hits per line