commit 66c1631e1441586c51f8c6feb83cf38f0495ffd3
author Michal Vasko <mvasko@cesnet.cz> Thu Dec 01 10:37:43 2022 +0100
committer Michal Vasko <mvasko@cesnet.cz> Thu Dec 01 15:56:59 2022 +0100
tree f4d707c1721d16cd0ab5113575200868b5aa9d5d
parent bc9f76ff8ca4710b631fa901e62aed42ce73e4eb

parser json BUGFIX log node memory leak

src/parser_json.c

diff --git a/src/parser_json.c b/src/parser_json.c
index 82df659..3b90783 100644
--- a/src/parser_json.c
+++ b/src/parser_json.c
@@ -743,6 +743,7 @@
     assert(snode || node);
 
     nodetype = snode ? snode->nodetype : LYS_CONTAINER;
+    LOG_LOCSET(snode, NULL, NULL, NULL);
 
     /* move to the second item in the name/X pair */
     ret = lyjson_ctx_next(lydctx->jsonctx, &status);
@@ -799,14 +800,13 @@
         LY_CHECK_GOTO(status != LYJSON_OBJECT, representation_error);
         break;
     default:
-        LOGINT_RET(ctx);
+        LOGINT(ctx);
+        ret = LY_EINT;
+        goto cleanup;
     }
 
     /* process all the members inside a single metadata object */
     assert(status == LYJSON_OBJECT);
-
-    LOG_LOCSET(snode, NULL, NULL, NULL);
-
     while (status != LYJSON_OBJECT_CLOSED) {
         LY_CHECK_GOTO(status != LYJSON_OBJECT, representation_error);
 
@@ -897,7 +897,6 @@
             "The attribute(s) of %s \"%s\" is expected to be represented as JSON %s, but input data contains @%s/%s.",
             lys_nodetype2str(nodetype), node ? LYD_NAME(node) : LYD_NAME(prev), expected, lyjson_token2str(status),
             in_parent ? "" : "name");
-
     ret = LY_EVALID;
 
 cleanup: