Browse Source

mpi/src/starpu_mpi.c: fix memory leak for detached requests

Nathalie Furmento 12 years ago
parent
commit
e2c56a04f9
1 changed files with 3 additions and 6 deletions
  1. 3 6
      mpi/src/starpu_mpi.c

+ 3 - 6
mpi/src/starpu_mpi.c

@@ -671,14 +671,11 @@ static void _starpu_mpi_test_detached_requests(void)
 		_STARPU_PTHREAD_MUTEX_LOCK(&detached_requests_mutex);
 
 		if (flag)
+		{
 			_starpu_mpi_req_list_erase(detached_requests, req);
+			free(req);
+		}
 
-#ifdef STARPU_DEVEL
-#warning TODO fix memleak
-#endif
-		/* Detached requests are automatically allocated by the lib */
-		//if (req->detached)
-		//	free(req);
 	}
 
 	_STARPU_PTHREAD_MUTEX_UNLOCK(&detached_requests_mutex);