diff options
| author | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-09-27 21:19:20 -0400 |
|---|---|---|
| committer | Donald Sharp <sharpd@cumulusnetworks.com> | 2017-10-05 10:53:17 -0400 |
| commit | acdf5e25101bafe334e6b500c3dd0a2babb3c1ec (patch) | |
| tree | d8b9483a3aeebc7e90ddbe5f765e5e63aecb8e1b /lib/linklist.c | |
| parent | affe9e99831408960b8b6f8477506ed2874a05dd (diff) | |
*: Convert list_free usage to list_delete
list_free is occassionally being used to delete the
list and accidently not deleting all the nodes.
We keep running across this usage pattern. Let's
remove the temptation and only allow list_delete
to handle list deletion.
Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
Diffstat (limited to 'lib/linklist.c')
| -rw-r--r-- | lib/linklist.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/linklist.c b/lib/linklist.c index 2f9e7c1e33..2306dd6d00 100644 --- a/lib/linklist.c +++ b/lib/linklist.c @@ -33,7 +33,7 @@ struct list *list_new(void) } /* Free list. */ -void list_free(struct list *l) +static void list_free_internal(struct list *l) { XFREE(MTYPE_LINK_LIST, l); } @@ -252,7 +252,7 @@ void list_delete_and_null(struct list **list) { assert(*list); list_delete_all_node(*list); - list_free(*list); + list_free_internal(*list); *list = NULL; } |