From: Arnaud Giersch <arnaud.giersch@iut-bm.univ-fcomte.fr>
Date: Wed, 1 Feb 2012 16:04:45 +0000 (+0100)
Subject: These free's are buggy.
X-Git-Tag: exp_20120216~94
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/c5cfdf988dc28c64bae948c5a749d0b44e881426?hp=46120a30db6252a5e7a9e527132589be62f8ee28;ds=inline

These free's are buggy.

AFAICT, it's the responsibility of smpi_mpi_waitall to free the
requests.
---

diff --git a/src/smpi/smpi_coll.c b/src/smpi/smpi_coll.c
index 1c05340fc9..7798de86ea 100644
--- a/src/smpi/smpi_coll.c
+++ b/src/smpi/smpi_coll.c
@@ -118,9 +118,6 @@ static void tree_bcast(void *buf, int count, MPI_Datatype datatype,
   }
   smpi_mpi_startall(tree->numChildren, requests);
   smpi_mpi_waitall(tree->numChildren, requests, MPI_STATUS_IGNORE);
-  for (i = 0; i < tree->numChildren; i++){
-    xbt_free (requests[i]);
-  }
   xbt_free(requests);
 }
 
@@ -159,9 +156,6 @@ static void tree_antibcast(void *buf, int count, MPI_Datatype datatype,
   }
   smpi_mpi_startall(tree->numChildren, requests);
   smpi_mpi_waitall(tree->numChildren, requests, MPI_STATUS_IGNORE);
-  for (i = 0; i < tree->numChildren; i++){
-    xbt_free (requests[i]);
-  }
   xbt_free(requests);
 }