[asterisk-bugs] [Asterisk 0014727]: [patch] lock or crash after changing sip 'transport'

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=14727 
====================================================================== 
Reported By:                pj
Assigned To:                dvossel
====================================================================== 
Project:                    Asterisk
Issue ID:                   14727
Category:                   Channels/chan_sip/General
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 183057 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-03-23 16:59 CDT
Last Modified:              2009-05-22 17:51 CDT
====================================================================== 
Summary:                    [patch] lock or crash after changing sip 'transport'
Description: 
When I change sip transport protocol order, eg. udp,tcp -> tcp,udp and do
'sip reload' asterisk locks or crash


======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0013117 [patch] multiple 'transport' on peer do...
====================================================================== 

---------------------------------------------------------------------- 
 (0105346) svnbot (reporter) - 2009-05-22 17:51
 https://issues.asterisk.org/view.php?id=14727#c105346 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 196454

_U  branches/1.6.0/
U   branches/1.6.0/channels/chan_sip.c
U   branches/1.6.0/configs/sip.conf.sample

------------------------------------------------------------------------
r196454 | dvossel | 2009-05-22 17:51:10 -0500 (Fri, 22 May 2009) | 25
lines

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

........
  r196416 | dvossel | 2009-05-22 16:09:45 -0500 (Fri, 22 May 2009) | 19
lines
  
  SIP set outbound transport type from Registration
  
  In sip.conf the transport option allows for the configuration of what
transport types (udp, tcp, and tls) a peer will accept, but only the first
type listed was used for outbound connections.  This patch changes this. 
Now the default transport type is only used until the peer registers.  When
registration takes place the transport type is parsed out of the Contact
header.  If the Contact header's transport type is equal to one that the
peer supports, the peer's default transport type for outbound connections
is set to match the Contact header's type.  If the Contact header's
transport type is not present, then the peer's default transport type is
set to match the one the peer registered with.  When a peer unregisters or
the registration expires, the default transport type for that peer is
reset.
  
  (closes issue https://issues.asterisk.org/view.php?id=12282)
  Reported by: rjain
  Patches:
        reg_patch_1.diff uploaded by dvossel (license 671)
  Tested by: dvossel
  
  (closes issue https://issues.asterisk.org/view.php?id=14727)
  Reported by: pj
  Patches:
        reg_patch_3.diff uploaded by dvossel (license 671)
  Tested by: pj, dvossel
  
  Review: https://reviewboard.asterisk.org/r/249/
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-05-22 17:51 svnbot         Checkin                                      
2009-05-22 17:51 svnbot         Note Added: 0105346                          
======================================================================