err = UCI_ERR_INVAL;
goto error;
}
+ if (s)
+ free(s);
if (!err)
return 1;
goto error;
error:
+ if (s)
+ free(s);
return uci_push_status(L, ctx, false);
}
goto error;
error:
+ if (s)
+ free(s);
return uci_push_status(L, ctx, false);
}
bool istable = false;
int err = UCI_ERR_MEM;
char *s = NULL;
+ const char *v;
int i, nargs, offset = 0;
ctx = find_context(L, &offset);
if (istable) {
if (lua_objlen(L, nargs) == 1) {
i = 1;
- if (ptr.o)
+ if (ptr.o) {
+ v = ptr.value;
+ ptr.value = NULL;
err = uci_delete(ctx, &ptr);
+ if (err)
+ goto error;
+ ptr.value = v;
+ }
} else {
i = 2;
err = uci_set(ctx, &ptr);
error:
+ if (s)
+ free(s);
return uci_push_status(L, ctx, false);
}
}
err:
+ if (s)
+ free(s);
return uci_push_status(L, ctx, false);
}
if (name) {
lua_getfield(L, -1, name);
- /* there seems to be no value yet so simply push it as string */
- if (lua_isnil(L, -1)) {
- lua_pushstring(L, value);
- lua_setfield(L, -3, name);
+ /* 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);
+
+ /* 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_rawseti(L, -2, lua_objlen(L, -2) + 1);
+ }
- /* there is already a value, so this change is a list_add,
- * coerce string into one-element array and append new value */
- } else if (lua_isstring(L, -1)) {
- lua_newtable(L);
- lua_pushvalue(L, -2);
- lua_rawseti(L, -2, 1);
+ /* non-list change, simply set/replace field */
+ } else {
lua_pushstring(L, value);
- lua_rawseti(L, -2, 2);
lua_setfield(L, -3, name);
-
- /* 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)) {
- lua_pushstring(L, value);
- lua_rawseti(L, -2, lua_objlen(L, -2) + 1);
}
lua_pop(L, 1);
}
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;
{ "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 },