From c0eef39a83284953abd0cdee79ed07f3c3d921b1 Mon Sep 17 00:00:00 2001 From: Colin Ian King Date: Sat, 14 Oct 2017 13:14:38 +0100 Subject: [PATCH] apparmor: remove unused redundant variable stop The boolean variable 'stop' is being set but never read. This is a redundant variable and can be removed. Cleans up clang warning: Value stored to 'stop' is never read Signed-off-by: Colin Ian King Signed-off-by: John Johansen --- security/apparmor/lib.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/security/apparmor/lib.c b/security/apparmor/lib.c index 3d0a2bf87abdf..4d5e98e49d5e0 100644 --- a/security/apparmor/lib.c +++ b/security/apparmor/lib.c @@ -423,7 +423,6 @@ int aa_check_perms(struct aa_profile *profile, struct aa_perms *perms, void (*cb)(struct audit_buffer *, void *)) { int type, error; - bool stop = false; u32 denied = request & (~perms->allow | perms->deny); if (likely(!denied)) { @@ -444,8 +443,6 @@ int aa_check_perms(struct aa_profile *profile, struct aa_perms *perms, else type = AUDIT_APPARMOR_DENIED; - if (denied & perms->stop) - stop = true; if (denied == (denied & perms->hide)) error = -ENOENT; -- 2.39.5