From d1ea0b490434a8cb3a2a2e9188fe6bd6b3ca607c Mon Sep 17 00:00:00 2001
From: Tim Wickberg <tim@schedmd.com>
Date: Wed, 28 Sep 2016 08:20:01 -0400
Subject: [PATCH] Prevent 'sacctmgr shutdown' from always returning 1.
Add in missing brackets. Caught by GCC 6.1 -Wall.
---
src/sacctmgr/sacctmgr.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/sacctmgr/sacctmgr.c b/src/sacctmgr/sacctmgr.c
index 963c16b4c4e..effd6a77794 100644
--- a/src/sacctmgr/sacctmgr.c
+++ b/src/sacctmgr/sacctmgr.c
@@ -519,10 +519,11 @@ _process_command (int argc, char *argv[])
}
rc = slurmdb_shutdown(db_conn);
- if (rc != SLURM_SUCCESS)
+ if (rc != SLURM_SUCCESS) {
fprintf(stderr, " Problem shutting down server: %s\n",
slurm_strerror(rc));
exit_code = 1;
+ }
} else if (strncasecmp (argv[0], "version", MAX(command_len, 4)) == 0) {
if (argc > 1) {
exit_code = 1;
--
GitLab