[Asterisk-code-review] stasis: Only place stasis created and dialed channels into dial bridge. (...asterisk[16])

Joshua Colp asteriskteam at digium.com
Thu May 2 10:43:57 CDT 2019


Joshua Colp has uploaded a new patch set (#2) to the change originally created by Abhay Gupta. ( https://gerrit.asterisk.org/c/asterisk/+/11320 )

Change subject: stasis: Only place stasis created and dialed channels into dial bridge.
......................................................................

stasis: Only place stasis created and dialed channels into dial bridge.

The dial bridge is meant to hold channels which have been created
and dialed in stasis. It handles the frames coming from them and raises
the appropriate events.

It was possible for the code to mistakenly place calls which came
from the dialplan into the dial bridge if they were not in an
answered state. These channels are not outgoing channels and
should not be placed into the dial bridge.

The code now checks to ensure that only stasis created channels are
placed into the dial bridge by checking that a PBX does not exist
on the channel.

ASTERISK-27756

Change-Id: Ideee69ff06c9a0b31f7ed61165f5c055f51d21b6
---
M res/stasis/control.c
1 file changed, 5 insertions(+), 1 deletion(-)


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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: Ideee69ff06c9a0b31f7ed61165f5c055f51d21b6
Gerrit-Change-Number: 11320
Gerrit-PatchSet: 2
Gerrit-Owner: Abhay Gupta <abhay at avissol.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Holger Hans Peter Freyther <automatic at freyther.de>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190502/70539c4e/attachment-0001.html>


More information about the asterisk-code-review mailing list