[asterisk-bugs] [Asterisk 0013547]: [patch] Asterisk crash getting fax by sip channel

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13547 
====================================================================== 
Reported By:                tecnoxarxa
Assigned To:                dwaynemh
====================================================================== 
Project:                    Asterisk
Issue ID:                   13547
Category:                   Channels/chan_sip/General
Reproducibility:            sometimes
Severity:                   crash
Priority:                   normal
Status:                     closed
Asterisk Version:           1.6.0-rc6 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2008-09-23 13:07 CDT
Last Modified:              2009-02-10 11:08 CST
====================================================================== 
Summary:                    [patch] Asterisk crash getting fax by sip channel
Description: 
Function: sip_uri_headers_cmp
====================================================================== 

---------------------------------------------------------------------- 
 (0099802) svnbot (reporter) - 2009-02-10 11:08
 http://bugs.digium.com/view.php?id=13547#c99802 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 174085

_U  branches/1.6.0/
U   branches/1.6.0/channels/chan_sip.c

------------------------------------------------------------------------
r174085 | dhubbard | 2009-02-06 17:59:42 -0600 (Fri, 06 Feb 2009) | 32
lines

Merged revisions 174084 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
r174084 | dhubbard | 2009-02-06 17:51:56 -0600 (Fri, 06 Feb 2009) | 13
lines

Merged revisions 174082 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r174082 | dhubbard | 2009-02-06 17:36:03 -0600 (Fri, 06 Feb 2009) | 5
lines

check ast_strlen_zero() before calling ast_strdupa() in
sip_uri_headers_cmp()
and sip_uri_params_cmp()

The reporter didn't actually upload a properly-formed patch, instead a 
modified chan_sip.c file was uploaded.  I created a patch to determine the
changes, then modified the suggested changes to create a proper fix.  The
summary above is a complete description of the changes.

(closes issue http://bugs.digium.com/view.php?id=13547)
Reported by: tecnoxarxa
Patches:
      chan_sip.c.gz uploaded by tecnoxarxa (license 258)
Tested by: tecnoxarxa

........

................

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

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-02-10 11:08 svnbot         Checkin                                      
2009-02-10 11:08 svnbot         Note Added: 0099802                          
======================================================================