----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3066/#review10392 -----------------------------------------------------------
branches/12/bridges/bridge_native_rtp.c <https://reviewboard.asterisk.org/r/3066/#comment19798> This doxygen should get a retval section as well (probably my fault). branches/12/bridges/bridge_native_rtp.c <https://reviewboard.asterisk.org/r/3066/#comment19799> This should be reverted back to the native_rtp_bridge_stop() call and do the same locking dance as the _start() call. This was a result of my attempt to perform these actions without locking the bridge. - opticron On Dec. 11, 2013, 5:20 p.m., Kevin Harwell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3066/ > ----------------------------------------------------------- > > (Updated Dec. 11, 2013, 5:20 p.m.) > > > Review request for Asterisk Developers and kmoore. > > > Bugs: ASTERISK-22749 > https://issues.asterisk.org/jira/browse/ASTERISK-22749 > > > Repository: Asterisk > > > Description > ------- > > This contains the patch on the issue (submitted by kmoore), as well as the > fix for the adding in a bridge lock while calling bridge_start from the > framehook callback. Since the framehook callback is not called from the > bridging core the bridge is not locked, but needs to be before calling > bridge_start. The addition to the given patch adds in the necessary bridge > locking in order to avoid a deadlock. > > > Diffs > ----- > > branches/12/main/channel.c 403687 > branches/12/include/asterisk/channel.h 403687 > branches/12/channels/chan_sip.c 403687 > branches/12/channels/chan_pjsip.c 403687 > branches/12/bridges/bridge_native_rtp.c 403687 > > Diff: https://reviewboard.asterisk.org/r/3066/diff/ > > > Testing > ------- > > Added channels via DTMF attended transfer to get to a 4-way bridge and then > removed them and noted all channels and the bridge had been torn down. > > > Thanks, > > Kevin Harwell > >
-- _____________________________________________________________________ -- Bandwidth and Colocation Provided by http://www.api-digital.com -- asterisk-dev mailing list To UNSUBSCRIBE or update options visit: http://lists.digium.com/mailman/listinfo/asterisk-dev
