diff options
author | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-24 15:32:12 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-24 15:32:12 -0400 |
commit | 6e8a43c8432649686921b57255db8172edcebf6f (patch) | |
tree | 57867bdcb71b5bd0339f822ddb8032974a37a822 | |
parent | b2e961eb2e7a54ffaae82f8e0198b26b54ade98e (diff) | |
parent | 0ec8abd7086ee4f760cb1b477fe376805b17558c (diff) |
Merge branch 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/selinux-2.6
* 'for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/selinux-2.6:
SELinux: fix memory leak in security_netlbl_cache_add()
-rw-r--r-- | security/selinux/ss/services.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/security/selinux/ss/services.c b/security/selinux/ss/services.c index b5f017f07a75..0ae032f3876b 100644 --- a/security/selinux/ss/services.c +++ b/security/selinux/ss/services.c | |||
@@ -2417,8 +2417,10 @@ static void security_netlbl_cache_add(struct netlbl_lsm_secattr *secattr, | |||
2417 | 2417 | ||
2418 | cache->type = NETLBL_CACHE_T_MLS; | 2418 | cache->type = NETLBL_CACHE_T_MLS; |
2419 | if (ebitmap_cpy(&cache->data.mls_label.level[0].cat, | 2419 | if (ebitmap_cpy(&cache->data.mls_label.level[0].cat, |
2420 | &ctx->range.level[0].cat) != 0) | 2420 | &ctx->range.level[0].cat) != 0) { |
2421 | kfree(cache); | ||
2421 | return; | 2422 | return; |
2423 | } | ||
2422 | cache->data.mls_label.level[1].cat.highbit = | 2424 | cache->data.mls_label.level[1].cat.highbit = |
2423 | cache->data.mls_label.level[0].cat.highbit; | 2425 | cache->data.mls_label.level[0].cat.highbit; |
2424 | cache->data.mls_label.level[1].cat.node = | 2426 | cache->data.mls_label.level[1].cat.node = |