----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4001/ -----------------------------------------------------------
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