[asterisk-bugs] [Asterisk 0013634]: Asterisk fills "via" header not correctly.

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13634 
====================================================================== 
Reported By:                performer
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13634
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.6.0 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-10-07 00:42 CDT
Last Modified:              2008-12-16 09:44 CST
====================================================================== 
Summary:                    Asterisk fills "via" header not correctly.
Description: 
Some provider couldn't send responses to Asterisk because information in
"via" header has port "0".

My solution:
diff chan_sip.c.bak chan_sip.c
2628c2628
<                       ntohs(p->ourip.sin_port), p->branch, rport);
---
>                       ntohs(p->socket.port), p->branch, rport);

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

---------------------------------------------------------------------- 
 (0096447) svnbot (reporter) - 2008-12-16 09:44
 http://bugs.digium.com/view.php?id=13634#c96447 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 164659

U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r164659 | file | 2008-12-16 09:44:24 -0600 (Tue, 16 Dec 2008) | 4 lines

When using externhost make sure the port gets set to the bindaddr port if
one was not specified in the externhost value itself.
(closes issue http://bugs.digium.com/view.php?id=13634)
Reported by: performer

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=164659 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-12-16 09:44 svnbot         Checkin                                      
2008-12-16 09:44 svnbot         Note Added: 0096447                          
======================================================================