diff options
| author | Christian Göttsche <cgzones@googlemail.com> | 2023-07-06 16:23:34 +0300 |
|---|---|---|
| committer | Paul Moore <paul@paul-moore.com> | 2023-07-19 01:29:50 +0300 |
| commit | c867248cf451964a14c64b4ca232055f117df9c1 (patch) | |
| tree | a388122f9deb7dc755d79a3e2e8e5ce1f19bc8d0 /security/selinux/selinuxfs.c | |
| parent | 0e83c9c6fb0d7d2fddd2cd02575d7ad157e12837 (diff) | |
| download | linux-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/selinuxfs.c')
| -rw-r--r-- | security/selinux/selinuxfs.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/security/selinux/selinuxfs.c b/security/selinux/selinuxfs.c index bad1f6b685fd..f79e96f0f221 100644 --- a/security/selinux/selinuxfs.c +++ b/security/selinux/selinuxfs.c @@ -138,7 +138,8 @@ static ssize_t sel_write_enforce(struct file *file, const char __user *buf, { char *page = NULL; ssize_t length; - int old_value, new_value; + int scan_value; + bool old_value, new_value; if (count >= PAGE_SIZE) return -ENOMEM; @@ -152,10 +153,10 @@ static ssize_t sel_write_enforce(struct file *file, const char __user *buf, return PTR_ERR(page); length = -EINVAL; - if (sscanf(page, "%d", &new_value) != 1) + if (sscanf(page, "%d", &scan_value) != 1) goto out; - new_value = !!new_value; + new_value = !!scan_value; old_value = enforcing_enabled(); if (new_value != old_value) { |