diff options
author | Xiaotian Feng <dfeng@redhat.com> | 2010-02-09 08:22:24 +1100 |
---|---|---|
committer | James Morris <jmorris@namei.org> | 2010-02-09 08:22:24 +1100 |
commit | 8007f10259d04f37044c2c731bf9ccdd9161d825 (patch) | |
tree | 6accff6b70b4780bc62824c419582f4cace56f23 /security/tomoyo | |
parent | ea13ddbad0eb4be9cdc406cd7e0804fa4011f6e4 (diff) | |
download | op-kernel-dev-8007f10259d04f37044c2c731bf9ccdd9161d825.zip op-kernel-dev-8007f10259d04f37044c2c731bf9ccdd9161d825.tar.gz |
selinux: fix memory leak in sel_make_bools
In sel_make_bools, kernel allocates memory for bool_pending_names[i]
with security_get_bools. So if we just free bool_pending_names, those
memories for bool_pending_names[i] will be leaked.
This patch resolves dozens of following kmemleak report after resuming
from suspend:
unreferenced object 0xffff88022e4c7380 (size 32):
comm "init", pid 1, jiffies 4294677173
backtrace:
[<ffffffff810f76b5>] create_object+0x1a2/0x2a9
[<ffffffff810f78bb>] kmemleak_alloc+0x26/0x4b
[<ffffffff810ef3eb>] __kmalloc+0x18f/0x1b8
[<ffffffff811cd511>] security_get_bools+0xd7/0x16f
[<ffffffff811c48c0>] sel_write_load+0x12e/0x62b
[<ffffffff810f9a39>] vfs_write+0xae/0x10b
[<ffffffff810f9b56>] sys_write+0x4a/0x6e
[<ffffffff81011b82>] system_call_fastpath+0x16/0x1b
[<ffffffffffffffff>] 0xffffffffffffffff
Signed-off-by: Xiaotian Feng <dfeng@redhat.com>
Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security/tomoyo')
0 files changed, 0 insertions, 0 deletions