From 456701fc7c7488dd0033923c3a38a2f272eb036d Mon Sep 17 00:00:00 2001
From: file <file@f38db490-d61c-443f-a65b-d21fe96a405b>
Date: Tue, 11 Apr 2006 03:50:17 +0000
Subject: Data stores do not need a lock. As well change the way they are
 removed from the channel when it is destroyed (thanks Russell Wussell) and
 finally... because C++ is silly... change our list macro info thing to be
 "entry" instead of "list".

git-svn-id: http://svn.digium.com/svn/asterisk/trunk@19062 f38db490-d61c-443f-a65b-d21fe96a405b
---
 channel.c | 26 +++++++-------------------
 1 file changed, 7 insertions(+), 19 deletions(-)

(limited to 'channel.c')

diff --git a/channel.c b/channel.c
index a0f15f2dc..2c7804559 100644
--- a/channel.c
+++ b/channel.c
@@ -984,16 +984,10 @@ void ast_channel_free(struct ast_channel *chan)
 	}
 	
 	/* Get rid of each of the data stores on the channel */
-	AST_LIST_LOCK(&chan->datastores);
-	AST_LIST_TRAVERSE_SAFE_BEGIN(&chan->datastores, datastore, list) {
-		/* Remove from the list */
-		AST_LIST_REMOVE_CURRENT(&chan->datastores, list);
+	while ((datastore = AST_LIST_REMOVE_HEAD(&chan->datastores, entry)))
 		/* Free the data store */
 		ast_channel_datastore_free(datastore);
-	}
-	AST_LIST_TRAVERSE_SAFE_END
-	AST_LIST_UNLOCK(&chan->datastores);
-	AST_LIST_HEAD_DESTROY(&chan->datastores);
+	AST_LIST_HEAD_INIT_NOLOCK(&chan->datastores);
 
 	/* loop over the variables list, freeing all data and deleting list items */
 	/* no need to lock the list, as the channel is already locked */
@@ -1059,9 +1053,7 @@ int ast_channel_datastore_add(struct ast_channel *chan, struct ast_datastore *da
 {
 	int res = 0;
 
-	AST_LIST_LOCK(&chan->datastores);
-	AST_LIST_INSERT_HEAD(&chan->datastores, datastore, list);
-	AST_LIST_UNLOCK(&chan->datastores);
+	AST_LIST_INSERT_HEAD(&chan->datastores, datastore, entry);
 
 	return res;
 }
@@ -1072,16 +1064,14 @@ int ast_channel_datastore_remove(struct ast_channel *chan, struct ast_datastore
 	int res = -1;
 
 	/* Find our position and remove ourselves */
-	AST_LIST_LOCK(&chan->datastores);
-	AST_LIST_TRAVERSE_SAFE_BEGIN(&chan->datastores, datastore2, list) {
+	AST_LIST_TRAVERSE_SAFE_BEGIN(&chan->datastores, datastore2, entry) {
 		if (datastore2 == datastore) {
-			AST_LIST_REMOVE_CURRENT(&chan->datastores, list);
+			AST_LIST_REMOVE_CURRENT(&chan->datastores, entry);
 			res = 0;
 			break;
 		}
 	}
 	AST_LIST_TRAVERSE_SAFE_END
-	AST_LIST_UNLOCK(&chan->datastores);
 
 	return res;
 }
@@ -1093,8 +1083,7 @@ struct ast_datastore *ast_channel_datastore_find(struct ast_channel *chan, const
 	if (info == NULL)
 		return NULL;
 
-	AST_LIST_LOCK(&chan->datastores);
-	AST_LIST_TRAVERSE_SAFE_BEGIN(&chan->datastores, datastore, list) {
+	AST_LIST_TRAVERSE_SAFE_BEGIN(&chan->datastores, datastore, entry) {
 		if (datastore->info == info) {
 			if (uid != NULL && datastore->uid != NULL) {
 				if (!strcasecmp(uid, datastore->uid)) {
@@ -1108,7 +1097,6 @@ struct ast_datastore *ast_channel_datastore_find(struct ast_channel *chan, const
 		}
 	}
 	AST_LIST_TRAVERSE_SAFE_END
-	AST_LIST_UNLOCK(&chan->datastores);
 
 	return datastore;
 }
@@ -3169,7 +3157,7 @@ int ast_do_masquerade(struct ast_channel *original)
 	}
 	/* Move data stores over */
 	if (AST_LIST_FIRST(&clone->datastores))
-                AST_LIST_INSERT_TAIL(&original->datastores, AST_LIST_FIRST(&clone->datastores), list);
+                AST_LIST_INSERT_TAIL(&original->datastores, AST_LIST_FIRST(&clone->datastores), entry);
 	AST_LIST_HEAD_INIT_NOLOCK(&clone->datastores);
 
 	clone_variables(original, clone);
-- 
cgit v1.2.3