summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLukas Fleischer <calcurse@cryptocrack.de>2011-04-18 01:18:03 +0200
committerLukas Fleischer <calcurse@cryptocrack.de>2011-04-22 02:47:36 +0200
commit112fbe00a4e7fe23b7314485304df7a9b029fda6 (patch)
tree4a50c891a39a74c453940a6ae5dbef90a1558603
parentfb5f9d0155d63451bd16d7d50204da90ff604949 (diff)
downloadcalcurse-112fbe00a4e7fe23b7314485304df7a9b029fda6.tar.gz
calcurse-112fbe00a4e7fe23b7314485304df7a9b029fda6.zip
llist.c: Nullify pointers after free()'ing.
* Set all data members to "NULL" in llist_free_inner() after freeing them. Altough we normally shouldn't continue working with a list that already went through llist_free_inner(), this will protect against dangling pointer bugs in case anyone will ever come up with the idea of doing so. * Set list head to "NULL" in llist_free(), basically to put the list into an initialized state. Signed-off-by: Lukas Fleischer <calcurse@cryptocrack.de>
-rw-r--r--src/llist.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/src/llist.c b/src/llist.c
index 021de82..e24f9fc 100644
--- a/src/llist.c
+++ b/src/llist.c
@@ -52,6 +52,8 @@ llist_free (llist_t *l)
t = i->next;
mem_free (i);
}
+
+ l->head = NULL;
}
void
@@ -62,7 +64,10 @@ llist_free_inner (llist_t *l, llist_fn_free_t fn_free)
for (i = l->head; i; i = i->next)
{
if (i->data)
- fn_free(i->data);
+ {
+ fn_free(i->data);
+ i->data = NULL;
+ }
}
}