Commit f49534e1 authored by Min Si's avatar Min Si Committed by Pavan Balaji
Browse files

Bugfix: free internal req object in fn_fail.



In fn_fail path, we should free the internal req object since req_ptr is
pointed to the req object only at fn_exit.
Signed-off-by: default avatarXin Zhao <xinzhao3@illinois.edu>
Signed-off-by: Pavan Balaji's avatarPavan Balaji <balaji@anl.gov>
parent 5f544987
...@@ -483,10 +483,10 @@ static int issue_from_origin_buffer(MPIDI_RMA_Op_t * rma_op, MPIDI_VC_t * vc, ...@@ -483,10 +483,10 @@ static int issue_from_origin_buffer(MPIDI_RMA_Op_t * rma_op, MPIDI_VC_t * vc,
MPIDI_FUNC_EXIT(MPID_STATE_ISSUE_FROM_ORIGIN_BUFFER); MPIDI_FUNC_EXIT(MPID_STATE_ISSUE_FROM_ORIGIN_BUFFER);
return mpi_errno; return mpi_errno;
fn_fail: fn_fail:
if ((*req_ptr)) { if (req) {
if ((*req_ptr)->dev.datatype_ptr) if (req->dev.datatype_ptr)
MPID_Datatype_release((*req_ptr)->dev.datatype_ptr); MPID_Datatype_release(req->dev.datatype_ptr);
MPID_Request_release((*req_ptr)); MPID_Request_release(req);
} }
(*req_ptr) = NULL; (*req_ptr) = NULL;
goto fn_exit; goto fn_exit;
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment