[asterisk-commits] rmudgett: branch group/bridge_construction r388379 - /team/group/bridge_const...

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri May 10 15:50:42 CDT 2013


Author: rmudgett
Date: Fri May 10 15:50:41 2013
New Revision: 388379

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=388379
Log:
Add BUGBUG note about a race condition.

Modified:
    team/group/bridge_construction/main/bridging_basic.c

Modified: team/group/bridge_construction/main/bridging_basic.c
URL: http://svnview.digium.com/svn/asterisk/team/group/bridge_construction/main/bridging_basic.c?view=diff&rev=388379&r1=388378&r2=388379
==============================================================================
--- team/group/bridge_construction/main/bridging_basic.c (original)
+++ team/group/bridge_construction/main/bridging_basic.c Fri May 10 15:50:41 2013
@@ -99,6 +99,7 @@
  */
 static int basic_hangup_hook(struct ast_bridge *bridge, struct ast_bridge_channel *bridge_channel, void *hook_pvt)
 {
+/* BUGBUG Race condition.  If all parties but one hangup at the same time, the bridge may not be dissolved on the remaining party. */
 	ast_bridge_channel_lock_bridge(bridge_channel);
 	if (2 < bridge_channel->bridge->num_channels) {
 		/* Just allow this channel to leave the multi-party bridge. */




More information about the asterisk-commits mailing list