diff options
| author |   John Johansen <john.johansen@canonical.com> | 2022-01-25 00:37:42 -0800 |
|---|---|---|
| committer | John Johansen <john.johansen@canonical.com> | 2022-07-09 15:13:59 -0700 |
| commit | ec240b5905bbb09a03dccffee03062cf39e38dc2 (patch) | |
| tree | 8c7d00f2aae7a858eddc862a10b860eb332f01cf /security | |
| parent | security/apparmor: remove redundant ret variable (diff) | |
| download | linux-dev-ec240b5905bbb09a03dccffee03062cf39e38dc2.tar.xz linux-dev-ec240b5905bbb09a03dccffee03062cf39e38dc2.zip | |
apparmor: Fix failed mount permission check error message
When the mount check fails due to a permission check failure instead
of explicitly at one of the subcomponent checks, AppArmor is reporting
a failure in the flags match. However this is not true and AppArmor
can not attribute the error at this point to any particular component,
and should only indicate the mount failed due to missing permissions.
Fixes: 2ea3ffb7782a ("apparmor: add mount mediation")
Signed-off-by: John Johansen <john.johansen@canonical.com>
Diffstat (limited to 'security')
| -rw-r--r-- | security/apparmor/mount.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/security/apparmor/mount.c b/security/apparmor/mount.c index aa6fcfde3051..23aafe68d49a 100644 --- a/security/apparmor/mount.c +++ b/security/apparmor/mount.c @@ -229,7 +229,8 @@ static const char * const mnt_info_table[] = { "failed srcname match", "failed type match", "failed flags match", - "failed data match" + "failed data match", + "failed perms check" }; /* @@ -284,8 +285,8 @@ static int do_match_mnt(struct aa_dfa *dfa, unsigned int start, return 0; } - /* failed at end of flags match */ - return 4; + /* failed at perms check, don't confuse with flags match */ + return 6; } |