Skip to content

Commit 1ceb35b

Browse files
author
Ralph Castain
committed
Fix singletons - do not include the PMIx tool URI in the environment provided to child processes
1 parent 2414244 commit 1ceb35b

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

opal/mca/pmix/pmix2x/pmix/src/server/pmix_server.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -314,6 +314,8 @@ PMIX_EXPORT pmix_status_t PMIx_server_init(pmix_server_module_t *module,
314314
}
315315
snprintf(tl->address.sun_path, sizeof(tl->address.sun_path) - 1, "%s", pmix_pid);
316316
free(pmix_pid);
317+
/* we don't provide a URI for this listener as we don't pass
318+
* the TOOL connection URI to a child process */
317319
pmix_server_globals.tool_connections_allowed = true;
318320
pmix_list_append(&pmix_server_globals.listeners, &tl->super);
319321
/* push this onto our protected list of keys not
@@ -981,6 +983,9 @@ PMIX_EXPORT pmix_status_t PMIx_server_setup_fork(const pmix_proc_t *proc, char *
981983
pmix_setenv("PMIX_RANK", rankstr, true, env);
982984
/* pass our rendezvous info */
983985
PMIX_LIST_FOREACH(lt, &pmix_server_globals.listeners, pmix_listener_t) {
986+
if (NULL == lt->uri) {
987+
continue;
988+
}
984989
pmix_setenv(lt->varname, lt->uri, true, env);
985990
}
986991
/* pass our active security mode */

0 commit comments

Comments
 (0)