commit d46e46a8b883242117c66a872eaa2a601d801653
author Radek Krejci <rkrejci@cesnet.cz> Tue Sep 15 14:22:42 2020 +0200
committer Radek Krejci <rkrejci@cesnet.cz> Tue Sep 15 14:22:42 2020 +0200
tree ea7326a58561e153a9f7857d128117ff0c6e644a
parent 87e25251d3617d902ba0ba5f13c06ab4bf6d5b0f

data parsers BUGFIX memory leaks

src/parser_lyb.c

diff --git a/src/parser_lyb.c b/src/parser_lyb.c
index fdc9257..9395a13 100644
--- a/src/parser_lyb.c
+++ b/src/parser_lyb.c
@@ -503,7 +503,7 @@
 
         /* value */
         ret = lyb_read_string(&value, 0, lybctx);
-        LY_CHECK_GOTO(ret, cleanup);
+        LY_CHECK_ERR_GOTO(ret, ly_free_val_prefs(lybctx->ctx, val_prefs), cleanup);
         dynamic = 1;
 
         /* attr2 is always changed to the created attribute */
@@ -517,7 +517,6 @@
         module_name = NULL;
         free(name);
         name = NULL;
-        val_prefs = NULL;
         assert(!dynamic);
         value = NULL;
 
@@ -536,7 +535,6 @@
     if (dynamic) {
         free(value);
     }
-    ly_free_val_prefs(lybctx->ctx, val_prefs);
     if (ret) {
         lyd_free_attr_siblings(lybctx->ctx, *attr);
         *attr = NULL;
@@ -745,7 +743,7 @@
 
         /* parse value */
         ret = lyb_read_string(&value, 0, lybctx->lybctx);
-        LY_CHECK_GOTO(ret, cleanup);
+        LY_CHECK_ERR_GOTO(ret, ly_free_val_prefs(ctx, val_prefs), cleanup);
         dynamic = 1;
 
         /* create node */
@@ -879,7 +877,6 @@
     if (dynamic) {
         free(value);
     }
-    ly_free_val_prefs(ctx, val_prefs);
 
     lyd_free_meta_siblings(meta);
     lyd_free_attr_siblings(ctx, attr);