[Tarantool-patches] [PATCH] lua: assert on lua_gettop() negative return value
Ilya Kosarev
i.kosarev at tarantool.org
Fri Jul 17 20:25:29 MSK 2020
In case lua_gettop() called from encode_lua_call() returns negative
value, we will segfault in iproto_reply_error() with empty diag, as far
as it is unexpected error path not covered with diagnostics. Thus
corresponding sane check with assert is introduced.
Closes #4649
---
Branch: https://github.com/tarantool/tarantool/tree/i.kosarev/gh-4649-sane-check-on-lua_gettop
Issue: https://github.com/tarantool/tarantool/issues/4649
@ChangeLog:
* Raise assert on lua_gettop() negative return value (gh-4649).
src/box/lua/call.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/box/lua/call.c b/src/box/lua/call.c
index ca871e077..75634de5b 100644
--- a/src/box/lua/call.c
+++ b/src/box/lua/call.c
@@ -361,6 +361,7 @@ encode_lua_call(lua_State *L)
struct luaL_serializer *cfg = luaL_msgpack_default;
int size = lua_gettop(port->L);
+ assert(size >= 0);
for (int i = 1; i <= size; ++i)
luamp_encode(port->L, cfg, &stream, i);
port->size = size;
--
2.17.1
More information about the Tarantool-patches
mailing list