[Asterisk-code-review] bridge: Deny full Local channel pair in bridge. (asterisk[master])

Joshua Colp asteriskteam at digium.com
Wed Nov 17 12:36:20 CST 2021


Attention is currently required from: Joshua Colp.
Hello Kevin Harwell, Friendly Automation, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/c/asterisk/+/17474

to look at the new patch set (#2).

Change subject: bridge: Deny full Local channel pair in bridge.
......................................................................

bridge: Deny full Local channel pair in bridge.

Local channels are made up of two pairs - the 1 and 2
sides. When a frame goes in one side, it comes out the
other. Back and forth. When both halves are in a
bridge this creates an infinite loop of frames.

This change makes it so that bridging no longer
allows both of these sides to exist in the same
bridge.

ASTERISK-29748

Change-Id: I29928b6de87cd9be996a77daccefd7c360fef651
---
M main/bridge_channel.c
1 file changed, 25 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/74/17474/2
-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/17474
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I29928b6de87cd9be996a77daccefd7c360fef651
Gerrit-Change-Number: 17474
Gerrit-PatchSet: 2
Gerrit-Owner: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Attention: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211117/4757feaf/attachment.html>


More information about the asterisk-code-review mailing list