This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "MPICH primary repository". The branch, master has been updated via a00839739260055c6d2373f41e2a0c53abf679d1 (commit) from 172968b2ed596d5ccd444ea10b9bead5104ef7bf (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- http://git.mpich.org/mpich.git/commitdiff/a00839739260055c6d2373f41e2a0c53ab... commit a00839739260055c6d2373f41e2a0c53abf679d1 Author: Su Huang <[email protected]> Date: Tue Apr 29 08:49:51 2014 -0400 pamid: datatype is not freed in MPID_Win_accumulate In MPID_Win_accumulate, when processing non-contiguous datatype for origin, the ref_count of datatype is incremented by one which prevents the space allocated for contents and datatloop from being freed. MPID_Datatype_add_ref(req->origin.dt.pointer); In the routine, the data is processed by MPIR_Localcopy and no need to increment the reference count (ref_count) of the object by 1. Signed-off-by: Michael Blocksome <[email protected]> diff --git a/src/mpid/pamid/src/onesided/mpid_win_accumulate.c b/src/mpid/pamid/src/onesided/mpid_win_accumulate.c index 6ee57d2..714852f 100644 --- a/src/mpid/pamid/src/onesided/mpid_win_accumulate.c +++ b/src/mpid/pamid/src/onesided/mpid_win_accumulate.c @@ -270,7 +270,6 @@ MPID_Accumulate(const void *origin_addr, req->buffer_free = 1; req->buffer = MPIU_Malloc(req->origin.dt.size); MPID_assert(req->buffer != NULL); - MPID_Datatype_add_ref(req->origin.dt.pointer); int mpi_errno = 0; mpi_errno = MPIR_Localcopy(origin_addr, origin_count, ----------------------------------------------------------------------- Summary of changes: src/mpid/pamid/src/onesided/mpid_win_accumulate.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) hooks/post-receive -- MPICH primary repository