diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-04-03 00:48:12 +0200 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-04-03 18:49:59 +0200 |
| commit | b61c37f57988567c84359645f8202a7c84bc798a (patch) | |
| tree | a808c891711d060060a751f4119198dc06e2c847 /security/selinux | |
| parent | SELinux: do not allocate stack space for AVC data unless needed (diff) | |
| download | linux-b61c37f57988567c84359645f8202a7c84bc798a.tar.xz linux-b61c37f57988567c84359645f8202a7c84bc798a.zip | |
lsm_audit: don't specify the audit pre/post callbacks in 'struct common_audit_data'
It just bloats the audit data structure for no good reason, since the
only time those fields are filled are just before calling the
common_lsm_audit() function, which is also the only user of those
fields.
So just make them be the arguments to common_lsm_audit(), rather than
bloating that structure that is passed around everywhere, and is
initialized in hot paths.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'security/selinux')
| -rw-r--r-- | security/selinux/avc.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/security/selinux/avc.c b/security/selinux/avc.c index 36c42bb52d81..8ee42b2a5f19 100644 --- a/security/selinux/avc.c +++ b/security/selinux/avc.c @@ -492,9 +492,7 @@ static noinline int slow_avc_audit(u32 ssid, u32 tsid, u16 tclass, slad.denied = denied; a->selinux_audit_data->slad = &slad; - a->lsm_pre_audit = avc_audit_pre_callback; - a->lsm_post_audit = avc_audit_post_callback; - common_lsm_audit(a); + common_lsm_audit(a, avc_audit_pre_callback, avc_audit_post_callback); return 0; } |