[asterisk-bugs] [Asterisk 0018951]: [patch] [regression] Cisco phones do not register

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18951 
====================================================================== 
Reported By:                jmls
Assigned To:                mnicholson
====================================================================== 
Project:                    Asterisk
Issue ID:                   18951
Category:                   Channels/chan_sip/Registration
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Target Version:             1.4/1.6.2 Issues (Final Release)
Asterisk Version:           SVN 
JIRA:                       SWP-3257 
Regression:                 Yes 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 309947 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2011-03-09 15:40 CST
Last Modified:              2011-04-27 14:15 CDT
====================================================================== 
Summary:                    [patch] [regression] Cisco phones do not register
Description: 
Prior to r304241, cisco phones worked fine. After this revision, cisco 79xx
phones do not register with asterisk.
====================================================================== 

---------------------------------------------------------------------- 
 (0134209) svnbot (reporter) - 2011-04-27 14:15
 https://issues.asterisk.org/view.php?id=18951#c134209 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 315895

_U  trunk/
U   trunk/channels/chan_sip.c
U   trunk/channels/sip/reqresp_parser.c

------------------------------------------------------------------------
r315895 | mnicholson | 2011-04-27 14:15:50 -0500 (Wed, 27 Apr 2011) | 35
lines

Merged revisions 315894 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r315894 | mnicholson | 2011-04-27 14:14:27 -0500 (Wed, 27 Apr 2011) | 28
lines
  
  Merged revisions 315893 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2
  
  ................
    r315893 | mnicholson | 2011-04-27 14:03:05 -0500 (Wed, 27 Apr 2011) |
21 lines
    
    Merged revisions 315891 via svnmerge from 
    https://origsvn.digium.com/svn/asterisk/branches/1.4
    
    ........
      r315891 | mnicholson | 2011-04-27 13:57:56 -0500 (Wed, 27 Apr 2011)
| 14 lines
      
      Fix our compliance with RFC 3261 section 18.2.2.
      
      This change optimizes the free_via() function and removes some
redundant null
      checking. It also fixes compliance with RFC 3261 section 18.2.2 by
always using
      the port specified in the Via header for routing responses (even
when maddr is
      not set). Also the htons() function is now used when setting the
port.
      Additional documentation comments have been added in various places
to make the
      logic in the code clearer.
      
      (closes issue https://issues.asterisk.org/view.php?id=18951)
      Reported by: jmls
      Patches:
            issue18951_set_proper_port_from_via.patch uploaded by wdoekes
(license 717) (modified)
    ........
  ................
................

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-04-27 14:15 svnbot         Checkin                                      
2011-04-27 14:15 svnbot         Note Added: 0134209                          
======================================================================