summaryrefslogtreecommitdiffstats
path: root/kernel/audit.c
diff options
context:
space:
mode:
authorYaowei Bai <bywxiaobai@163.com>2015-11-04 08:23:51 -0500
committerPaul Moore <pmoore@redhat.com>2015-11-04 08:23:51 -0500
commit9fcf836b215ca5685030ecab3e35ecc14ee3bcfb (patch)
tree7f4724ee44e7ed835ff5459324492aa9b025186a /kernel/audit.c
parent36734810488e618d48cc14782f7111b3dfaffb83 (diff)
downloadop-kernel-dev-9fcf836b215ca5685030ecab3e35ecc14ee3bcfb.zip
op-kernel-dev-9fcf836b215ca5685030ecab3e35ecc14ee3bcfb.tar.gz
audit: audit_string_contains_control can be boolean
This patch makes audit_string_contains_control return bool to improve readability due to this particular function only using either one or zero as its return value. Signed-off-by: Yaowei Bai <bywxiaobai@163.com> [PM: tweaked subject line] Signed-off-by: Paul Moore <pmoore@redhat.com>
Diffstat (limited to 'kernel/audit.c')
-rw-r--r--kernel/audit.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/kernel/audit.c b/kernel/audit.c
index c4c98d8..648036f 100644
--- a/kernel/audit.c
+++ b/kernel/audit.c
@@ -1583,14 +1583,14 @@ void audit_log_n_string(struct audit_buffer *ab, const char *string,
* @string: string to be checked
* @len: max length of the string to check
*/
-int audit_string_contains_control(const char *string, size_t len)
+bool audit_string_contains_control(const char *string, size_t len)
{
const unsigned char *p;
for (p = string; p < (const unsigned char *)string + len; p++) {
if (*p == '"' || *p < 0x21 || *p > 0x7e)
- return 1;
+ return true;
}
- return 0;
+ return false;
}
/**
OpenPOWER on IntegriCloud