ctdb-common: Mark ctdb_fatal() and ctdb_die() as _NORETURN_

This avoids static analysers continuing analysis after calls to these
functions and producing incorrect warnings.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>

diff --git a/ctdb/common/common.h b/ctdb/common/common.h
index 6b2c98ef383e28841ca8fb4f19cbf4ac98d3ed5d..79f6b9ed03a51083ab75815f18d8dac75c0450dc 100644 (file)
--- a/ctdb/common/common.h
+++ b/ctdb/common/common.h
@@ -20,6 +20,8 @@
 #ifndef __CTDB_COMMON_H__
 #define __CTDB_COMMON_H__
 
+#include "lib/util/attr.h"
+
 /* From common/ctdb_io.c */
 
 typedef void (*ctdb_queue_cb_fn_t)(uint8_t *data, size_t length,
@@ -91,9 +93,9 @@ const char *ctdb_errstr(struct ctdb_context *ctdb);
 void ctdb_set_error(struct ctdb_context *ctdb, const char *fmt, ...)
                    PRINTF_ATTRIBUTE(2,3);
 
-void ctdb_fatal(struct ctdb_context *ctdb, const char *msg);
+void ctdb_fatal(struct ctdb_context *ctdb, const char *msg) _NORETURN_;
 
-void ctdb_die(struct ctdb_context *ctdb, const char *msg);
+void ctdb_die(struct ctdb_context *ctdb, const char *msg) _NORETURN_;
 
 bool ctdb_set_helper(const char *type, char *helper, size_t size,
                     const char *envvar,