summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2017-10-14 13:14:38 +0100
committerJohn Johansen <john.johansen@canonical.com>2017-11-21 02:17:13 -0800
commite3bcfc148588e409685479f3d20ba3d66ae30035 (patch)
treec0ad64c052051ff1e99dcfd7c1eb0ff90190c0fe
parent954317fef2ee789af55f82903dbc574905250f7c (diff)
downloadlinux-stable-e3bcfc148588e409685479f3d20ba3d66ae30035.tar.gz
linux-stable-e3bcfc148588e409685479f3d20ba3d66ae30035.tar.bz2
linux-stable-e3bcfc148588e409685479f3d20ba3d66ae30035.zip
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 <colin.king@canonical.com> Signed-off-by: John Johansen <john.johansen@canonical.com>
-rw-r--r--security/apparmor/lib.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/security/apparmor/lib.c b/security/apparmor/lib.c
index 3d0a2bf87abd..4d5e98e49d5e 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;