----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4265/#review13967 -----------------------------------------------------------
Ship it! branches/13/res/res_pjsip_sdp_rtp.c <https://reviewboard.asterisk.org/r/4265/#comment24486> Might as well drop this since it's duplicated just below your addition. - opticron On Dec. 15, 2014, 11:59 a.m., Kevin Harwell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4265/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2014, 11:59 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-24459 > https://issues.asterisk.org/jira/browse/ASTERISK-24459 > > > Repository: Asterisk > > > Description > ------- > > A native rtp bridge was being chosen (it shouldn't have been) when using two > pjsip channels with incompatible DTMF modes. This patch sets the rtp > instance property, AST_RTP_PROPERTY_DTMF, for the appropriate DTMF mode(s) > for pjsip. It was not being set before, meaning all DTMF modes for pjsip > were being treated as compatible, thus native bridging would be chosen as the > bridge type when it shouldn't have been. > > > Diffs > ----- > > branches/13/res/res_pjsip_sdp_rtp.c 429572 > > Diff: https://reviewboard.asterisk.org/r/4265/diff/ > > > Testing > ------- > > Using pjsip, set up two endpoints with incompatible DTMF modes. After > establishing a call between noted, using the CLI, that that a native_rtp > bridge was being chosen. After the patch noted the channels were now in a > simple bridge. > > > 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
