From: Augustin Degomme <degomme@idpann.imag.fr>
Date: Tue, 25 Jun 2013 09:34:29 +0000 (+0200)
Subject: mem leaks
X-Git-Tag: v3_9_90~228
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/44a38683bc1680820d3c51bf3d02d462a575ead0

mem leaks
---

diff --git a/src/smpi/colls/coll_tuned_topo.c b/src/smpi/colls/coll_tuned_topo.c
index ce7cec739e..2fc8df425c 100644
--- a/src/smpi/colls/coll_tuned_topo.c
+++ b/src/smpi/colls/coll_tuned_topo.c
@@ -421,7 +421,7 @@ ompi_coll_tuned_topo_build_in_order_bmtree( MPI_Comm comm,
 
     vrank = (rank - root + size) % size;
 
-    bmtree = (ompi_coll_tree_t*)malloc(sizeof(ompi_coll_tree_t));
+    bmtree = (ompi_coll_tree_t*)xbt_malloc(sizeof(ompi_coll_tree_t));
     if (!bmtree) {
         XBT_DEBUG("coll:tuned:topo:build_bmtree PANIC out of memory");
         return NULL;
diff --git a/src/smpi/colls/scatter-ompi.c b/src/smpi/colls/scatter-ompi.c
index 205d60287c..dbc223f715 100644
--- a/src/smpi/colls/scatter-ompi.c
+++ b/src/smpi/colls/scatter-ompi.c
@@ -153,6 +153,8 @@ smpi_coll_tuned_scatter_ompi_binomial(void *sbuf, int scount,
 	smpi_mpi_recv(ptmp, rcount, rdtype, bmtree->tree_prev,
 				MCA_COLL_BASE_TAG_SCATTER, comm, &status);
     }
+    //!FIXME : store the tree, as done in ompi, instead of calculating it each time ?
+    xbt_free(bmtree);
 
     return MPI_SUCCESS;