----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3107/#review10580 -----------------------------------------------------------
/branches/11/channels/chan_sip.c <https://reviewboard.asterisk.org/r/3107/#comment20042> Should the sip_park() call hangup the channels as well if the masquerades or creating the sip_park_thread() fail? /branches/11/channels/chan_sip.c <https://reviewboard.asterisk.org/r/3107/#comment20041> You might want to delay setting transferer_pvt to after the call to ast_park_call_exten(). That call can take awhile to return because it plays parking messages. - rmudgett On Jan. 8, 2014, 5:41 p.m., Matt Jordan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3107/ > ----------------------------------------------------------- > > (Updated Jan. 8, 2014, 5:41 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-22834 and ASTERISK-23047 > https://issues.asterisk.org/jira/browse/ASTERISK-22834 > https://issues.asterisk.org/jira/browse/ASTERISK-23047 > > > Repository: Asterisk > > > Description > ------- > > When performing a SIP transfer to a Park extension, if the Park fails, > chan_sip will currently not hang up either the transferer or the transfer > target. This results in the channels being orphaned with no thread to service > frames, resulting in stuck channels. > > This patch immediately hangs up the two channels if a Park fails. > > Note that this is also a bug in 1.8; the code is identical between 11 and 1.8 > in this respect. This is not a bug in 12+. > > > Diffs > ----- > > /branches/11/channels/chan_sip.c 404856 > > Diff: https://reviewboard.asterisk.org/r/3107/diff/ > > > Testing > ------- > > Prior to the patch, the channels were stuck. > > With the patch, the two channels are properly hung up. > > > Thanks, > > Matt Jordan > >
-- _____________________________________________________________________ -- 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