diff options
| author | kaf24@firebug.cl.cam.ac.uk <kaf24@firebug.cl.cam.ac.uk> | 2005-11-11 12:55:47 +0100 |
|---|---|---|
| committer | kaf24@firebug.cl.cam.ac.uk <kaf24@firebug.cl.cam.ac.uk> | 2005-11-11 12:55:47 +0100 |
| commit | f8e8438eefb8d51a5c65db04dcdbc1467b991e1a (patch) | |
| tree | 592c1f9278c327fd01b0287137243c588fe7908a /tools/xenstat | |
| parent | 8b31a682c1cac7752ca1b8e0886b04bef93c89da (diff) | |
| download | xen-f8e8438eefb8d51a5c65db04dcdbc1467b991e1a.tar.gz xen-f8e8438eefb8d51a5c65db04dcdbc1467b991e1a.tar.bz2 xen-f8e8438eefb8d51a5c65db04dcdbc1467b991e1a.zip | |
Remove unnecessary NULL checks before freeing memory blocks.
The following functions check for NULL internally:
free, kfree, xfree, free_xenheap_page[s]
Signed-off-by: Keir Fraser <keir@xensource.com>
Diffstat (limited to 'tools/xenstat')
| -rw-r--r-- | tools/xenstat/libxenstat/src/xenstat.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tools/xenstat/libxenstat/src/xenstat.c b/tools/xenstat/libxenstat/src/xenstat.c index 58221ebdc6..e01ee8c4eb 100644 --- a/tools/xenstat/libxenstat/src/xenstat.c +++ b/tools/xenstat/libxenstat/src/xenstat.c @@ -284,10 +284,8 @@ void xenstat_free_node(xenstat_node * node) if (node) { if (node->domains) { - for (i = 0; i < node->num_domains; i++) { - if (node->domains[i].name) - free(node->domains[i].name); - } + for (i = 0; i < node->num_domains; i++) + free(node->domains[i].name); for (i = 0; i < NUM_COLLECTORS; i++) if((node->flags & collectors[i].flag) |