diff options
| author | Colin Ian King <colin.king@canonical.com> | 2017-10-14 13:14:38 +0100 |
|---|---|---|
| committer | John Johansen <john.johansen@canonical.com> | 2017-11-21 02:17:13 -0800 |
| commit | e3bcfc148588e409685479f3d20ba3d66ae30035 (patch) | |
| tree | c0ad64c052051ff1e99dcfd7c1eb0ff90190c0fe /security/apparmor/lib.c | |
| parent | 954317fef2ee789af55f82903dbc574905250f7c (diff) | |
| download | lwn-e3bcfc148588e409685479f3d20ba3d66ae30035.tar.gz lwn-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>
Diffstat (limited to 'security/apparmor/lib.c')
| -rw-r--r-- | security/apparmor/lib.c | 3 |
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; |