From c5cfdf988dc28c64bae948c5a749d0b44e881426 Mon Sep 17 00:00:00 2001 From: Arnaud Giersch Date: Wed, 1 Feb 2012 17:04:45 +0100 Subject: [PATCH 1/1] These free's are buggy. AFAICT, it's the responsibility of smpi_mpi_waitall to free the requests. --- src/smpi/smpi_coll.c | 6 ------ 1 file changed, 6 deletions(-) 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); } -- 2.20.1