fyb/calcurse-edge
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

llist.c: Nullify pointers after free()'ing.

Browse Source 
* 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>
This commit is contained in:
Lukas Fleischer 2011-04-18 01:18:03 +02:00
parent fb5f9d0155
commit 112fbe00a4
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/llist.c
View File

@ -52,6 +52,8 @@ llist_free (llist_t *l)
t = i->next; t = i->next;
mem_free (i); mem_free (i);
} }
l->head = NULL;
} }
void 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) for (i = l->head; i; i = i->next)
{ {
if (i->data) if (i->data)
fn_free(i->data); {
fn_free(i->data);
i->data = NULL;
}
} }
} }