Skip to content

Commit 4bc0fb7

Browse files
committed
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 #12677 (at least partially) Signed-off-by: Edgar Gabriel <[email protected]> (cherry picked from commit 3fde6af)
1 parent 50d79c3 commit 4bc0fb7

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

ompi/mca/fcoll/vulcan/fcoll_vulcan_file_write_all.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -729,8 +729,10 @@ exit :
729729
}
730730
free(broken_iov_arrays);
731731
free(fh->f_procs_in_group);
732+
free(fh->f_aggr_list);
732733
fh->f_procs_in_group=NULL;
733734
fh->f_procs_per_group=0;
735+
fh->f_aggr_list=NULL;
734736
free(result_counts);
735737
free(reqs);
736738

0 commit comments

Comments
 (0)