[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:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18951
Category:                   Channels/chan_sip/Registration
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     acknowledged
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:              
====================================================================== 
Date Submitted:             2011-03-09 15:40 CST
Last Modified:              2011-04-26 14:45 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.
====================================================================== 

---------------------------------------------------------------------- 
 (0134151) wdoekes (reporter) - 2011-04-26 14:45
 https://issues.asterisk.org/view.php?id=18951#c134151 
---------------------------------------------------------------------- 
(1) I haven't tested the patch, apart from checking that it compiles.
(2) It touches 3 things:
(2a) ast_free already does NULL-checking, so I've been told
(2b) the port-setting from via should be done even if maddr is not set
(2c) I found a stray ntohs that should be htons

If you can find the time, please test if this works. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-04-26 14:45 wdoekes        Note Added: 0134151                          
======================================================================