----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3736/#review12539 -----------------------------------------------------------
team/group/media_formats-reviewed-trunk/main/channel.c <https://reviewboard.asterisk.org/r/3736/#comment22834> I'd move this further down until after the translation path checks. If we can't set up a translation patch, we probably don't want to change the formats on the channel. This should probably go just before the unlocking of the channel. - Matt Jordan On July 10, 2014, 10:28 a.m., opticron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3736/ > ----------------------------------------------------------- > > (Updated July 10, 2014, 10:28 a.m.) > > > Review request for Asterisk Developers, Corey Farrell, Joshua Colp, and Matt > Jordan. > > > Bugs: ASTERISK-23960 > https://issues.asterisk.org/jira/browse/ASTERISK-23960 > > > Repository: Asterisk > > > Description > ------- > > This contains the fixes necessary to get a PJSIP call up and playing files > from the Playback() application with a translation path in the mix. > > * channel.c: The set format helper functions were not actually setting the > newly chosen formats resulting in no audio. > * frame.c: The format on the frame was being overzealously unreffed resulting > in a crash. > * sorcery.c: The codec retrieval code was using the wrong level of > indirection for ast_format_cap structures resuting in a crash for "pjsip show > endpoint x" > * translate.c: The chosen codecs were being set backward on set vs native for > what was actually desired causing incorrect codecs to be chosen. > * res_pjsip_sdp_rtp.c: An ast_rtp_codecs struct was not being initialized > properly causing a crash. > > > Diffs > ----- > > team/group/media_formats-reviewed-trunk/res/res_pjsip_sdp_rtp.c 418253 > team/group/media_formats-reviewed-trunk/main/translate.c 418253 > team/group/media_formats-reviewed-trunk/main/sorcery.c 418253 > team/group/media_formats-reviewed-trunk/main/frame.c 418253 > team/group/media_formats-reviewed-trunk/main/channel.c 418253 > > Diff: https://reviewboard.asterisk.org/r/3736/diff/ > > > Testing > ------- > > Call testing. > > > Thanks, > > opticron > >
-- _____________________________________________________________________ -- 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
