file: raise parse error on conflicting section types in strict mode
authorJo-Philipp Wich <jow@openwrt.org>
Thu, 5 Nov 2015 18:15:40 +0000 (19:15 +0100)
committerJo-Philipp Wich <jow@openwrt.org>
Mon, 25 Jan 2016 10:42:55 +0000 (11:42 +0100)
If strict mode is enabled and we're parsing a config file with multiple
sections of the same name but different types, then raise a parse error
to notify the user that the subsequent section declaration would shadow
all prior ones of the same name.

The error would be triggered by a config like that:

  config typeA example
    option test 1

  config typeB example
    option test 2

In such a case, libuci will raise this error:

  uci: Parse error (section of different type overwrites prior section with same name) at line 4, byte 23

Signed-off-by: Jo-Philipp Wich <jow@openwrt.org>
file.c

diff --git a/file.c b/file.c
index 8e6052c70c168eab23be093bde4174c91d1c3076..35755d83ad8c04dc016d1045e9c3ae4f44b0354c 100644 (file)
--- a/file.c
+++ b/file.c
@@ -437,8 +437,13 @@ static void uci_parse_config(struct uci_context *ctx)
        } else {
                uci_fill_ptr(ctx, &ptr, &pctx->package->e);
                e = uci_lookup_list(&pctx->package->sections, name);
-               if (e)
+               if (e) {
                        ptr.s = uci_to_section(e);
+
+                       if ((ctx->flags & UCI_FLAG_STRICT) && strcmp(ptr.s->type, type))
+                               uci_parse_error(ctx, "section of different type overwrites prior section with same name");
+               }
+
                ptr.section = name;
                ptr.value = type;