From: Arnaud Giersch <arnaud.giersch@iut-bm.univ-fcomte.fr>
Date: Tue, 22 Nov 2011 12:39:39 +0000 (+0100)
Subject: Don't clear mem before free.
X-Git-Tag: exp_20120216~241^2~55
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/ee7c4c6ee1cd96af494f5f4ff9190ca7bbe82edc

Don't clear mem before free.

Use a tool like valgrind to check for memory errors.
---

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;
   }
 }