summaryrefslogtreecommitdiff
path: root/security/selinux/status.c
diff options
context:
space:
mode:
authorChristian Göttsche <cgzones@googlemail.com>2023-07-06 16:23:34 +0300
committerPaul Moore <paul@paul-moore.com>2023-07-19 01:29:50 +0300
commitc867248cf451964a14c64b4ca232055f117df9c1 (patch)
treea388122f9deb7dc755d79a3e2e8e5ce1f19bc8d0 /security/selinux/status.c
parent0e83c9c6fb0d7d2fddd2cd02575d7ad157e12837 (diff)
downloadlinux-c867248cf451964a14c64b4ca232055f117df9c1.tar.xz
selinux: avoid implicit conversions regarding enforcing status
Use the type bool as parameter type in selinux_status_update_setenforce(). The related function enforcing_enabled() returns the type bool, while the struct selinux_kernel_status member enforcing uses an u32. Signed-off-by: Christian Göttsche <cgzones@googlemail.com> [PM: subject line tweaks] Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security/selinux/status.c')
-rw-r--r--security/selinux/status.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/security/selinux/status.c b/security/selinux/status.c
index e436e4975adc..dffca22ce6f7 100644
--- a/security/selinux/status.c
+++ b/security/selinux/status.c
@@ -76,7 +76,7 @@ struct page *selinux_kernel_status_page(void)
*
* It updates status of the current enforcing/permissive mode.
*/
-void selinux_status_update_setenforce(int enforcing)
+void selinux_status_update_setenforce(bool enforcing)
{
struct selinux_kernel_status *status;
@@ -87,7 +87,7 @@ void selinux_status_update_setenforce(int enforcing)
status->sequence++;
smp_wmb();
- status->enforcing = enforcing;
+ status->enforcing = enforcing ? 1 : 0;
smp_wmb();
status->sequence++;