diff options
| author | Yaowei Bai <bywxiaobai@163.com> | 2015-11-04 08:23:51 -0500 |
|---|---|---|
| committer | Paul Moore <pmoore@redhat.com> | 2015-11-04 08:23:51 -0500 |
| commit | 9fcf836b215ca5685030ecab3e35ecc14ee3bcfb (patch) | |
| tree | 7f4724ee44e7ed835ff5459324492aa9b025186a /kernel/audit.c | |
| parent | 36734810488e618d48cc14782f7111b3dfaffb83 (diff) | |
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.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/kernel/audit.c b/kernel/audit.c index c4c98d87456f..648036f7690d 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; } /** |