----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4001/#review13350 -----------------------------------------------------------
Ship it! Ship It! - Matt Jordan On Sept. 18, 2014, 4:19 p.m., opticron wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4001/ > ----------------------------------------------------------- > > (Updated Sept. 18, 2014, 4:19 p.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > This changes gives framehooks a reverse-direction masquerade callback in > addition to chan_fixup_cb similar to the callback added to datastores to > handle the same situation. The new callback provides the same parameters as > the fixup callback, but is called on the new channel's framehooks before > moving framehooks from the old channel to the new channel. This gives the > framehooks an oppurtunity to decide whether they should remain on the new > channel or be removed. > > This new callback is used to prevent the PJSIP T.38 framehook from remaining > on a masqueraded channel if the new channel is not also a PJSIP channel. This > was causing a crash when a local channel was masqueraded into a PJSIP channel > and the framehook was executed on the local channel since the channel's tech > private data was not structured as expected. > > > Diffs > ----- > > branches/12/res/res_pjsip_t38.c 423230 > branches/12/main/framehook.c 423230 > branches/12/include/asterisk/framehook.h 423230 > > Diff: https://reviewboard.asterisk.org/r/4001/diff/ > > > Testing > ------- > > This corrected my reproduction of the crash and fixed the crash for the > original reporter. > > > 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