]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
MINOR: hlua: fix return type for hlua_checkfunction() and hlua_checktable()
authorAurelien DARRAGON <adarragon@haproxy.com>
Mon, 20 Mar 2023 17:36:08 +0000 (18:36 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Wed, 5 Apr 2023 06:58:17 +0000 (08:58 +0200)
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.

src/hlua.c

index f567c8d5b4fc28eaf351fb54361b792a957b3c31..355060886c8db8f303ae0bf066c385d8cb6ab765 100644 (file)
@@ -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));