commit 6d781b6b69fd1e26baac06ce5b6175dd84ac076a
author David Sedlák <xsedla1d@stud.fit.vutbr.cz> Fri Aug 02 15:22:52 2019 +0200
committer David Sedlák <xsedla1d@stud.fit.vutbr.cz> Fri Aug 02 15:22:52 2019 +0200
tree 30b9a969bfb9a37f62ec564d4281d7510150f83d
parent 8985a14587b22b7ab65a909699ab670eee02ccf3

yin parser CHANGE add detection of trailing characters after module/submodule

src/parser_yin.c

diff --git a/src/parser_yin.c b/src/parser_yin.c
index abd5261..7c278a0 100644
--- a/src/parser_yin.c
+++ b/src/parser_yin.c
@@ -3159,6 +3159,19 @@
     ret = yin_parse_submod(*yin_ctx, attrs, &data, mod_p);
     LY_CHECK_GOTO(ret, cleanup);
 
+    name = NULL;
+    if ((*yin_ctx)->xml_ctx.status == LYXML_ELEMENT) {
+        const char *temp_data = data;
+        ret = lyxml_get_element(&(*yin_ctx)->xml_ctx, &data, &prefix, &prefix_len, &name, &name_len);
+        data = temp_data;
+    }
+    if ((*yin_ctx)->xml_ctx.status != LYXML_END || name) {
+        LOGVAL_PARSER((struct lys_parser_ctx *)*yin_ctx, LYVE_SYNTAX, "Trailing garbage \"%.*s%s\" after submodule, expected end-of-input.",
+                    15, data, strlen(data) > 15 ? "..." : "");
+        ret = LY_EVALID;
+        goto cleanup;
+    }
+
     mod_p->parsing = 0;
     *submod = mod_p;
 
@@ -3214,6 +3227,17 @@
     ret = yin_parse_mod(*yin_ctx, attrs, &data, mod_p);
     LY_CHECK_GOTO(ret, cleanup);
 
+    if ((*yin_ctx)->xml_ctx.status == LYXML_ELEMENT) {
+        ret = lyxml_get_element(&(*yin_ctx)->xml_ctx, &data, &prefix, &prefix_len, &name, &name_len);
+    }
+    if ((*yin_ctx)->xml_ctx.status != LYXML_END || name) {
+        LOGVAL_PARSER((struct lys_parser_ctx *)*yin_ctx, LYVE_SYNTAX, "Trailing garbage \"%.*s%s\" after module, expected end-of-input.",
+                    15, data, strlen(data) > 15 ? "..." : "");
+
+        ret = LY_EVALID;
+        goto cleanup;
+    }
+
     mod_p->parsing = 0;
     mod->parsed = mod_p;