diff options
author | Prarit Bhargava <prarit@redhat.com> | 2016-04-14 10:40:57 -0400 |
---|---|---|
committer | Paul Moore <paul@paul-moore.com> | 2016-04-14 11:24:50 -0400 |
commit | 0fd71a620be8648486a126fccadf9f7c2a818676 (patch) | |
tree | 7aa1433928d9738e64a01b57a5ce241f2f2ffe1d /security | |
parent | 61d612ea731e57dc510472fb746b55cdc017f371 (diff) |
selinux: Change bool variable name to index.
security_get_bool_value(int bool) argument "bool" conflicts with
in-kernel macros such as BUILD_BUG(). This patch changes this to
index which isn't a type.
Cc: Paul Moore <paul@paul-moore.com>
Cc: Stephen Smalley <sds@tycho.nsa.gov>
Cc: Eric Paris <eparis@parisplace.org>
Cc: James Morris <james.l.morris@oracle.com>
Cc: "Serge E. Hallyn" <serge@hallyn.com>
Cc: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Cc: Andrew Perepechko <anserper@ya.ru>
Cc: Jeff Vander Stoep <jeffv@google.com>
Cc: selinux@tycho.nsa.gov
Cc: Eric Paris <eparis@redhat.com>
Cc: Paul Moore <pmoore@redhat.com>
Cc: David Howells <dhowells@redhat.com>
Signed-off-by: Prarit Bhargava <prarit@redhat.com>
Acked-by: David Howells <dhowells@redhat.com>
[PM: wrapped description for checkpatch.pl, use "selinux:..." as subj]
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security')
-rw-r--r-- | security/selinux/include/conditional.h | 2 | ||||
-rw-r--r-- | security/selinux/ss/services.c | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/security/selinux/include/conditional.h b/security/selinux/include/conditional.h index 67ce7a8d8301..ff4fddca9050 100644 --- a/security/selinux/include/conditional.h +++ b/security/selinux/include/conditional.h | |||
@@ -17,6 +17,6 @@ int security_get_bools(int *len, char ***names, int **values); | |||
17 | 17 | ||
18 | int security_set_bools(int len, int *values); | 18 | int security_set_bools(int len, int *values); |
19 | 19 | ||
20 | int security_get_bool_value(int bool); | 20 | int security_get_bool_value(int index); |
21 | 21 | ||
22 | #endif | 22 | #endif |
diff --git a/security/selinux/ss/services.c b/security/selinux/ss/services.c index ebda97333f1b..89df64672b89 100644 --- a/security/selinux/ss/services.c +++ b/security/selinux/ss/services.c | |||
@@ -2696,7 +2696,7 @@ out: | |||
2696 | return rc; | 2696 | return rc; |
2697 | } | 2697 | } |
2698 | 2698 | ||
2699 | int security_get_bool_value(int bool) | 2699 | int security_get_bool_value(int index) |
2700 | { | 2700 | { |
2701 | int rc; | 2701 | int rc; |
2702 | int len; | 2702 | int len; |
@@ -2705,10 +2705,10 @@ int security_get_bool_value(int bool) | |||
2705 | 2705 | ||
2706 | rc = -EFAULT; | 2706 | rc = -EFAULT; |
2707 | len = policydb.p_bools.nprim; | 2707 | len = policydb.p_bools.nprim; |
2708 | if (bool >= len) | 2708 | if (index >= len) |
2709 | goto out; | 2709 | goto out; |
2710 | 2710 | ||
2711 | rc = policydb.bool_val_to_struct[bool]->state; | 2711 | rc = policydb.bool_val_to_struct[index]->state; |
2712 | out: | 2712 | out: |
2713 | read_unlock(&policy_rwlock); | 2713 | read_unlock(&policy_rwlock); |
2714 | return rc; | 2714 | return rc; |