Skip to content

Commit 088b6cd

Browse files
author
Ralph Castain
committed
Silence coverity warnings
Signed-off-by: Ralph Castain <[email protected]>
1 parent 4e76379 commit 088b6cd

File tree

2 files changed

+4
-26
lines changed

2 files changed

+4
-26
lines changed

opal/mca/pmix/pmix2x/pmix/src/mca/ptl/tcp/ptl_tcp.c

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -319,6 +319,9 @@ static pmix_status_t connect_to_peer(struct pmix_peer_t *peer,
319319

320320
/* do a final bozo check */
321321
if (NULL == nspace || PMIX_RANK_WILDCARD == rank) {
322+
if (NULL != nspace) {
323+
free(nspace);
324+
}
322325
CLOSE_THE_SOCKET(sd);
323326
return PMIX_ERR_UNREACH;
324327
}

orte/mca/ess/base/ess_base_std_tool.c

Lines changed: 1 addition & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,7 @@ int orte_ess_base_tool_setup(void)
9898
ORTE_PROC_MY_NAME->jobid = OPAL_PROC_MY_NAME.jobid;
9999
ORTE_PROC_MY_NAME->vpid = OPAL_PROC_MY_NAME.vpid;
100100
} else {
101-
/* if we connected to a PMIx server, then we were assigned
102-
* a name that we should use. Otherwise, we have to define
103-
* one here */
101+
/* we have to define our name here */
104102
if (NULL != orte_ess_base_jobid &&
105103
NULL != orte_ess_base_vpid) {
106104
opal_output_verbose(2, orte_ess_base_framework.framework_output,
@@ -142,29 +140,6 @@ int orte_ess_base_tool_setup(void)
142140
orte_process_info.super.proc_arch = opal_local_arch;
143141
opal_proc_local_set(&orte_process_info.super);
144142

145-
/* setup the PMIx framework - ensure it skips all non-PMIx components,
146-
* but do not override anything we were given */
147-
opal_setenv("OMPI_MCA_pmix", "^s1,s2,cray,isolated", false, &environ);
148-
if (OPAL_SUCCESS != (ret = mca_base_framework_open(&opal_pmix_base_framework, 0))) {
149-
ORTE_ERROR_LOG(ret);
150-
error = "orte_pmix_base_open";
151-
goto error;
152-
}
153-
if (ORTE_SUCCESS != (ret = opal_pmix_base_select())) {
154-
ORTE_ERROR_LOG(ret);
155-
error = "opal_pmix_base_select";
156-
goto error;
157-
}
158-
/* initialize - the layer below has our name in opal_process_name_t
159-
* and will pass it to PMIx to sync */
160-
if (OPAL_SUCCESS != (ret = opal_pmix.tool_init(NULL))) {
161-
ORTE_ERROR_LOG(ret);
162-
error = "opal_pmix.init";
163-
goto error;
164-
}
165-
/* set the event base */
166-
opal_pmix_base_set_evbase(orte_event_base);
167-
168143
/* open and setup the state machine */
169144
if (ORTE_SUCCESS != (ret = mca_base_framework_open(&orte_state_base_framework, 0))) {
170145
ORTE_ERROR_LOG(ret);

0 commit comments

Comments
 (0)