[Asterisk-code-review] chan sip.c: Suppress T.38 SDP c= line if addr is the same. (asterisk[master])

Richard Mudgett asteriskteam at digium.com
Tue Feb 23 16:40:33 CST 2016


Richard Mudgett has uploaded a new change for review.

  https://gerrit.asterisk.org/2288

Change subject: chan_sip.c: Suppress T.38 SDP c= line if addr is the same.
......................................................................

chan_sip.c: Suppress T.38 SDP c= line if addr is the same.

Use the correct comparison function since we only care if the address
without the port is the same.

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


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/88/2288/1

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index aaf0b6d..e1c391e 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -13447,7 +13447,7 @@
 
 		ast_str_append(&m_modem, 0, "m=image %d udptl t38\r\n", ast_sockaddr_port(&udptldest));
 
-		if (ast_sockaddr_cmp(&udptldest, &dest)) {
+		if (ast_sockaddr_cmp_addr(&udptldest, &dest)) {
 			ast_str_append(&m_modem, 0, "c=IN %s %s\r\n",
 					(ast_sockaddr_is_ipv6(&udptldest) && !ast_sockaddr_is_ipv4_mapped(&udptldest)) ?
 					"IP6" : "IP4", ast_sockaddr_stringify_addr_remote(&udptldest));

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ibf6c485f843a1be6dee58a47b33d81a7a8cbe3b0
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list