[Asterisk-code-review] chan sip: Make autocreated peers send PeerStatus events (asterisk[master])

Kirill Katsnelson asteriskteam at digium.com
Fri Apr 22 16:02:39 CDT 2016


Kirill Katsnelson has uploaded a new change for review.

  https://gerrit.asterisk.org/2684

Change subject: chan_sip: Make autocreated peers send PeerStatus events
......................................................................

chan_sip: Make autocreated peers send PeerStatus events

Since Stasis has been introduced, an attempt to send AMI messages by an
autocreated peer caused a crash, and all events from autocreated peers were
semi-inadvertently disabled altogether in 0b83761. This change restores the
disabled functionality.

ASTERISK-25950

Change-Id: Iecc350f23db603fadb2f302064643ebe9664e974
---
M channels/chan_sip.c
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/84/2684/1

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index d927f06..830a524 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -17713,6 +17713,10 @@
 	if (!peer && sip_cfg.autocreatepeer != AUTOPEERS_DISABLED) {
 		/* Create peer if we have autocreate mode enabled */
 		peer = temp_peer(name);
+		if (peer && !(peer->endpoint = ast_endpoint_create("SIP", name))) {
+			ao2_t_ref(peer, -1, "failed to allocate Stasis endpoint, drop peer");
+			peer = NULL;
+		}
 		if (peer) {
 			ao2_t_link(peers, peer, "link peer into peer table");
 			if (!ast_sockaddr_isnull(&peer->addr)) {

-- 
To view, visit https://gerrit.asterisk.org/2684
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iecc350f23db603fadb2f302064643ebe9664e974
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Kirill Katsnelson <kkm at smartaction.com>



More information about the asterisk-code-review mailing list