selinux: rename the {is,set}_enforcing() functions
authorPaul Moore <paul@paul-moore.com>
Thu, 1 Mar 2018 22:38:30 +0000 (17:38 -0500)
committerPaul Moore <paul@paul-moore.com>
Fri, 2 Mar 2018 19:18:55 +0000 (14:18 -0500)
Rename is_enforcing() to enforcing_enabled() and
enforcing_set() to set_enforcing().

Signed-off-by: Paul Moore <paul@paul-moore.com>
security/selinux/avc.c
security/selinux/hooks.c
security/selinux/include/security.h
security/selinux/selinuxfs.c
security/selinux/ss/services.c
security/selinux/ss/status.c

index 36124f4..54b09cc 100644 (file)
@@ -985,7 +985,7 @@ static noinline int avc_denied(u32 ssid, u32 tsid,
        if (flags & AVC_STRICT)
                return -EACCES;
 
-       if (is_enforcing(&selinux_state) &&
+       if (enforcing_enabled(&selinux_state) &&
            !(avd->flags & AVD_FLAGS_PERMISSIVE))
                return -EACCES;
 
index d78f9e2..337fb32 100644 (file)
@@ -5407,7 +5407,7 @@ static int selinux_nlmsg_perm(struct sock *sk, struct sk_buff *skb)
                               sk->sk_protocol, nlh->nlmsg_type,
                               secclass_map[sksec->sclass - 1].name,
                               task_pid_nr(current), current->comm);
-                       if (!is_enforcing(&selinux_state) ||
+                       if (!enforcing_enabled(&selinux_state) ||
                            security_get_allow_unknown(&selinux_state))
                                err = 0;
                }
@@ -6937,7 +6937,7 @@ static __init int selinux_init(void)
        printk(KERN_INFO "SELinux:  Initializing.\n");
 
        memset(&selinux_state, 0, sizeof(selinux_state));
-       set_enforcing(&selinux_state, selinux_enforcing_boot);
+       enforcing_set(&selinux_state, selinux_enforcing_boot);
        selinux_state.checkreqprot = selinux_checkreqprot_boot;
        selinux_ss_init(&selinux_state.ss);
 
index c3a1ef1..f1db09a 100644 (file)
@@ -111,22 +111,22 @@ void selinux_ss_init(struct selinux_ss **ss);
 extern struct selinux_state selinux_state;
 
 #ifdef CONFIG_SECURITY_SELINUX_DEVELOP
-static inline bool is_enforcing(struct selinux_state *state)
+static inline bool enforcing_enabled(struct selinux_state *state)
 {
        return state->enforcing;
 }
 
-static inline void set_enforcing(struct selinux_state *state, bool value)
+static inline void enforcing_set(struct selinux_state *state, bool value)
 {
        state->enforcing = value;
 }
 #else
-static inline bool is_enforcing(struct selinux_state *state)
+static inline bool enforcing_enabled(struct selinux_state *state)
 {
        return true;
 }
 
-static inline void set_enforcing(struct selinux_state *state, bool value)
+static inline void enforcing_set(struct selinux_state *state, bool value)
 {
 }
 #endif
index 9849275..0dbd5fd 100644 (file)
@@ -98,7 +98,7 @@ static ssize_t sel_read_enforce(struct file *filp, char __user *buf,
        ssize_t length;
 
        length = scnprintf(tmpbuf, TMPBUFLEN, "%d",
-                          is_enforcing(&selinux_state));
+                          enforcing_enabled(&selinux_state));
        return simple_read_from_buffer(buf, count, ppos, tmpbuf, length);
 }
 
@@ -128,7 +128,7 @@ static ssize_t sel_write_enforce(struct file *file, const char __user *buf,
 
        new_value = !!new_value;
 
-       old_value = is_enforcing(&selinux_state);
+       old_value = enforcing_enabled(&selinux_state);
 
        if (new_value != old_value) {
                length = avc_has_perm(current_sid(), SECINITSID_SECURITY,
@@ -141,7 +141,7 @@ static ssize_t sel_write_enforce(struct file *file, const char __user *buf,
                        new_value, old_value,
                        from_kuid(&init_user_ns, audit_get_loginuid(current)),
                        audit_get_sessionid(current));
-               set_enforcing(&selinux_state, new_value);
+               enforcing_set(&selinux_state, new_value);
                if (new_value)
                        avc_ss_reset(0);
                selnl_notify_setenforce(new_value);
index 3698352..4785ca5 100644 (file)
@@ -752,7 +752,7 @@ out:
        kfree(n);
        kfree(t);
 
-       if (!is_enforcing(state))
+       if (!enforcing_enabled(state))
                return 0;
        return -EPERM;
 }
@@ -1596,7 +1596,7 @@ out:
        kfree(s);
        kfree(t);
        kfree(n);
-       if (!is_enforcing(state))
+       if (!enforcing_enabled(state))
                return 0;
        return -EACCES;
 }
@@ -1907,7 +1907,7 @@ static inline int convert_context_handle_invalid_context(
        char *s;
        u32 len;
 
-       if (is_enforcing(state))
+       if (enforcing_enabled(state))
                return -EINVAL;
 
        if (!context_struct_to_string(policydb, context, &s, &len)) {
index 043efc5..a121de4 100644 (file)
@@ -56,7 +56,7 @@ struct page *selinux_kernel_status_page(struct selinux_state *state)
 
                        status->version = SELINUX_KERNEL_STATUS_VERSION;
                        status->sequence = 0;
-                       status->enforcing = is_enforcing(state);
+                       status->enforcing = enforcing_enabled(state);
                        /*
                         * NOTE: the next policyload event shall set
                         * a positive value on the status->policyload,