diff options
author | Simo Sorce <idra@samba.org> | 2001-09-17 02:19:44 +0000 |
---|---|---|
committer | Simo Sorce <idra@samba.org> | 2001-09-17 02:19:44 +0000 |
commit | 484a7c0341fe033fe26fe1e6b597ed1c456c39d4 (patch) | |
tree | c2f2344c757fffba5b6cc190e6ed13ef64b89266 /source3/lib/util.c | |
parent | 31c3f7a8b2ff62d4cfc1ed9d831b95eba0da525f (diff) | |
download | samba-484a7c0341fe033fe26fe1e6b597ed1c456c39d4.tar.gz samba-484a7c0341fe033fe26fe1e6b597ed1c456c39d4.tar.bz2 samba-484a7c0341fe033fe26fe1e6b597ed1c456c39d4.zip |
move to SAFE_FREE()
(This used to be commit 60e907b7e8e1c008463a88ed2b076344278986ef)
Diffstat (limited to 'source3/lib/util.c')
-rw-r--r-- | source3/lib/util.c | 24 |
1 files changed, 10 insertions, 14 deletions
diff --git a/source3/lib/util.c b/source3/lib/util.c index d1d052d4a0..3d2de63f38 100644 --- a/source3/lib/util.c +++ b/source3/lib/util.c @@ -171,7 +171,7 @@ char *get_numlist(char *p, uint32 **num, int *count) tn = Realloc((*num), ((*count)+1) * sizeof(uint32)); if (tn == NULL) { - if (*num) free(*num); + SAFE_FREE(*num); return NULL; } else (*num) = tn; @@ -638,7 +638,7 @@ void *Realloc(void *p,size_t size) void *ret=NULL; if (size == 0) { - if (p) free(p); + SAFE_FREE(p); DEBUG(5,("Realloc asked for 0 bytes\n")); return NULL; } @@ -656,17 +656,15 @@ void *Realloc(void *p,size_t size) /**************************************************************************** -free memory, checks for NULL +free memory, checks for NULL and set to NULL +use directly SAFE_FREE() +exist only because we need to pass a function pointer somewhere --SSS ****************************************************************************/ void safe_free(void *p) { - if (p != NULL) - { - free(p); - } + SAFE_FREE(p); } - /**************************************************************************** get my own name and IP ****************************************************************************/ @@ -1243,13 +1241,11 @@ routine to free a namearray. void free_namearray(name_compare_entry *name_array) { - if(name_array == 0) + if(name_array == NULL) return; - if(name_array->name != NULL) - free(name_array->name); - - free((char *)name_array); + SAFE_FREE(name_array->name); + SAFE_FREE(name_array); } /**************************************************************************** @@ -1498,7 +1494,7 @@ zero a memory area then free it. Used to catch bugs faster void zero_free(void *p, size_t size) { memset(p, 0, size); - free(p); + SAFE_FREE(p); } |