diff options
author | Jeff Vander Stoep <jeffv@google.com> | 2015-04-20 17:45:42 -0700 |
---|---|---|
committer | Ziyan <jaraidaniel@gmail.com> | 2016-03-11 01:10:44 +0100 |
commit | e27358b95e22e19338ace0f7ef140a8ef74e5e04 (patch) | |
tree | e07a5eb146957d32f8a93322b78613269881e030 /security | |
parent | 65053b82e73f8b4a806955a92aff7ce0ff589ea1 (diff) | |
download | kernel_samsung_tuna-e27358b95e22e19338ace0f7ef140a8ef74e5e04.zip kernel_samsung_tuna-e27358b95e22e19338ace0f7ef140a8ef74e5e04.tar.gz kernel_samsung_tuna-e27358b95e22e19338ace0f7ef140a8ef74e5e04.tar.bz2 |
SELinux: use deletion-safe iterator to free list
This code is not exercised by policy version 26, but will be upon
upgrade to policy version 30.
Bug: 18087110
Change-Id: I07c6f34607713294a6a12c43a64d9936f0602200
Signed-off-by: Jeff Vander Stoep <jeffv@google.com>
Diffstat (limited to 'security')
-rw-r--r-- | security/selinux/avc.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/security/selinux/avc.c b/security/selinux/avc.c index 21597ab..6c1c590 100644 --- a/security/selinux/avc.c +++ b/security/selinux/avc.c @@ -306,13 +306,15 @@ static void avc_operation_decision_free( static void avc_operation_free(struct avc_operation_node *ops_node) { - struct avc_operation_decision_node *od_node; + struct avc_operation_decision_node *od_node, *tmp; if (!ops_node) return; - list_for_each_entry(od_node, &ops_node->od_head, od_list) + list_for_each_entry_safe(od_node, tmp, &ops_node->od_head, od_list) { + list_del(&od_node->od_list); avc_operation_decision_free(od_node); + } kmem_cache_free(avc_operation_node_cachep, ops_node); } |