[asterisk-commits] russell: branch russell/chan_refcount r89869 - /team/russell/chan_refcount/apps/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Nov 27 17:26:13 CST 2007


Author: russell
Date: Tue Nov 27 17:26:13 2007
New Revision: 89869

URL: http://svn.digium.com/view/asterisk?view=rev&rev=89869
Log:
Remove some locking that is no longer needed

Modified:
    team/russell/chan_refcount/apps/app_minivm.c

Modified: team/russell/chan_refcount/apps/app_minivm.c
URL: http://svn.digium.com/view/asterisk/team/russell/chan_refcount/apps/app_minivm.c?view=diff&rev=89869&r1=89868&r2=89869
==============================================================================
--- team/russell/chan_refcount/apps/app_minivm.c (original)
+++ team/russell/chan_refcount/apps/app_minivm.c Tue Nov 27 17:26:13 2007
@@ -737,8 +737,6 @@
 		return;
 	}
 
-	ast_channel_lock(channel);
-
 	for (var = vmu->chanvars ; var ; var = var->next)
                 pbx_builtin_setvar_helper(channel, var->name, var->value);
 
@@ -753,8 +751,6 @@
 	pbx_builtin_setvar_helper(channel, "MVM_DATE", date);
 	if (!ast_strlen_zero(counter))
 		pbx_builtin_setvar_helper(channel, "MVM_COUNTER", counter);
-
-	ast_channel_unlock(channel);
 }
 
 /*! \brief Set default values for Mini-Voicemail users */
@@ -1047,9 +1043,7 @@
 		char *passdata;
 		int vmlen = strlen(template->subject) * 3 + 200;
 		if ((passdata = alloca(vmlen))) {
-			ast_channel_lock(ast);
 			pbx_substitute_variables_helper(ast, template->subject, passdata, vmlen);
-			ast_channel_unlock(ast);
 			fprintf(p, "Subject: %s\n", passdata);
 		} else {
 			ast_log(LOG_WARNING, "Cannot allocate workspace for variable substitution\n");




More information about the asterisk-commits mailing list