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

Kirill Katsnelson asteriskteam at digium.com
Mon Apr 25 17:31:51 CDT 2016


Kirill Katsnelson has uploaded a new change for review.

  https://gerrit.asterisk.org/2703

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/03/2703/1

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 45827b4..e886772 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -17669,6 +17669,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/2703
To unsubscribe, visit https://gerrit.asterisk.org/settings

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



More information about the asterisk-code-review mailing list