fcoll/vulcan: fix memory leak

we didn't correctly free the fh->f_aggr_list array in the vulcan file_write_all file. Thanks @andymwood for reporting the issue and @ggouaillardet for identifying the cause for the leak. Fixes Issue open-mpi#12677 (at least partially) Signed-off-by: Edgar Gabriel <[email protected]>
edgargabriel · Sep 6, 2024 · 3fde6af · 3fde6af
1 parent e25e897
commit 3fde6af
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/ompi/mca/fcoll/vulcan/fcoll_vulcan_file_write_all.c b/ompi/mca/fcoll/vulcan/fcoll_vulcan_file_write_all.c
@@ -768,8 +768,10 @@ exit :
     }
     free(broken_iov_arrays);
     free(fh->f_procs_in_group);
+    free(fh->f_aggr_list);
     fh->f_procs_in_group=NULL;
     fh->f_procs_per_group=0;
+    fh->f_aggr_list=NULL;
     free(result_counts);
     free(reqs);