summaryrefslogtreecommitdiffstats
path: root/libcnary/node_iterator.c
diff options
context:
space:
mode:
authorGravatar Nikias Bassen2016-11-10 02:51:54 +0100
committerGravatar Nikias Bassen2016-11-10 02:51:54 +0100
commit52f3a565deca246e0a331f4a014178c3f6927f31 (patch)
tree058463dd9cf789546b90fe4078545bec22231577 /libcnary/node_iterator.c
parentad1a95e96218e411b4dbde38c002fc5f4b530a65 (diff)
downloadlibplist-52f3a565deca246e0a331f4a014178c3f6927f31.tar.gz
libplist-52f3a565deca246e0a331f4a014178c3f6927f31.tar.bz2
libcnary: Fix possible NULL pointer dereference in node_iterator_create()
When NULL is passed to node_iterator_create() the code tries to access the begin element of the node list and causes a NULL pointer dereference. The value of list is checked a few lines down and iterator->value is then properly assigned in node_iterator_bind().
Diffstat (limited to 'libcnary/node_iterator.c')
-rw-r--r--libcnary/node_iterator.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/libcnary/node_iterator.c b/libcnary/node_iterator.c
index dedf3b4..e629b73 100644
--- a/libcnary/node_iterator.c
+++ b/libcnary/node_iterator.c
@@ -47,7 +47,7 @@ node_iterator_t* node_iterator_create(node_list_t* list) {
iterator->end = NULL;
iterator->begin = NULL;
- iterator->value = list->begin;
+ iterator->value = NULL;
iterator->list = NULL;
iterator->next = node_iterator_next;