commit f7e16e25f08eb70042f896daf90a3ca0b4a0ca1b
author Michal Vasko <mvasko@cesnet.cz> Wed Oct 21 09:24:39 2020 +0200
committer Michal Vasko <mvasko@cesnet.cz> Wed Oct 21 12:21:31 2020 +0200
tree f235e2b0f73f9425a0984e0f9c0d03a4f8b76e13
parent 40308e729099e8c9a02e0619fcf5d29c09345a8b

path REFACTOR nodes are not needed when freeing predicate values

Also solves problem of freeing order between the
path and the referenced nodes.

src/path.c

diff --git a/src/path.c b/src/path.c
index 0552e66..0ec1d02 100644
--- a/src/path.c
+++ b/src/path.c
@@ -525,7 +525,7 @@
             /* names (keys) are unique - it was checked when parsing */
             LOGVAL_P(ctx, cur_node, LYVE_XPATH, "Predicate missing for a key of %s \"%s\" in path.",
                     lys_nodetype2str(ctx_node->nodetype), ctx_node->name);
-            ly_path_predicates_free(ctx, LY_PATH_PREDTYPE_LIST, NULL, *predicates);
+            ly_path_predicates_free(ctx, LY_PATH_PREDTYPE_LIST, *predicates);
             *predicates = NULL;
             return LY_EVALID;
         }
@@ -985,8 +985,7 @@
 }
 
 void
-ly_path_predicates_free(const struct ly_ctx *ctx, enum ly_path_pred_type pred_type, const struct lysc_node *llist,
-        struct ly_path_predicate *predicates)
+ly_path_predicates_free(const struct ly_ctx *ctx, enum ly_path_pred_type pred_type, struct ly_path_predicate *predicates)
 {
     LY_ARRAY_COUNT_TYPE u;
 
@@ -1001,10 +1000,8 @@
             /* nothing to free */
             break;
         case LY_PATH_PREDTYPE_LIST:
-            ((struct lysc_node_leaf *)predicates[u].key)->type->plugin->free(ctx, &predicates[u].value);
-            break;
         case LY_PATH_PREDTYPE_LEAFLIST:
-            ((struct lysc_node_leaflist *)llist)->type->plugin->free(ctx, &predicates[u].value);
+            predicates[u].value.realtype->plugin->free(ctx, &predicates[u].value);
             break;
         }
     }
@@ -1017,7 +1014,7 @@
     LY_ARRAY_COUNT_TYPE u;
 
     LY_ARRAY_FOR(path, u) {
-        ly_path_predicates_free(ctx, path[u].pred_type, path[u].node, path[u].predicates);
+        ly_path_predicates_free(ctx, path[u].pred_type, path[u].predicates);
     }
     LY_ARRAY_FREE(path);
 }