apparmor: fix leak of null profile name if profile allocation fails
[ Upstream commitpull/10/head4633307e5e
] Fixes:d07881d2ed
("apparmor: move new_null_profile to after profile lookup fns()") Reported-by: Seth Arnold <seth.arnold@canonical.com> Signed-off-by: John Johansen <john.johansen@canonical.com> Signed-off-by: Sasha Levin <alexander.levin@verizon.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
parent
86d1d015fe
commit
897088926c
|
@ -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 @@ out:
|
||||||
return profile;
|
return profile;
|
||||||
|
|
||||||
fail:
|
fail:
|
||||||
|
kfree(name);
|
||||||
aa_free_profile(profile);
|
aa_free_profile(profile);
|
||||||
return NULL;
|
return NULL;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue