From ee7c4c6ee1cd96af494f5f4ff9190ca7bbe82edc Mon Sep 17 00:00:00 2001 From: Arnaud Giersch Date: Tue, 22 Nov 2011 13:39:39 +0100 Subject: [PATCH] Don't clear mem before free. Use a tool like valgrind to check for memory errors. --- src/xbt/dynar.c | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) diff --git a/src/xbt/dynar.c b/src/xbt/dynar.c index c16e2458bb..2ab87598bb 100644 --- a/src/xbt/dynar.c +++ b/src/xbt/dynar.c @@ -228,18 +228,11 @@ xbt_dynar_new_sync(const unsigned long elmsize, void xbt_dynar_free_container(xbt_dynar_t * dynar) { if (dynar && *dynar) { - - if ((*dynar)->data) { - _xbt_clear_mem((*dynar)->data, (*dynar)->size); - free((*dynar)->data); - } - - if ((*dynar)->mutex) - xbt_mutex_destroy((*dynar)->mutex); - - _xbt_clear_mem(*dynar, sizeof(s_xbt_dynar_t)); - - free(*dynar); + xbt_dynar_t d = *dynar; + free(d->data); + if (d->mutex) + xbt_mutex_destroy(d->mutex); + free(d); *dynar = NULL; } } -- 2.20.1