The way the logic was presented, it was awkward to read and verify. Invert the
logic using DeMorgan's Law to be more easily able to read and understand.
Signed-off-by: Richard Guy Briggs <rgb(a)redhat.com>
---
security/commoncap.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/security/commoncap.c b/security/commoncap.c
index ffcaff0..eb2da69 100644
--- a/security/commoncap.c
+++ b/security/commoncap.c
@@ -552,10 +552,10 @@ static inline bool nonroot_raised_pE(struct cred *cred, kuid_t
root)
bool ret = false;
if (cap_grew(effective, ambient, cred) &&
- (!cap_full(effective, cred) ||
- !is_eff(root, cred) ||
- !is_real(root, cred) ||
- !root_privileged()))
+ !(cap_full(effective, cred) &&
+ is_eff(root, cred) &&
+ is_real(root, cred) &&
+ root_privileged()))
ret = true;
return ret;
}
--
1.7.1