From eae182adecd515b7a4bf7f7a25a98b039be3cd6d Mon Sep 17 00:00:00 2001
From: Danny Auble <da@schedmd.com>
Date: Tue, 19 Aug 2014 14:34:20 -0700
Subject: [PATCH] Clearer checking for return from route_g_init
---
src/common/slurm_route.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/common/slurm_route.c b/src/common/slurm_route.c
index aba6db6dbc6..d85af0edf2b 100644
--- a/src/common/slurm_route.c
+++ b/src/common/slurm_route.c
@@ -348,7 +348,7 @@ extern int route_g_split_hostlist(hostlist_t hl,
char *buf;
nnodes = nnodex = 0;
- if (route_g_init(NULL) < 0)
+ if (route_g_init(NULL) != SLURM_SUCCESS)
return SLURM_ERROR;
if (debug_flags & DEBUG_FLAG_ROUTE) {
@@ -387,7 +387,7 @@ extern int route_g_split_hostlist(hostlist_t hl,
*/
extern int route_g_reconfigure(void)
{
- if (route_g_init(NULL) < 0)
+ if (route_g_init(NULL) != SLURM_SUCCESS)
return SLURM_ERROR;
debug_flags = slurm_get_debug_flags();
tree_width = slurm_get_tree_width();
@@ -407,7 +407,7 @@ extern int route_g_reconfigure(void)
*/
extern slurm_addr_t* route_g_next_collector ( bool *is_collector )
{
- if (route_g_init(NULL) < 0)
+ if (route_g_init(NULL) != SLURM_SUCCESS)
return NULL;
return (*(ops.next_collector))(is_collector);
@@ -423,7 +423,7 @@ extern slurm_addr_t* route_g_next_collector ( bool *is_collector )
*/
extern slurm_addr_t* route_g_next_collector_backup ( void )
{
- if (route_g_init(NULL) < 0)
+ if (route_g_init(NULL) != SLURM_SUCCESS)
return NULL;
return (*(ops.next_collector_backup))();
--
GitLab