MINOR: hlua: some luaL_checktype() calls were not guarded with MAY_LJMP
In hlua code, we mark every function that may longjump using
MAY_LJMP macro so it's easier to identify them by reading the code.
However, some luaL_checktypes() were performed without the MAY_LJMP.
According to lua doc:
Functions called luaL_check* always raise an error if
the check is not satisfied.
-> Adding the missing MAY_LJMP for those luaLchecktypes() calls.
No backport needed.
diff --git a/src/hlua.c b/src/hlua.c
index 1595368..02cba1e 100644
--- a/src/hlua.c
+++ b/src/hlua.c
@@ -8232,7 +8232,7 @@
const char *reason = MAY_LJMP(luaL_optlstring(L, 3, NULL, NULL));
/* First argument (self) must be a table */
- luaL_checktype(L, 1, LUA_TTABLE);
+ MAY_LJMP(luaL_checktype(L, 1, LUA_TTABLE));
if (status < 100 || status > 599) {
lua_pushboolean(L, 0);
@@ -8262,7 +8262,7 @@
int ret;
/* First argument (self) must be a table */
- luaL_checktype(L, 1, LUA_TTABLE);
+ MAY_LJMP(luaL_checktype(L, 1, LUA_TTABLE));
/* Push in the stack the "headers" entry. */
ret = lua_getfield(L, 1, "headers");
@@ -8308,7 +8308,7 @@
int ret;
/* First argument (self) must be a table */
- luaL_checktype(L, 1, LUA_TTABLE);
+ MAY_LJMP(luaL_checktype(L, 1, LUA_TTABLE));
/* Push in the stack the "headers" entry. */
ret = lua_getfield(L, 1, "headers");
@@ -8331,7 +8331,7 @@
const char *payload = MAY_LJMP(luaL_checkstring(L, 2));
/* First argument (self) must be a table */
- luaL_checktype(L, 1, LUA_TTABLE);
+ MAY_LJMP(luaL_checktype(L, 1, LUA_TTABLE));
lua_pushstring(L, payload);
lua_setfield(L, 1, "body");