diff options
| author | Paul Moore <paul@paul-moore.com> | 2019-05-30 12:53:42 -0400 |
|---|---|---|
| committer | Paul Moore <paul@paul-moore.com> | 2019-05-30 12:53:42 -0400 |
| commit | 839d05e413856bd686a33b59294d4e8238169320 (patch) | |
| tree | dbf4bd3494b443ac7e6907a4aaa503e8d6a7e284 /kernel/auditfilter.c | |
| parent | 0223fad3c98a9588c159a35dda2ef6e68ca27e3f (diff) | |
| download | linux-839d05e413856bd686a33b59294d4e8238169320.tar.bz2 | |
audit: remove the BUG() calls in the audit rule comparison functions
The audit_data_to_entry() function ensures that the operator is valid
so we can get rid of these BUG() calls. We keep the "return 0" just
so the system behaves in a sane-ish manner should something go
horribly wrong.
Signed-off-by: Paul Moore <paul@paul-moore.com>
Acked-by: Richard Guy Briggs <rgb@redhat.com>
Diffstat (limited to 'kernel/auditfilter.c')
| -rw-r--r-- | kernel/auditfilter.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/kernel/auditfilter.c b/kernel/auditfilter.c index e69d136eeaf6..1a21b6aa50d1 100644 --- a/kernel/auditfilter.c +++ b/kernel/auditfilter.c @@ -1220,7 +1220,6 @@ int audit_comparator(u32 left, u32 op, u32 right) case Audit_bittest: return ((left & right) == right); default: - BUG(); return 0; } } @@ -1243,7 +1242,6 @@ int audit_uid_comparator(kuid_t left, u32 op, kuid_t right) case Audit_bitmask: case Audit_bittest: default: - BUG(); return 0; } } @@ -1266,7 +1264,6 @@ int audit_gid_comparator(kgid_t left, u32 op, kgid_t right) case Audit_bitmask: case Audit_bittest: default: - BUG(); return 0; } } |