----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4575/#review15068 -----------------------------------------------------------
Ship it! Ship It! - Joshua Colp On April 2, 2015, 7:32 p.m., Kevin Harwell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4575/ > ----------------------------------------------------------- > > (Updated April 2, 2015, 7:32 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-24782 > https://issues.asterisk.org/jira/browse/ASTERISK-24782 > > > Repository: Asterisk > > > Description > ------- > > After completing an attended transfer the transfer target channel (the one > that gets swapped out) was not being hung up after leaving the bridge. This > resulted in a channel possibly being left around. Added an explicit > softhangup for the channel in question after the transfer is successfully > completed in order to make sure the channel is hung up. > > > Diffs > ----- > > branches/13/main/bridge.c 433917 > > Diff: https://reviewboard.asterisk.org/r/4575/diff/ > > > Testing > ------- > > There was a testsuite test that initially caught the problem. Ran the > modified test (had been set to always pass) after applying the patch and it > worked as expected. Also did some manual testing with different attended > transfer scenarios. > > > 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
