From 9ab1cc22a4d3d9f100e1b021221b45ec4b8744cd Mon Sep 17 00:00:00 2001 From: Olle Johansson Date: Tue, 31 Oct 2006 10:26:16 +0000 Subject: [PATCH] Support ;rport when we're supposed to support ;rport. Issue #7473. git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@46628 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- channels/chan_sip.c | 6 ++---- configs/sip.conf.sample | 2 +- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/channels/chan_sip.c b/channels/chan_sip.c index b5f15cff49..0b1d7d1e97 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -5265,7 +5265,8 @@ static int copy_via_headers(struct sip_pvt *p, struct sip_request *req, const st if (rport && *(rport+6) == '=') rport = NULL; /* We already have a parameter to rport */ - if (rport && ast_test_flag(&p->flags[0], SIP_NAT) == SIP_NAT_ALWAYS) { + /* Check rport if NAT=yes or NAT=rfc3581 (which is the default setting) */ + if (rport && ((ast_test_flag(&p->flags[0], SIP_NAT) == SIP_NAT_ALWAYS) || (ast_test_flag(&p->flags[0], SIP_NAT) == SIP_NAT_RFC3581))) { /* We need to add received port - rport */ char tmp[256], *end; @@ -5282,9 +5283,6 @@ static int copy_via_headers(struct sip_pvt *p, struct sip_request *req, const st } /* Add rport to first VIA header if requested */ - /* Whoo hoo! Now we can indicate port address translation too! Just - another RFC (RFC3581). I'll leave the original comments in for - posterity. */ snprintf(new, sizeof(new), "%s;received=%s;rport=%d", tmp, ast_inet_ntoa(p->recv.sin_addr), ntohs(p->recv.sin_port)); diff --git a/configs/sip.conf.sample b/configs/sip.conf.sample index ef4fa2cda7..eb53346931 100644 --- a/configs/sip.conf.sample +++ b/configs/sip.conf.sample @@ -243,7 +243,7 @@ srvlookup=yes ; Enable DNS SRV lookups on outbound calls ; ;nat=no ; Global NAT settings (Affects all peers and users) ; yes = Always ignore info and assume NAT - ; no = Use NAT mode only according to RFC3581 + ; no = Use NAT mode only according to RFC3581 (;rport) ; never = Never attempt NAT mode or RFC3581 support ; route = Assume NAT, don't send rport ; (work around more UNIDEN bugs)