style: fix whitespace errors

Signed-off-by: Luka Perkov <luka@openwrt.org>

style: free(NULL) is perfectly valid so we can drop some checks

Signed-off-by: Luka Perkov <luka@openwrt.org>

file: make uci commits atomic

Avoids issues with UBIFS file system.

http://www.linux-mtd.infradead.org/faq/ubifs.html#L_atomic_change

Signed-off-by: Luka Perkov <luka@openwrt.org>
Reported-by: Tim Harvey <tharvey@gateworks.com>

uci_delete: check ptr->o and its type before checking ptr->value

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

uci: fix excessive reorder delta entry duplication

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

blob: add uci<->blob conversion library code

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

util: fix compile error

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

lua: emit a proper deletion when setting an existing list to a new list with only a single item

uci: Fix uci_delete() when ptr->value is empty string

After fix for deleting list elements by index this function was
broken and didn't allow to invoke from command line:
uci set some.fancy.list=''
i.e. list wasn't cleared

Signed-off-by: Krzysztof Kuźnik <k.kuznik@avsystem.com>

uci: copy permisions of /etc/config/ files for temp files

Hi

Using uci as a non root user i can get network configuration but not network state (for exemple).
The idea of this patch is to copy permission from config file, or if it doesn't exist use UCI_FILEMODE / UCI_DIRMODE

Tested on tplink wr1043nd. (you have to remove some mkdir from /etc/init.d/boot to fully see this patch work)
If you apply this patch (or a modified version), please also backport to AA

Signed-off-by: Etienne CHAMPETIER <etienne.champetier@free.fr>
openwrt-devel mailing list
openwrt-devel@lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel

Change delta encoding of list deletions

When encoding list_del commands in delta files, '_' is used as prefix but this
breaks any command operating on files starting with an underscore since the
command prefix becomes indistinguishable from the filename.

The regression was introduced with ec8cbbbe0f20e67cbd3bd471b9ac55ceacaef7b8.

Avoid this problem by using '~' as command prefix for list deletions.

Add multiple free for each call to lookup_args()

This memory leak can be reproduced using a tight loop to uci:get() a config.

lua -e 'require("posix"); require("uci");
function getconf()
   local tmp, uc = uci.cursor();
   tmp = uc:get("system", "ntp", "enable_server");
end;

while 1
do
   for i=0,1000 do getconf() end;
   print("gc");
   print(collectgarbage("count"));
   collectgarbage();
   print(collectgarbage("count"));
   print("sleep 1");
   posix.sleep(1);
end'

uci: libuci leaking memory on non-existent config file

In file file.c function uci_file_load() allocates memory for filename.
When uci_open_stream() failed with exception this memory was never freed.
This patch makes call to uci_open_stream() wrapped in exception handling
inside uci_file_load() where filename is freed.

Signed-off-by: Krzysztof Kuznik <k.kuznik@avsystem.com>

ignore trailing carriage returns in file parser

add main directory to include path

build: remove install prefix override

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

get rid of -rdynamic on link

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

remove unused hook support

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

remove the libdl dependency

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

remove plugin support, it has been unused for a long time and should not be used for any new code

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

remove static libraries/binaries

Signed-off-by: Felix Fietkau <nbd@openwrt.org>

extend delete command so it can delete list elemets using index

add new command del_list

lua: include /opt/local/lib and /opt/local/include in the search path on mac os x

cli: return non-zero status on errors

Do quote section names to keep the same style as option values

Section names, option names, list names can only be identifiers, so they don't need to be quoted.
Strip quotes except from values.
Signed-off-by: Philip Prindeville <philipp@redfish-solutions.com>

lua: properly handle corner cases in changes
The previous rework of the change serialization caused some deltas to incorrectly
get promoted to list appends. Rework the code to explicitely check for an add list
change type, in all other cases simply overwrite existing fields with the current
string value instead of coercing existing values into a table and append to that.
This hopefully sorts out the remaining issues.

