diff options
author | Christoph Paasch <christoph.paasch@uclouvain.be> | 2013-12-01 23:05:20 (GMT) |
---|---|---|
committer | Mimi Zohar <zohar@linux.vnet.ibm.com> | 2013-12-03 01:46:32 (GMT) |
commit | 09ae6345721afbb7cf3e0920209b140cbe7bff0d (patch) | |
tree | 36196a39f1b68dc76f4d4b1103d67ee118b999fd /security/apparmor/path.c | |
parent | a45299e72737c528975546a0680cace5d7364d27 (diff) | |
download | linux-09ae6345721afbb7cf3e0920209b140cbe7bff0d.tar.xz |
ima: Do not free 'entry' before it is initialized
7bc5f447ce9d0 (ima: define new function ima_alloc_init_template() to
API) moved the initialization of 'entry' in ima_add_boot_aggregate() a
bit more below, after the if (ima_used_chip).
So, 'entry' is not initialized while being inside this if-block. So, we
should not attempt to free it.
Found by Coverity (CID: 1131971)
Fixes: 7bc5f447ce9d0 (ima: define new function ima_alloc_init_template() to API)
Signed-off-by: Christoph Paasch <christoph.paasch@uclouvain.be>
Signed-off-by: Mimi Zohar <zohar@us.ibm.com>
Diffstat (limited to 'security/apparmor/path.c')
0 files changed, 0 insertions, 0 deletions