[asterisk-bugs] [Asterisk 0017663]: [patch] IPv6: get_refer_info

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17663 
====================================================================== 
Reported By:                oej
Assigned To:                mmichelson
====================================================================== 
Project:                    Asterisk
Issue ID:                   17663
Category:                   Channels/chan_sip/IPv6
Reproducibility:            have not tried
Severity:                   minor
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
JIRA:                       SWP-1864 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!): 277027 
Request Review:              
====================================================================== 
Date Submitted:             2010-07-16 07:45 CDT
Last Modified:              2010-07-22 15:17 CDT
====================================================================== 
Summary:                    [patch] IPv6: get_refer_info
Description: 
14207                 domain = ptr;
14208                 if ((ptr = strchr(domain, ':'))) {      /* Remove
:port */
14209                         *ptr = '\0';
14210                 }
14211 


Another case where we just search for : to separate host and port
====================================================================== 

---------------------------------------------------------------------- 
 (0124874) sperreault (reporter) - 2010-07-22 15:17
 https://issues.asterisk.org/view.php?id=17663#c124874 
---------------------------------------------------------------------- 
I think each problem should be solved individually. I don't see any big
architectural problem here.

For the auto_sip_domains part in reload_config(), we need to be using
ast_sockaddr_stringify_addr() instead of ast_sockaddr_stringify(), so that
the port is not present.

For the broken port removal that this bug initially identified, we need to
remove the port but use something more involved than strchr(). Why not use
AST_NONSTANDARD_RAW_ARGS()? It will skip matching [] pairs. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-07-22 15:17 sperreault     Note Added: 0124874                          
======================================================================