diff options
| author | Jeff Vander Stoep <jeffv@google.com> | 2015-04-20 17:45:42 -0700 |
|---|---|---|
| committer | Mohammed Habibulla <moch@google.com> | 2015-10-26 10:28:42 -0700 |
| commit | 5507743f08723ff6825c7723830adbaded8eebe3 (patch) | |
| tree | e5dcc2614787cc2f674cff78d2d33b69f1d9e83a | |
| parent | 8c342cec1baa95529a2160d1431b6e12ed852fe1 (diff) | |
| download | pxa-v3.14-5507743f08723ff6825c7723830adbaded8eebe3.tar.gz | |
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>
| -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 76766f4c920..21a34153df1 100644 --- a/security/selinux/avc.c +++ b/security/selinux/avc.c @@ -298,13 +298,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); } |