Add multiple free for each call to lookup_args()

[project/uci.git] / lua / uci.c

diff --git a/lua/uci.c b/lua/uci.c
index cc6f370..64e509e 100644 (file)
--- a/lua/uci.c
+++ b/lua/uci.c
@@ -364,6 +364,8 @@ uci_lua_get_any(lua_State *L, bool all)
                        err = UCI_ERR_INVAL;
                        goto error;
        }
+       if (s)
+               free(s);
        if (!err)
                return 1;
 
@@ -484,6 +486,8 @@ uci_lua_rename(lua_State *L)
                goto error;
 
 error:
+       if (s)
+               free(s);
        return uci_push_status(L, ctx, false);
 }
 
@@ -533,6 +537,8 @@ uci_lua_reorder(lua_State *L)
                goto error;
 
 error:
+       if (s)
+               free(s);
        return uci_push_status(L, ctx, false);
 }
 
@@ -616,6 +622,8 @@ uci_lua_set(lua_State *L)
 
 
 error:
+       if (s)
+               free(s);
        return uci_push_status(L, ctx, false);
 }
 
@@ -665,6 +673,8 @@ uci_lua_package_cmd(lua_State *L, enum pkg_cmd cmd)
        }
 
 err:
+       if (s)
+               free(s);
        return uci_push_status(L, ctx, false);
 }
 
@@ -711,24 +721,38 @@ uci_lua_add_change(lua_State *L, struct uci_element *e)
        if (name) {
                lua_getfield(L, -1, name);
 
-               /* there seems to be no value yet */
-               if (lua_isnil(L, -1)) {
-                       /* this delta is a list add operation, initialize table */
-                       if (h->cmd == UCI_CMD_LIST_ADD) {
+               /* this delta is a list add operation */
+               if (h->cmd == UCI_CMD_LIST_ADD) {
+                       /* there seems to be no table yet */
+                       if (!lua_istable(L, -1)) {
                                lua_newtable(L);
-                               lua_pushstring(L, value);
-                               lua_rawseti(L, -2, 1);
+
+                               /* if there is a value on the stack already, add */
+                               if (!lua_isnil(L, -2)) {
+                                       lua_pushvalue(L, -2);
+                                       lua_rawseti(L, -2, 1);
+                                       lua_pushstring(L, value);
+                                       lua_rawseti(L, -2, 2);
+
+                               /* this is the first table item */
+                               } else {
+                                       lua_pushstring(L, value);
+                                       lua_rawseti(L, -2, 1);
+                               }
+
                                lua_setfield(L, -3, name);
+
+                       /* a table is on the top of the stack and this is a subsequent,
+                        * list_add, append this value to table */
                        } else {
                                lua_pushstring(L, value);
-                               lua_setfield(L, -3, name);
+                               lua_rawseti(L, -2, lua_objlen(L, -2) + 1);
                        }
 
-               /* a table is on the top of the stack so this is a subsequent,
-                * list_add, append this value to table */
-               } else if (lua_istable(L, -1)) {
+               /* non-list change, simply set/replace field */
+               } else {
                        lua_pushstring(L, value);
-                       lua_rawseti(L, -2, lua_objlen(L, -2) + 1);
+                       lua_setfield(L, -3, name);
                }
 
                lua_pop(L, 1);
@@ -849,20 +873,6 @@ uci_lua_add_delta(lua_State *L)
 }
 
 static int
-uci_lua_load_plugins(lua_State *L)
-{
-       struct uci_context *ctx;
-       int offset = 0;
-       const char *str = NULL;
-
-       ctx = find_context(L, &offset);
-       if (lua_isstring(L, -1))
-               str = lua_tostring(L, -1);
-       uci_load_plugins(ctx, str);
-       return uci_push_status(L, ctx, false);
-}
-
-static int
 uci_lua_set_savedir(lua_State *L)
 {
        struct uci_context *ctx;
@@ -931,7 +941,6 @@ static const luaL_Reg uci[] = {
        { "foreach", uci_lua_foreach },
        { "add_history", uci_lua_add_delta },
        { "add_delta", uci_lua_add_delta },
-       { "load_plugins", uci_lua_load_plugins },
        { "get_confdir", uci_lua_get_confdir },
        { "set_confdir", uci_lua_set_confdir },
        { "get_savedir", uci_lua_get_savedir },