diff options
author | John Johansen <john.johansen@canonical.com> | 2016-04-16 17:19:38 -0400 |
---|---|---|
committer | John Johansen <john.johansen@canonical.com> | 2016-07-12 11:43:10 -0400 |
commit | 7ee6da25dcce27b6023a8673fdf8be98dcf7cacf (patch) | |
tree | 8b4b76224b6162db3d940fc5669a808e182b7e45 /security/apparmor | |
parent | bf15cf0c641be8e57d45f110a9d91464f5bb461a (diff) |
apparmor: fix audit full profile hname on successful load
Currently logging of a successful profile load only logs the basename
of the profile. This can result in confusion when a child profile has
the same name as the another profile in the set. Logging the hname
will ensure there is no confusion.
Signed-off-by: John Johansen <john.johansen@canonical.com>
Acked-by: Seth Arnold <seth.arnold@canonical.com>
Diffstat (limited to 'security/apparmor')
-rw-r--r-- | security/apparmor/policy.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/security/apparmor/policy.c b/security/apparmor/policy.c index db31bc5e459f..ca402d028db8 100644 --- a/security/apparmor/policy.c +++ b/security/apparmor/policy.c | |||
@@ -1159,7 +1159,7 @@ ssize_t aa_replace_profiles(void *udata, size_t size, bool noreplace) | |||
1159 | list_del_init(&ent->list); | 1159 | list_del_init(&ent->list); |
1160 | op = (!ent->old && !ent->rename) ? OP_PROF_LOAD : OP_PROF_REPL; | 1160 | op = (!ent->old && !ent->rename) ? OP_PROF_LOAD : OP_PROF_REPL; |
1161 | 1161 | ||
1162 | audit_policy(op, GFP_ATOMIC, ent->new->base.name, NULL, error); | 1162 | audit_policy(op, GFP_ATOMIC, ent->new->base.hname, NULL, error); |
1163 | 1163 | ||
1164 | if (ent->old) { | 1164 | if (ent->old) { |
1165 | __replace_profile(ent->old, ent->new, 1); | 1165 | __replace_profile(ent->old, ent->new, 1); |