audit: eliminate audit_enabled magic number comparison

Remove comparison of audit_enabled to magic numbers outside of audit.

Related: https://github.com/linux-audit/audit-kernel/issues/86

Signed-off-by: Richard Guy Briggs <rgb@redhat.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>

diff --git a/drivers/tty/tty_audit.c b/drivers/tty/tty_audit.c
index e30aa6bf9ff9b269db368ff8fc67609c3bdd3736..50f567b6a66ee4daab87aa5ab347019c592ca274 100644 (file)
--- a/drivers/tty/tty_audit.c
+++ b/drivers/tty/tty_audit.c
@@ -92,7 +92,7 @@ static void tty_audit_buf_push(struct tty_audit_buf *buf)
 {
        if (buf->valid == 0)
                return;
-       if (audit_enabled == 0) {
+       if (audit_enabled == AUDIT_OFF) {
                buf->valid = 0;
                return;
        }

diff --git a/include/linux/audit.h b/include/linux/audit.h
index 69c78477590b089727d1e44f61230e1f4a79544b..9334fbef7bae63fab7993ead40177297ca1c9ca3 100644 (file)
--- a/include/linux/audit.h
+++ b/include/linux/audit.h
@@ -117,6 +117,9 @@ struct filename;
 
 extern void audit_log_session_info(struct audit_buffer *ab);
 
+#define AUDIT_OFF      0
+#define AUDIT_ON       1
+#define AUDIT_LOCKED   2
 #ifdef CONFIG_AUDIT
 /* These are defined in audit.c */
                                /* Public API */
@@ -202,7 +205,7 @@ static inline int audit_log_task_context(struct audit_buffer *ab)
 static inline void audit_log_task_info(struct audit_buffer *ab,
                                       struct task_struct *tsk)
 { }
-#define audit_enabled 0
+#define audit_enabled AUDIT_OFF
 #endif /* CONFIG_AUDIT */
 
 #ifdef CONFIG_AUDIT_COMPAT_GENERIC

diff --git a/include/net/xfrm.h b/include/net/xfrm.h
index 557122846e0e470ddd998ed972a84be46fae0ef7..f7f297727ed89479fbc15c23633acd2d39d158e7 100644 (file)
--- a/include/net/xfrm.h
+++ b/include/net/xfrm.h
@@ -735,7 +735,7 @@ static inline struct audit_buffer *xfrm_audit_start(const char *op)
 {
        struct audit_buffer *audit_buf = NULL;
 
-       if (audit_enabled == 0)
+       if (audit_enabled == AUDIT_OFF)
                return NULL;
        audit_buf = audit_log_start(audit_context(), GFP_ATOMIC,
                                    AUDIT_MAC_IPSEC_EVENT);

diff --git a/kernel/audit.c b/kernel/audit.c
index 5c0a1d7b0c7b4ca577421f64688e56dcefd6be20..0f3222e4eddeb72603ae53f355bcf28f70171355 100644 (file)
--- a/kernel/audit.c
+++ b/kernel/audit.c
@@ -83,9 +83,6 @@
 #define AUDIT_INITIALIZED      1
 static int     audit_initialized;
 
-#define AUDIT_OFF      0
-#define AUDIT_ON       1
-#define AUDIT_LOCKED   2
 u32            audit_enabled = AUDIT_OFF;
 bool           audit_ever_enabled = !!AUDIT_OFF;
 

diff --git a/net/netfilter/xt_AUDIT.c b/net/netfilter/xt_AUDIT.c
index f368ee6741db556ec06aa3569b21872acfeb710d..af883f1b64f927708d4bbdf462038aaaff9d9721 100644 (file)
--- a/net/netfilter/xt_AUDIT.c
+++ b/net/netfilter/xt_AUDIT.c
@@ -72,7 +72,7 @@ audit_tg(struct sk_buff *skb, const struct xt_action_param *par)
        struct audit_buffer *ab;
        int fam = -1;
 
-       if (audit_enabled == 0)
+       if (audit_enabled == AUDIT_OFF)
                goto errout;
        ab = audit_log_start(NULL, GFP_ATOMIC, AUDIT_NETFILTER_PKT);
        if (ab == NULL)

diff --git a/net/netlabel/netlabel_user.c b/net/netlabel/netlabel_user.c
index 2f328af91a5207012e14a865acde450899224db3..4676f5bb16aef85d0fa2a937e2b048b8111caf2b 100644 (file)
--- a/net/netlabel/netlabel_user.c
+++ b/net/netlabel/netlabel_user.c
@@ -101,7 +101,7 @@ struct audit_buffer *netlbl_audit_start_common(int type,
        char *secctx;
        u32 secctx_len;
 
-       if (audit_enabled == 0)
+       if (audit_enabled == AUDIT_OFF)
                return NULL;
 
        audit_buf = audit_log_start(audit_context(), GFP_ATOMIC, type);