diff options
author | Zhen Lei <thunder.leizhen@huawei.com> | 2024-08-22 22:08:58 +0800 |
---|---|---|
committer | Paul Moore <paul@paul-moore.com> | 2024-08-28 13:42:11 -0400 |
commit | 68cfb28332420e0515cb6ffdb46921d59ba9739f (patch) | |
tree | 05359c4b18ebbdc7fa17bb7d4006b33ebf1f863a /net/unix/garbage.c | |
parent | a3422eb4facdebb685b9b4688feb60430450e3c9 (diff) |
selinux: simplify avc_xperms_audit_required()
By associative and commutative laws, the result of the two 'audited' is
zero. Take the second 'audited' as an example:
1) audited = requested & avd->auditallow;
2) audited &= ~requested;
==> audited = ~requested & (requested & avd->auditallow);
==> audited = (~requested & requested) & avd->auditallow;
==> audited = 0 & avd->auditallow;
==> audited = 0;
In fact, it is more readable to directly write zero. The value of the
first 'audited' is 0 because AUDIT is not allowed. The second 'audited'
is zero because there is no AUDITALLOW permission.
Signed-off-by: Zhen Lei <thunder.leizhen@huawei.com>
Signed-off-by: Paul Moore <paul@paul-moore.com>
Diffstat (limited to 'net/unix/garbage.c')
0 files changed, 0 insertions, 0 deletions