[asterisk-bugs] [Asterisk 0018790]: [patch] Deadlock In chan_sip (conlock / cb_extensionstate

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18790 
====================================================================== 
Reported By:                irroot
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18790
Category:                   Channels/chan_sip/General
Reproducibility:            random
Severity:                   major
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.2.16.1 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2011-02-11 01:40 CST
Last Modified:              2011-02-11 03:31 CST
====================================================================== 
Summary:                    [patch] Deadlock In chan_sip (conlock /
cb_extensionstate
Description: 

There is a deadlock in processing state changes it would appear conlock
should be locked when calling cb_extensionstate from chan_sip.c ...

have a patch ill be loading up in a follow up.

this was a problem in 1.4 that i hacked to death to make it go away this
patch seems to make more sense.

NB we have only now moved from 1.4.36 [with backports] to 1.6.2.16.1 this
problem affects sites with higher call volume and reliance on BLF
[Snom/Polycom] and may be triggered on a sip/system reload
====================================================================== 

---------------------------------------------------------------------- 
 (0131846) schmidts (manager) - 2011-02-11 03:31
 https://issues.asterisk.org/view.php?id=18790#c131846 
---------------------------------------------------------------------- 
there is allready a solution for this submitted in 1.6.2 branch rev 302265
but i dont think this made it into 1.6.2.16.1

this patch written by jeff peeler solves the deadlock in
handle_statechange instead of chan_sip.c which makes more sense cause its
called more often than the place you attached your patch.

please take a look if this revision is allready in 1.6.2.16.1 if yes and
you still got this problem please reply.

if not try the patch from rev 302265.

thanks!

best regards
stefan 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-02-11 03:31 schmidts       Note Added: 0131846                          
======================================================================