[Asterisk-code-review] Fixed format of IPv6 addresses for external media addresses (asterisk[16.9])

Daniel Heckl asteriskteam at digium.com
Wed Apr 15 15:19:08 CDT 2020


Daniel Heckl has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/14261 )


Change subject: Fixed format of IPv6 addresses for external media addresses
......................................................................

Fixed format of IPv6 addresses for external media addresses

Change-Id: I66289afd164c5cdd6c5caa39e79d629a467e7a26
---
M res/res_pjsip_sdp_rtp.c
M res/res_pjsip_session.c
2 files changed, 4 insertions(+), 4 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/61/14261/1

diff --git a/res/res_pjsip_sdp_rtp.c b/res/res_pjsip_sdp_rtp.c
index fb249a7..5bf5de7 100644
--- a/res/res_pjsip_sdp_rtp.c
+++ b/res/res_pjsip_sdp_rtp.c
@@ -2091,8 +2091,8 @@
 	if (ast_sip_transport_is_nonlocal(transport_state, &our_sdp_addr) && transport_state->localnet) {
 		return;
 	}
-	ast_debug(5, "Setting media address to %s\n", ast_sockaddr_stringify_host(&transport_state->external_media_address));
-	pj_strdup2(tdata->pool, &stream->conn->addr, ast_sockaddr_stringify_host(&transport_state->external_media_address));
+	ast_debug(5, "Setting media address to %s\n", ast_sockaddr_stringify_addr(&transport_state->external_media_address));
+	pj_strdup2(tdata->pool, &stream->conn->addr, ast_sockaddr_stringify_addr(&transport_state->external_media_address));
 }
 
 /*! \brief Function which stops the RTP instance */
diff --git a/res/res_pjsip_session.c b/res/res_pjsip_session.c
index b9448b9..37a9350 100644
--- a/res/res_pjsip_session.c
+++ b/res/res_pjsip_session.c
@@ -4420,8 +4420,8 @@
 		 * outgoing session IP is local. If it is, we'll do
 		 * rewriting. No localnet configured? Always rewrite. */
 		if (ast_sip_transport_is_local(transport_state, &our_sdp_addr) || !transport_state->localnet) {
-			ast_debug(5, "Setting external media address to %s\n", ast_sockaddr_stringify_host(&transport_state->external_media_address));
-			pj_strdup2(tdata->pool, &sdp->conn->addr, ast_sockaddr_stringify_host(&transport_state->external_media_address));
+			ast_debug(5, "Setting external media address to %s\n", ast_sockaddr_stringify_addr(&transport_state->external_media_address));
+			pj_strdup2(tdata->pool, &sdp->conn->addr, ast_sockaddr_stringify_addr(&transport_state->external_media_address));
 			pj_strassign(&sdp->origin.addr, &sdp->conn->addr);
 		}
 	}

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

Gerrit-Project: asterisk
Gerrit-Branch: 16.9
Gerrit-Change-Id: I66289afd164c5cdd6c5caa39e79d629a467e7a26
Gerrit-Change-Number: 14261
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Heckl <daniel.heckl at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200415/41313d02/attachment.html>


More information about the asterisk-code-review mailing list