Minor change: check for NULL before using the job map to avoid segfault when erroring out prior to creating the map

Ralph Castain · Ralph Castain · commit fb51d65049a7 · 2016-09-04T07:53:12.000-07:00
diff --git a/orte/mca/grpcomm/base/grpcomm_base_stubs.c b/orte/mca/grpcomm/base/grpcomm_base_stubs.c
@@ -324,7 +324,7 @@ static int create_dmns(orte_grpcomm_signature_t *sig,
             *dmns = NULL;
             return ORTE_ERR_NOT_FOUND;
         }
-        if (0 == jdata->map->num_nodes) {
+        if (NULL == jdata->map || 0 == jdata->map->num_nodes) {
             /* we haven't generated a job map yet - if we are the HNP,
              * then we should only involve ourselves. Otherwise, we have
              * no choice but to abort to avoid hangs */

Original file line number	Diff line number	Diff line change
`@@ -324,7 +324,7 @@ static int create_dmns(orte_grpcomm_signature_t *sig,`
`324`	`324`	`*dmns = NULL;`
`325`	`325`	`return ORTE_ERR_NOT_FOUND;`
`326`	`326`	`}`
`327`		`- if (0 == jdata->map->num_nodes) {`
	`327`	`+ if (NULL == jdata->map \|\| 0 == jdata->map->num_nodes) {`
`328`	`328`	`/* we haven't generated a job map yet - if we are the HNP,`
`329`	`329`	`* then we should only involve ourselves. Otherwise, we have`
`330`	`330`	`* no choice but to abort to avoid hangs */`