[asterisk-dev] [Code Review] 3585: bridge_native_rtp: Reconfigure bridge on removal of framehook - take 2

Matt Jordan reviewboard at asterisk.org
Sun Jun 8 13:11:59 CDT 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3585/
-----------------------------------------------------------

(Updated June 8, 2014, 1:11 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 415443


Repository: Asterisk


Description
-------

This patch is a re-do of review 3535.

When review request 3535 was merged, a major problem with it was uncovered. UNBRIDGE soft hangup flags have a catastrophic effect on the pbx core if they leak out from the bridge layer: the channel gets hung up. With the number of threads involved in a blind transfer, and with the initial patch, it was likely that this would occur. This caused a large number of test failures

This patch is nearly identical with the one proposed in review 3535, save for the following changes:
 - We explicitly clear the UNBRIDGE flag when setting an after goto on a channel in a bridge
 - Defensively, if we encounter an UNBRIDGE flag in the pbx core, we handle it


Diffs
-----

  /branches/12/main/pbx.c 415300 
  /branches/12/main/framehook.c 415300 
  /branches/12/main/channel.c 415300 
  /branches/12/main/bridge_channel.c 415300 
  /branches/12/main/bridge_after.c 415300 
  /branches/12/include/asterisk/channel.h 415300 
  /branches/12/bridges/bridge_native_rtp.c 415300 

Diff: https://reviewboard.asterisk.org/r/3585/diff/


Testing
-------

The blind transfer tests for PJSIP now pass (fingers crossed). See review 3586.


Thanks,

Matt Jordan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140608/5540339f/attachment.html>


More information about the asterisk-dev mailing list