Skip to content

Commit 3fde6af

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]>
1 parent e25e897 commit 3fde6af

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
@@ -768,8 +768,10 @@ exit :
768768
}
769769
free(broken_iov_arrays);
770770
free(fh->f_procs_in_group);
771+
free(fh->f_aggr_list);
771772
fh->f_procs_in_group=NULL;
772773
fh->f_procs_per_group=0;
774+
fh->f_aggr_list=NULL;
773775
free(result_counts);
774776
free(reqs);
775777

0 commit comments

Comments
 (0)