[Tarantool-patches] [PATCH v2] lua: assert on lua_gettop() negative return value

Ilya Kosarev i.kosarev at tarantool.org
Tue Jul 21 14:05:14 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

Changes in v2:
- added reasoning in comment

 src/box/lua/call.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/box/lua/call.c b/src/box/lua/call.c
index ca871e077..516276a9f 100644
--- a/src/box/lua/call.c
+++ b/src/box/lua/call.c
@@ -361,6 +361,12 @@ encode_lua_call(lua_State *L)
 
 	struct luaL_serializer *cfg = luaL_msgpack_default;
 	int size = lua_gettop(port->L);
+	/*
+	 * lua_gettop() might return negative value in case the internal state
+	 * of the given Lua coroutine is seriously broken. In case of such
+	 * behavior execution has to be aborted immediately.
+	 */
+	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