Fix memory leak in mca_spml_ucx_register

David Wootton · David Wootton · commit 296d9badc7a9 · 2022-12-05T11:49:51.000-05:00
Coverity CID 1498721

Signed-off-by: David Wootton &lt;dwootton@us.ibm.com&gt;
diff --git a/oshmem/mca/spml/ucx/spml_ucx.c b/oshmem/mca/spml/ucx/spml_ucx.c
@@ -709,7 +709,7 @@ sshmem_mkey_t *mca_spml_ucx_register(void* addr,
     if (MEMHEAP_SEG_INVALID == segno) {
         SPML_UCX_ERROR("mca_spml_ucx_register failed because of invalid "
             "segment number: %d\n", segno);
-        return NULL;
+        goto error_out;
     }
     mem_seg = memheap_find_seg(segno);
 

Original file line number	Diff line number	Diff line change
`@@ -709,7 +709,7 @@ sshmem_mkey_t mca_spml_ucx_register(void addr,`
`709`	`709`	`if (MEMHEAP_SEG_INVALID == segno) {`
`710`	`710`	`SPML_UCX_ERROR("mca_spml_ucx_register failed because of invalid "`
`711`	`711`	`"segment number: %d\n", segno);`
`712`		`- return NULL;`
	`712`	`+ goto error_out;`
`713`	`713`	`}`
`714`	`714`	`mem_seg = memheap_find_seg(segno);`
`715`	`715`