[asterisk-bugs] [Asterisk 0013823]: [patch] Response to REGISTER is sent to wrong address when using rport

[Asterisk Bug Tracker]

The following issue has been UPDATED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=13823 
====================================================================== 
Reported By:                dimas
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13823
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     acknowledged
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2008-11-02 12:08 CST
Last Modified:              2010-03-18 18:06 CDT
====================================================================== 
Summary:                    [patch] Response to REGISTER is sent to wrong
address when using rport
Description: 
I have client behind NAT. 
when client (10.0.0.17) sends REGISTER request, it puts rport into Via
header. check_via notices that and sets SIP_NAT_ROUTE to the &p->flags[0] -
note the log line:

Sending to 192.168.7.211 : 5079 (NAT)

however the register_verify function later does 
  ast_copy_flags(&p->flags[0], &peer->flags[0], SIP_NAT); 
effectively overwritting the value with SIP_NAT_RFC3581. And finally,
check_auth send response back to client to the wrong address which is
visible from the log line 

<--- Transmitting (no NAT) to 10.0.0.17:5079 --->

====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-03-18 18:06 lmadsen        Description Updated                          
2010-03-18 18:06 lmadsen        Additional Information Updated                  
 
======================================================================