Skip to content

Commit

Permalink
Merge pull request #8 from dmitrygx/topic/fix_request_leak
Browse files Browse the repository at this point in the history
TEST/APPS: Fix UCP request leak in case of unsuccessful ucp_stream_send_nb
  • Loading branch information
yosefe authored Apr 28, 2020
2 parents 5d6c73d + c08f82c commit 2ba6834
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions test/apps/iodemo/ucx_wrapper.cc
Original file line number Diff line number Diff line change
Expand Up @@ -594,6 +594,8 @@ bool UcxConnection::connect_common(ucp_ep_params_t& ep_params)
stream_send_callback, 0);
if (!_context.wait_completion(sreq, 10)) {
UCX_CONN_LOG << "failed to send remote connection id";
ucp_request_cancel(_context.worker(), rreq);
ucp_request_release(rreq);
ep_close(UCP_EP_CLOSE_MODE_FORCE);
return false;
}
Expand Down

0 comments on commit 2ba6834

Please sign in to comment.