commit 27a9bff3db294b0de9ec5fe2881c2ed9b49efc64
author Andy Hung <hunga@google.com> Tue Sep 08 17:31:36 2015 -0700
committer The Android Automerger <android-build@google.com> Mon Sep 28 17:08:25 2015 -0700
tree 86978083165cd70e604b1adf5a42dd527e513845
parent 9fb88063e241ad5fa1d62a48c90fa6fe0e250f4c

DO NOT MERGE - volume_listener: fix release

use safe iteration of list when removing node.

Bug: 23731264
Change-Id: I6d0c51638b6a3de684cf595833dc4d0fc7e8c8b5
(cherry picked from commit 591ef60fb95d863cb8f3a3923d59bc36c8cd4b8c)

post_proc/volume_listener.c

diff --git a/post_proc/volume_listener.c b/post_proc/volume_listener.c
index e0193dd..cf15f1c 100644
--- a/post_proc/volume_listener.c
+++ b/post_proc/volume_listener.c
@@ -664,7 +664,7 @@
 
 static int vol_prc_lib_release(effect_handle_t handle)
 {
-    struct listnode *node = NULL;
+    struct listnode *node, *temp_node_next;
     vol_listener_context_t *context = NULL;
     vol_listener_context_t *recv_contex = (vol_listener_context_t *)handle;
     int status = -1;
@@ -674,7 +674,7 @@
     pthread_mutex_lock(&vol_listner_init_lock);
 
     // check if the handle/context provided is valid
-    list_for_each(node, &vol_effect_list) {
+    list_for_each_safe(node, temp_node_next, &vol_effect_list) {
         context = node_to_item(node, struct vol_listener_context_s, effect_list_node);
         if ((memcmp(&(context->desc->uuid), &(recv_contex->desc->uuid), sizeof(effect_uuid_t)) == 0)
             && (context->session_id == recv_contex->session_id)