diff options
author | John Johansen <john.johansen@canonical.com> | 2017-11-16 00:25:30 +0100 |
---|---|---|
committer | John Johansen <john.johansen@canonical.com> | 2017-11-21 11:17:13 +0100 |
commit | 4633307e5ed6128975595df43f796a10c41d11c1 (patch) | |
tree | 69a59e0adcd575811ee7b05190111929d89ca173 /security/apparmor | |
parent | apparmor: remove unused redundant variable stop (diff) | |
download | linux-4633307e5ed6128975595df43f796a10c41d11c1.tar.xz linux-4633307e5ed6128975595df43f796a10c41d11c1.zip |
apparmor: fix leak of null profile name if profile allocation fails
Fixes: d07881d2edb0 ("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>
Diffstat (limited to 'security/apparmor')
-rw-r--r-- | security/apparmor/policy.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/security/apparmor/policy.c b/security/apparmor/policy.c index 4243b0c3f0e4..586b249d3b46 100644 --- a/security/apparmor/policy.c +++ b/security/apparmor/policy.c @@ -502,7 +502,7 @@ struct aa_profile *aa_new_null_profile(struct aa_profile *parent, bool hat, { struct aa_profile *p, *profile; const char *bname; - char *name; + char *name = NULL; AA_BUG(!parent); @@ -562,6 +562,7 @@ out: return profile; fail: + kfree(name); aa_free_profile(profile); return NULL; } |