diff options
| author |   Paul Moore <paul@paul-moore.com> | 2022-04-14 16:40:10 -0400 |
|---|---|---|
| committer | Paul Moore <paul@paul-moore.com> | 2022-04-14 16:44:21 -0400 |
| commit | 6a9e261cbbee08c499f2331910027e8c40c8f81f (patch) | |
| tree | 9478231587f3a1750bf1a200c57997381f5e715e /security/selinux/include/security.h | |
| parent | selinux: checkreqprot is deprecated, add some ssleep() discomfort (diff) | |
| download | linux-dev-6a9e261cbbee08c499f2331910027e8c40c8f81f.tar.xz linux-dev-6a9e261cbbee08c499f2331910027e8c40c8f81f.zip | |
selinux: don't sleep when CONFIG_SECURITY_SELINUX_CHECKREQPROT_VALUE is true
Unfortunately commit 81200b0265b1 ("selinux: checkreqprot is
deprecated, add some ssleep() discomfort") added a five second sleep
during early kernel boot, e.g. start_kernel(), which could cause a
"scheduling while atomic" panic. This patch fixes this problem by
moving the sleep out of checkreqprot_set() and into
sel_write_checkreqprot() so that we only sleep when the checkreqprot
setting is set during runtime, after the kernel has booted. The
error message remains the same in both cases.
Fixes: 81200b0265b1 ("selinux: checkreqprot is deprecated, add some ssleep() discomfort")
Reported-by: J. Bruce Fields <bfields@fieldses.org>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'security/selinux/include/security.h')
| -rw-r--r-- | security/selinux/include/security.h | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/security/selinux/include/security.h b/security/selinux/include/security.h index f7e6be63adfb..393aff41d3ef 100644 --- a/security/selinux/include/security.h +++ b/security/selinux/include/security.h @@ -152,10 +152,8 @@ static inline bool checkreqprot_get(const struct selinux_state *state) static inline void checkreqprot_set(struct selinux_state *state, bool value) { - if (value) { + if (value) pr_err("SELinux: https://github.com/SELinuxProject/selinux-kernel/wiki/DEPRECATE-checkreqprot\n"); - ssleep(5); - } WRITE_ONCE(state->checkreqprot, value); } |