From: Paul Moore <paul@paul-moore.com>
Date: Thu, 30 May 2019 16:53:42 +0000 (-0400)
Subject: audit: remove the BUG() calls in the audit rule comparison functions
X-Git-Tag: v5.15~6042^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=839d05e413856bd686a33b59294d4e8238169320;p=platform%2Fkernel%2Flinux-starfive.git

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>
---

diff --git a/kernel/auditfilter.c b/kernel/auditfilter.c
index e69d136..1a21b6a 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;
 	}
 }