[asterisk-bugs] [Asterisk 0014110]: [patch] SIPAddHeader problems with escaping and quoting

[Asterisk Bug Tracker]

The following issue is now READY FOR REVIEW. 
====================================================================== 
http://bugs.digium.com/view.php?id=14110 
====================================================================== 
Reported By:                gork
Assigned To:                Corydon76
====================================================================== 
Project:                    Asterisk
Issue ID:                   14110
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   tweak
Priority:                   normal
Status:                     ready for review
Asterisk Version:           1.6.0.3-rc1 
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-12-18 17:58 CST
Last Modified:              2008-12-23 12:35 CST
====================================================================== 
Summary:                    [patch] SIPAddHeader problems with escaping and
quoting
Description: 
To include semicolons in an argument to SIPAddHeader in extensions.conf
they must be escaped. SipAddHeader does not, however, unescape them when
adding the header, thus it is impossible to include semicolons in a sip
header using extensions.conf. Semicolons are a very important part of many
SIP headers that users might want to add with this application.

Additionally, if a double quote is the last character of the argument
passed to SIPAddHeader and the entire header is not contained within double
quotes, it will not be included in the header.
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-12-23 12:35 blitzrage      Status                   ready for testing =>
ready for review
======================================================================