Skip to content

Commit d692373

Browse files
committed
spawn with info: do not erroneously free(tmp)
In some cases we would issue free(tmp) on values that have not been created from asprintf, thus crashing Signed-off-by: Aurelien Bouteiller <[email protected]>
1 parent ca65630 commit d692373

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)