tree data UPDATE forbid freeing keys of lists

Refs #2091
CESNET · Sep 4, 2023 · 0dc4a26 · 0dc4a26
1 parent b5992f5
commit 0dc4a26
Showing 1 changed file with 10 additions and 0 deletions.
diff --git a/src/tree_data_free.c b/src/tree_data_free.c
@@ -202,6 +202,11 @@ lyd_free_tree(struct lyd_node *node)
         return;
     }
 
+    if (lysc_is_key(node->schema) && node->parent) {
+        LOGERR(LYD_CTX(node), LY_EINVAL, "Cannot free a list key \"%s\", free the list instance instead.", LYD_NAME(node));
+        return;
+    }
+
     lyd_free_subtree(node, 1);
 }
 
@@ -223,6 +228,11 @@ lyd_free_(struct lyd_node *node, ly_bool top)
     }
 
     LY_LIST_FOR_SAFE(node, next, iter) {
+        if (lysc_is_key(iter->schema) && iter->parent) {
+            LOGERR(LYD_CTX(node), LY_EINVAL, "Cannot free a list key \"%s\", free the list instance instead.", LYD_NAME(iter));
+            return;
+        }
+
         /* in case of the top-level nodes (node->parent is NULL), no unlinking needed */
         lyd_free_subtree(iter, iter->parent ? 1 : 0);
     }