Skip to content

Commit 70a2098

Browse files
authored
Merge pull request #4655 from yosefe/topic/spml-ucx-fix-rkey-leak
spml_ucx: fix rkey leak
2 parents 697a943 + 1193e1e commit 70a2098

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

oshmem/mca/spml/ucx/spml_ucx.c

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -528,12 +528,13 @@ int mca_spml_ucx_deregister(sshmem_mkey_t *mkeys)
528528
if (!mkeys[0].spml_context)
529529
return OSHMEM_SUCCESS;
530530

531-
mem_seg = memheap_find_va(mkeys[0].va_base);
531+
mem_seg = memheap_find_va(mkeys[0].va_base);
532+
ucx_mkey = (spml_ucx_mkey_t*)mkeys[0].spml_context;
532533

533534
if (MAP_SEGMENT_ALLOC_UCX != mem_seg->type) {
534-
ucx_mkey = (spml_ucx_mkey_t *)mkeys[0].spml_context;
535535
ucp_mem_unmap(mca_spml_ucx.ucp_context, ucx_mkey->mem_h);
536536
}
537+
ucp_rkey_destroy(ucx_mkey->rkey);
537538

538539
if (0 < mkeys[0].len) {
539540
ucp_rkey_buffer_release(mkeys[0].u.data);

0 commit comments

Comments
 (0)