Commit 4633307e authored by John Johansen's avatar John Johansen

apparmor: fix leak of null profile name if profile allocation fails

Fixes: d07881d2 ("apparmor: move new_null_profile to after profile lookup fns()")
Reported-by: default avatarSeth Arnold <seth.arnold@canonical.com>
Signed-off-by: default avatarJohn Johansen <john.johansen@canonical.com>
parent e3bcfc14
...@@ -502,7 +502,7 @@ struct aa_profile *aa_new_null_profile(struct aa_profile *parent, bool hat, ...@@ -502,7 +502,7 @@ struct aa_profile *aa_new_null_profile(struct aa_profile *parent, bool hat,
{ {
struct aa_profile *p, *profile; struct aa_profile *p, *profile;
const char *bname; const char *bname;
char *name; char *name = NULL;
AA_BUG(!parent); AA_BUG(!parent);
...@@ -562,6 +562,7 @@ struct aa_profile *aa_new_null_profile(struct aa_profile *parent, bool hat, ...@@ -562,6 +562,7 @@ struct aa_profile *aa_new_null_profile(struct aa_profile *parent, bool hat,
return profile; return profile;
fail: fail:
kfree(name);
aa_free_profile(profile); aa_free_profile(profile);
return NULL; return NULL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment