[asterisk-commits] file: branch 12 r425131 - /branches/12/res/res_pjsip_nat.c

[SVN commits to the Asterisk project]

Author: file
Date: Fri Oct 10 07:09:39 2014
New Revision: 425131

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=425131
Log:
res_pjsip_nat: Place source port into rport of responses if 'force_rport' is on.

When the 'force_rport' option is enabled the behavior should be the same
as if the remote side placed rport into the message themselves. Therefore
any responses we send should include the source port of the request in the
rport of the Via header.

#SIPit31

ASTERISK-24387 #close
Reported by: Matt Jordan

Modified:
    branches/12/res/res_pjsip_nat.c

Modified: branches/12/res/res_pjsip_nat.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_pjsip_nat.c?view=diff&rev=425131&r1=425130&r2=425131
==============================================================================
--- branches/12/res/res_pjsip_nat.c (original)
+++ branches/12/res/res_pjsip_nat.c Fri Oct 10 07:09:39 2014
@@ -62,7 +62,7 @@
 	}
 
 	if (endpoint->nat.force_rport) {
-		rdata->msg_info.via->rport_param = 0;
+		rdata->msg_info.via->rport_param = rdata->pkt_info.src_port;
 	}
 
 	return PJ_FALSE;