From: Aurelien DARRAGON Date: Mon, 20 Mar 2023 17:36:08 +0000 (+0100) Subject: MINOR: hlua: fix return type for hlua_checkfunction() and hlua_checktable() X-Git-Tag: v2.8-dev7~50 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9ee0d04770a96dcad77b457628a2130862976908;p=thirdparty%2Fhaproxy.git MINOR: hlua: fix return type for hlua_checkfunction() and hlua_checktable() hlua_checktable() and hlua_checkfunction() both return the raw value of luaL_ref() function call. As luaL_ref() returns a signed int, both functions should return a signed int as well to prevent any misuse of the returned reference value. --- diff --git a/src/hlua.c b/src/hlua.c index f567c8d5b4..355060886c 100644 --- a/src/hlua.c +++ b/src/hlua.c @@ -475,7 +475,7 @@ static inline int reg_flt_to_stack_id(struct hlua_reg_filter *reg_flt) * error if the argument is not a "function". * When no longer used, the ref must be released with hlua_unref() */ -__LJMP unsigned int hlua_checkfunction(lua_State *L, int argno) +__LJMP int hlua_checkfunction(lua_State *L, int argno) { if (!lua_isfunction(L, argno)) { const char *msg = lua_pushfstring(L, "function expected, got %s", luaL_typename(L, argno)); @@ -490,7 +490,7 @@ __LJMP unsigned int hlua_checkfunction(lua_State *L, int argno) * error if the argument is not a "table". * When no longer used, the ref must be released with hlua_unref() */ -__LJMP unsigned int hlua_checktable(lua_State *L, int argno) +__LJMP int hlua_checktable(lua_State *L, int argno) { if (!lua_istable(L, argno)) { const char *msg = lua_pushfstring(L, "table expected, got %s", luaL_typename(L, argno));