lua: expose completely new lists in changes() as well
Since the first delta of a new list is of type UCI_CMD_CHANGE and
not UCI_CMD_LIST_ADD, the current code does not start a new table
and subsequent items are silently disacarded. Expose all items of
new list by coercing the existing string element into a table when
encountering subsequent items.

Merge branch 'master' of git://nbd.name/uci

lua: simplify add_list changes handling, always create a table for LIST_ADD commands

lua: expose add_list change items as table values

lua: simplify add_list changes handling, always create a table for LIST_ADD commands

lua: expose add_list change items as table values

fix segfault on import of anonymous sections

lua: always return after calling luaL_error (patch by Stanislav Fomichev)

cli: check against multiple -f invocations to avoid leaks (patch by Stanislav Fomichev)

ucimap: initialize an uninitialized pointer (patch by Stanislav Fomichev)

ucimap: fix a memory leak (patch by Stanislav Fomichev)

add a few missing null pointer checks, and fix check vs dereference order in some cases (patch by Stanislav Fomichev)

check result of memory allocations (patch by Stanislav Fomichev)

ucimap: fix use-after-free on cleanup (patch by Stanislav Fomichev)

free memory for uci hooks

lua: allow overriding of LUA_CFLAGS

lua: use pkg-config to find the correct lua cflags

fix lgpl license headers

fix one more unused-but-set variable

fix more unused-but-set errors

remove unused but set variable to fix a compiler warning

do not attempt to build lua if disabled

constify arguments to parser function

add parser helpers similar to the blobmsg parser in libubox

cli: do not return 0 if the command was invalid

also link the shared libuci to libdl

fix lua plugin linker error

fix building without plugin support

fix include path

use libdl

allow the user to override the lua path

move ucimap to a separate library

replace the existing build system with cmake

improve support for C++ stuff

fix uci_list_set_pos to reorder sections properly, even when moving a section to the last position

make uci_file_commit() static

lua: use uci_foreach_element_safe in uci_lua_foreach() - patch by jow

add a helper function for looking up an option string

fix return value in inline lookup functions

ucimap: ucimap_parse: consider smap_offset when allocating the section data

Currently when there is no custom section data allocator the ucimap_section_data
struct has to be at the beginning at the containing structure.
This patch gets rid of that restriction by taking smap_offset into account.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>

add convenience functions for looking up packages/sections/options

remove the unused "complete" parameter for uci_fill_ptr()

add support for partial lookups - skip lookup for filled-in sections or packages

provide backwards compatibility in the lua module after the history -> delta change

make uci_lookup_ptr() use the extended flag properly again

split off and compile delta.c separately

uci_history -> uci_delta

split off and compile util.c separately

export uci_parse_ptr

split off and compile file.c separately

simplify dependency handling

simplify code (thx, henning)

Fix memory leak in uci_list_config_files

When glob returns non-zero, the path buffer was not unallocated.

Allow setting of libdirname.

Modifies the Makefiles so that the installer can specify the name
of the directory that libraries should be installed into by setting
LIBDIRNAME.  For instance 'lib' or 'lib64'.

fix compile on snow leopard

lua: use extended lookups by default

lua: add uci.foreach() error handling patch by xMff

fix uci_lookup_ptr comments

add test/save to .gitignore

ucimap: simplify the parse/format callbacks. the data argument now always points to the field in the data structure

ucimap: fix list size initialization, validate list size on fixups as well

ucimap: fix list initialization

add some comments to the header file

replace the use of list.h in the ucimap core, relicense ucimap to LGPL v2.1, update copyright headers

remove ucimap type->name conversion unless DEBUG is enabled

fix a typo

ucimap: add support for saving lists to uci

test: do not print expected error messages

ucimap: add helper function for resizing lists and freeing items (both using ucimap internal allocation list), add test coverage for config change operations

add sanity checks for optmap section type vs sectionmap type

ucimap: add custom free() callbacks for options, only used on custom datatypes

add ucimap_free_section to the api