[asterisk-commits] russell: trunk r82776 - /trunk/channels/chan_zap.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Sep 18 11:10:49 CDT 2007


Author: russell
Date: Tue Sep 18 11:10:48 2007
New Revision: 82776

URL: http://svn.digium.com/view/asterisk?view=rev&rev=82776
Log:
Don't use ast_channel_lock_both() here, it only exists in one of my branches.
This is theoretically a potential deadlock, but it's the way it was before so
I'm going to leave it this way for now.

Modified:
    trunk/channels/chan_zap.c

Modified: trunk/channels/chan_zap.c
URL: http://svn.digium.com/view/asterisk/trunk/channels/chan_zap.c?view=diff&rev=82776&r1=82775&r2=82776
==============================================================================
--- trunk/channels/chan_zap.c (original)
+++ trunk/channels/chan_zap.c Tue Sep 18 11:10:48 2007
@@ -3474,7 +3474,8 @@
 	if (flags & (AST_BRIDGE_DTMF_CHANNEL_0 | AST_BRIDGE_DTMF_CHANNEL_1))
 		return AST_BRIDGE_FAILED_NOWARN;
 
-	ast_channel_lock_both(c0, c1);
+	ast_channel_lock(c0);
+	ast_channel_lock(c1);
 
 	p0 = c0->tech_pvt;
 	p1 = c1->tech_pvt;
@@ -3640,7 +3641,8 @@
 
 		/* Here's our main loop...  Start by locking things, looking for private parts, 
 		   and then balking if anything is wrong */
-		ast_channel_lock_both(c0, c1);
+		ast_channel_lock(c0);
+		ast_channel_lock(c1);
 
 		p0 = c0->tech_pvt;
 		p1 = c1->tech_pvt;




More information about the asterisk-commits mailing list