[Asterisk-code-review] ASTERISK-28057 chan sip: SipNotify on Chan Sip vi AMI behave... (asterisk[13])

[Peter Katzmann]

Peter Katzmann has uploaded this change for review. ( https://gerrit.asterisk.org/10176


Change subject: ASTERISK-28057 chan_sip: SipNotify on Chan_Sip vi AMI behave different to CLI
......................................................................

ASTERISK-28057 chan_sip: SipNotify on Chan_Sip vi AMI behave different to CLI

With tls and udp enabled asterisk generates a warning about sending
message via udp instead of tls.
sip notify command via cli works as expected and without warning.

asterisk has to set the connection information accordingly to connection
and not on presumption

Change-Id: Ib43315aba1f2c14ba077b52d8c5b00be0006656e
---
M channels/chan_sip.c
1 file changed, 1 insertion(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/76/10176/1

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index ebfcfeb..34299a8 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -15572,7 +15572,7 @@
 			return 0;
 		}
 
-		if (create_addr(p, channame, NULL, 0)) {
+		if (create_addr(p, channame, NULL, 1)) {
 			/* Maybe they're not registered, etc. */
 			dialog_unlink_all(p);
 			dialog_unref(p, "unref dialog inside for loop" );

-- 
To view, visit https://gerrit.asterisk.org/10176
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib43315aba1f2c14ba077b52d8c5b00be0006656e
Gerrit-Change-Number: 10176
Gerrit-PatchSet: 1
Gerrit-Owner: Peter Katzmann <peter.katzmann at edag.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180918/88c3277c/attachment.html>