[asterisk-bugs] [Asterisk 0013195]: chan_sip fails to parse reason parameter in Diversion: header

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13195 
====================================================================== 
Reported By:                woodsfsg
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13195
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 134222 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-07-30 04:29 CDT
Last Modified:              2008-07-30 15:16 CDT
====================================================================== 
Summary:                    chan_sip fails to parse reason parameter in
Diversion: header
Description: 
Recently, a call to get_in_brackets was introduced into the get_rdnis
function. get_in_brackets modifies the tmp string by putting a '/0'
character in place of the closing '>'. This breaks the subsequent
diversion-reason parsing as it effectively truncates tmp.

Two possible fixes:
start the parse from tmp + strlen(tmp) + 1

Or:
refresh tmp by calling get_header again.
====================================================================== 

---------------------------------------------------------------------- 
 (0090865) svnbot (reporter) - 2008-07-30 15:16
 http://bugs.digium.com/view.php?id=13195#c90865 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 134556

U   trunk/channels/chan_sip.c

------------------------------------------------------------------------
r134556 | mmichelson | 2008-07-30 15:16:34 -0500 (Wed, 30 Jul 2008) | 7
lines

Fix the parsing of the "reason" parameter in the
Diversion: header.

(closes issue http://bugs.digium.com/view.php?id=13195)
Reported by: woodsfsg


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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-07-30 15:16 svnbot         Note Added: 0090865                          
======================================================================