[asterisk-bugs] [Asterisk 0014176]: [patch] send out the incorrect register request URI to the (fromdomain) outbound proxy

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14176 
====================================================================== 
Reported By:                paraeco
Assigned To:                otherwiseguy
====================================================================== 
Project:                    Asterisk
Issue ID:                   14176
Category:                   Channels/chan_sip/Registration
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     confirmed
Asterisk Version:           1.4.22 
Regression:                 Yes 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 126902 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2009-01-05 21:31 CST
Last Modified:              2009-01-13 12:34 CST
====================================================================== 
Summary:                    [patch] send out the incorrect register request URI
to the (fromdomain) outbound proxy
Description: 
It is actually reopening the issue http://bugs.digium.com/view.php?id=12474 .
A bug is introduced into the file channel/chan_sip.c in the asterisk 1.4
branch -- SVN revision http://bugs.digium.com/view.php?id=126902.

ast_string_field_set(p, fromdomain, ++fromdomain);

In the above code, ++fromdomain would be executed twice in the macro
ast_string_field_set() .
It results in the first character of the domain name to be eliminated .
A fix is included in the additional information filed .

Edit: the fix has been removed as all patch submissions MUST be posted to
the tracker correctly; not inline.

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

---------------------------------------------------------------------- 
 (0097619) otherwiseguy (administrator) - 2009-01-13 12:34
 http://bugs.digium.com/view.php?id=14176#c97619 
---------------------------------------------------------------------- 
Whether the macro was fixed or not, I don't like passing a value with a
side effect to a macro anyway, so I'm committing this since it looks right. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-13 12:34 otherwiseguy   Note Added: 0097619                          
======================================================================