[v1.21.x] prov/ucx: Fix incorrect return value checking for fi_param_…

…get() When reading the two runtime parameters the condition for the actions was inverted by mistake. Signed-off-by: Jianxin Xiong <[email protected]> (cherry picked from commit 8341970)
ofiwg · Dec 10, 2024 · fd154b3 · fd154b3
1 parent f8dda14
commit fd154b3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/prov/ucx/src/ucx_domain.c b/prov/ucx/src/ucx_domain.c
@@ -368,7 +368,7 @@ int ucx_domain_open(struct fid_fabric *fabric, struct fi_info *info,
 		return -ENOMEM;
 
 	ofi_status = fi_param_get_size_t(NULL, "universe_size", &univ_size);
-	if (ofi_status) {
+	if (ofi_status == FI_SUCCESS) {
 		params.estimated_num_eps = univ_size;
 		params.field_mask |= UCP_PARAM_FIELD_ESTIMATED_NUM_EPS;
 	}

diff --git a/prov/ucx/src/ucx_fabric.c b/prov/ucx/src/ucx_fabric.c
@@ -87,7 +87,7 @@ static char* ucx_local_host_resolve()
 	char *result = NULL;
 
 	status = fi_param_get(&ucx_prov, "ns_iface", &iface);
-	if (!status)
+	if (status != FI_SUCCESS)
 		iface = NULL;
 
 	if (getifaddrs(&ifaddr) == -1) {