diff options
| author | Richard Guy Briggs <rgb@redhat.com> | 2019-04-08 12:50:57 -0400 |
|---|---|---|
| committer | Paul Moore <paul@paul-moore.com> | 2019-04-08 18:10:35 -0400 |
| commit | 699c1868a743f530081f429058616a2dd5d8a4b2 (patch) | |
| tree | 0981898759941fbcc0df283993f9446463c95b0a /kernel/auditfilter.c | |
| parent | a1aa08a01f734d48a8f76b12aca9f4e4adae7649 (diff) | |
| download | linux-699c1868a743f530081f429058616a2dd5d8a4b2.tar.bz2 | |
audit: purge unnecessary list_empty calls
The original conditions that led to the use of list_empty() to optimize
list_for_each_entry_rcu() in auditfilter.c and auditsc.c code have been
removed without removing the list_empty() call, but this code example
has been copied several times. Remove the unnecessary list_empty()
calls.
Please see upstream github issue
https://github.com/linux-audit/audit-kernel/issues/112
Signed-off-by: Richard Guy Briggs <rgb@redhat.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'kernel/auditfilter.c')
| -rw-r--r-- | kernel/auditfilter.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/kernel/auditfilter.c b/kernel/auditfilter.c index 63f8b3f26fab..2c3c2f349b23 100644 --- a/kernel/auditfilter.c +++ b/kernel/auditfilter.c @@ -1315,8 +1315,6 @@ int audit_filter(int msgtype, unsigned int listtype) int ret = 1; /* Audit by default */ rcu_read_lock(); - if (list_empty(&audit_filter_list[listtype])) - goto unlock_and_return; list_for_each_entry_rcu(e, &audit_filter_list[listtype], list) { int i, result = 0; |