Skip to content

Commit 470eaca

Browse files
authored
Merge pull request #8777 from abouteiller/bugfix/spawn-info
spawn with info: do not erroneously free(tmp)
2 parents 3d67c65 + d692373 commit 470eaca

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

ompi/dpm/dpm.c

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1247,7 +1247,6 @@ int ompi_dpm_spawn(int count, const char *array_of_commands[],
12471247
ompi_info_get (array_of_info[i], "pernode", &info_str, &flag);
12481248
if ( flag ) {
12491249
rc = dpm_convert(&job_info, "pernode", PMIX_MAPBY, "PPR:1:NODE", NULL, true);
1250-
free(tmp);
12511250
OBJ_RELEASE(info_str);
12521251
if (OMPI_SUCCESS != rc) {
12531252
OPAL_LIST_DESTRUCT(&job_info);
@@ -1298,7 +1297,6 @@ int ompi_dpm_spawn(int count, const char *array_of_commands[],
12981297
return MPI_ERR_SPAWN;
12991298
}
13001299
rc = dpm_convert(&job_info, "ppr", PMIX_MAPBY, info_str->string, NULL, true);
1301-
free(tmp);
13021300
OBJ_RELEASE(info_str);
13031301
if (OMPI_SUCCESS != rc) {
13041302
OPAL_LIST_DESTRUCT(&job_info);
@@ -1319,7 +1317,6 @@ int ompi_dpm_spawn(int count, const char *array_of_commands[],
13191317
ompi_info_get(array_of_info[i], "map_by", &info_str, &flag);
13201318
if ( flag ) {
13211319
rc = dpm_convert(&job_info, "map_by", PMIX_MAPBY, info_str->string, NULL, false);
1322-
free(tmp);
13231320
OBJ_RELEASE(info_str);
13241321
if (OMPI_SUCCESS != rc) {
13251322
OPAL_LIST_DESTRUCT(&job_info);
@@ -1357,7 +1354,6 @@ int ompi_dpm_spawn(int count, const char *array_of_commands[],
13571354
ompi_info_get(array_of_info[i], "rank_by", &info_str, &flag);
13581355
if ( flag ) {
13591356
rc = dpm_convert(&job_info, "rank_by", PMIX_RANKBY, info_str->string, NULL, false);
1360-
free(tmp);
13611357
OBJ_RELEASE(info_str);
13621358
if (OMPI_SUCCESS != rc) {
13631359
OPAL_LIST_DESTRUCT(&job_info);
@@ -1389,7 +1385,6 @@ int ompi_dpm_spawn(int count, const char *array_of_commands[],
13891385
ompi_info_get(array_of_info[i], "bind_to", &info_str, &flag);
13901386
if ( flag ) {
13911387
rc = dpm_convert(&job_info, "bind_to", PMIX_BINDTO, info_str->string, NULL, false);
1392-
free(tmp);
13931388
OBJ_RELEASE(info_str);
13941389
if (OMPI_SUCCESS != rc) {
13951390
OPAL_LIST_DESTRUCT(&job_info);

0 commit comments

Comments
 (0)