Merge pull request #3344 from rhc54/topic/cov

Ralph Castain · web-flow · commit 004d655e3803 · 2017-04-13T12:54:08.000-07:00
Minor coverity cleanups
diff --git a/orte/mca/odls/alps/odls_alps_module.c b/orte/mca/odls/alps/odls_alps_module.c
@@ -434,7 +434,7 @@ static int do_child(orte_odls_spawn_caddy_t *cd, int write_fd)
                                  "orted",
                                  cd->wdir,
                                  orte_process_info.nodename,
-                                 cd->child->app_rank);
+                                 (NULL == cd->child) ? 0 : cd->child->app_rank);
             /* Does not return */
         }
     }
diff --git a/orte/mca/odls/default/odls_default_module.c b/orte/mca/odls/default/odls_default_module.c
@@ -425,7 +425,7 @@ static int do_child(orte_odls_spawn_caddy_t *cd, int write_fd)
                                  "orted",
                                  cd->wdir,
                                  orte_process_info.nodename,
-                                 cd->child->app_rank);
+                                 (NULL == cd->child) ? 0 : cd->child->app_rank);
             /* Does not return */
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -434,7 +434,7 @@ static int do_child(orte_odls_spawn_caddy_t *cd, int write_fd)`
`434`	`434`	`"orted",`
`435`	`435`	`cd->wdir,`
`436`	`436`	`orte_process_info.nodename,`
`437`		`- cd->child->app_rank);`
	`437`	`+ (NULL == cd->child) ? 0 : cd->child->app_rank);`
`438`	`438`	`/* Does not return */`
`439`	`439`	`}`
`440`	`440`	`}`
Original file line number	Diff line number	Diff line change
`@@ -425,7 +425,7 @@ static int do_child(orte_odls_spawn_caddy_t *cd, int write_fd)`
`425`	`425`	`"orted",`
`426`	`426`	`cd->wdir,`
`427`	`427`	`orte_process_info.nodename,`
`428`		`- cd->child->app_rank);`
	`428`	`+ (NULL == cd->child) ? 0 : cd->child->app_rank);`
`429`	`429`	`/* Does not return */`
`430`	`430`	`}`
`431`	`431`	`}`