commit d45396aad911d496494a587bd2d3ef20c2e8a8d0
author Nikias Bassen <nikias@gmx.li> Wed Aug 30 01:03:44 2023 +0200
committer Nikias Bassen <nikias@gmx.li> Wed Aug 30 01:03:44 2023 +0200
tree 0f73b93cdb03cef6353b2d9c40034fdf1d8b999e
parent 2d8d7ef272db06783989f77ba1ed80aa0f4d2dfd

Prevent adding NULL items to array/dictionary nodes

Thanks to @tihmstar for pointing this out!

src/plist.c

diff --git a/src/plist.c b/src/plist.c
index 19279ed..2f4990c 100644
--- a/src/plist.c
+++ b/src/plist.c
@@ -654,6 +654,9 @@
 
 void plist_array_set_item(plist_t node, plist_t item, uint32_t n)
 {
+    if (!item) {
+        return;
+    }
     if (node && PLIST_ARRAY == plist_get_node_type(node) && n < INT_MAX)
     {
         plist_t old_item = plist_array_get_item(node, n);
@@ -675,6 +678,9 @@
 
 void plist_array_append_item(plist_t node, plist_t item)
 {
+    if (!item) {
+        return;
+    }
     if (node && PLIST_ARRAY == plist_get_node_type(node))
     {
         node_attach((node_t)node, (node_t)item);
@@ -684,6 +690,9 @@
 
 void plist_array_insert_item(plist_t node, plist_t item, uint32_t n)
 {
+    if (!item) {
+        return;
+    }
     if (node && PLIST_ARRAY == plist_get_node_type(node) && n < INT_MAX)
     {
         node_insert((node_t)node, n, (node_t)item);
@@ -852,6 +861,9 @@
 
 void plist_dict_set_item(plist_t node, const char* key, plist_t item)
 {
+    if (!item) {
+        return;
+    }
     if (node && PLIST_DICT == plist_get_node_type(node)) {
         plist_t old_item = plist_dict_get_item(node, key);
         plist_t key_node = NULL;