----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4268/#review13982 -----------------------------------------------------------
Ship it! Ship It! - Joshua Colp On Dec. 16, 2014, 4:24 p.m., Mark Michelson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4268/ > ----------------------------------------------------------- > > (Updated Dec. 16, 2014, 4:24 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-24536 > https://issues.asterisk.org/jira/browse/ASTERISK-24536 > > > Repository: Asterisk > > > Description > ------- > > The reporter of ASTERISK-24536 had an issue where performing a dual redirect > on two channels that were bridged together would result in the second channel > ("extrachannel") being hung up instead of being redirected. I could not > reproduce this issue, and the reporter also states that it no longer happens > for him, but having a look at the code, I did see something that looked fishy > and that could cause the reported issue. > > There is a special channel flag called AST_FLAG_BRIDGE_DUAL_REDIRECT_WAIT > that gets set in order for a bridge not to try to perform any actions on a > channel until the redirection operation has completed. This helps prevent > automatic bridge behaviors from occurring, such as hanging up the last > channel remaining in a basic bridge. The problem with the redirect code was > that it was setting this flag on the channel based on the presence of a PBX > on the channel, not based on whether the channel was in a bridge. This review > request changes the redirect code to set the > AST_FLAG_BRIDGE_DUAL_REDIRECT_WAIT flag on channels that are bridged. > > > Diffs > ----- > > /branches/13/main/manager.c 429611 > > Diff: https://reviewboard.asterisk.org/r/4268/diff/ > > > Testing > ------- > > A dual redirect of two channels in the same bridge continues to work for me > in my development tests. There are formal dual redirect tests coming down the > pipeline. Expect to see those on RB before the end of the year. > > > Thanks, > > Mark Michelson > >
-- _____________________________________________________________________ -- 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