[asterisk-bugs] [Asterisk 0016847]: [patch] Deadlock between dahdi_exception and dahdi_indicate

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16847 
====================================================================== 
Reported By:                shin-shoryuken
Assigned To:                rmudgett
====================================================================== 
Project:                    Asterisk
Issue ID:                   16847
Category:                   Channels/chan_dahdi
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Target Version:             1.6.2.15
Asterisk Version:           SVN 
JIRA:                       SWP-927 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2010-02-16 15:16 CST
Last Modified:              2010-10-13 18:45 CDT
====================================================================== 
Summary:                    [patch] Deadlock between dahdi_exception and
dahdi_indicate
Description: 
I've encountered a deadlock situation between dahdi_exception and
dahdi_indicate, running Asterisk 1.6.1.6 (confirmed as well in 1.6.1.14) on
Debian Lenny x86_64.  Deadlock occurs roughly once a week, usually under
lower call volume (strangely enough).  The issue seems to arise mostly when
swapping between call-waiting lines on a DAHDI channel.
====================================================================== 

---------------------------------------------------------------------- 
 (0128000) svnbot (reporter) - 2010-10-13 18:45
 https://issues.asterisk.org/view.php?id=16847#c128000 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 291656

_U  branches/1.8/
U   branches/1.8/channels/chan_dahdi.c
U   branches/1.8/channels/sig_analog.c
U   branches/1.8/channels/sig_analog.h

------------------------------------------------------------------------
r291656 | rmudgett | 2010-10-13 18:45:12 -0500 (Wed, 13 Oct 2010) | 34
lines

Merged revisions 291655 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r291655 | rmudgett | 2010-10-13 18:36:50 -0500 (Wed, 13 Oct 2010) | 27
lines
  
  Merged revisions 291643 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r291643 | rmudgett | 2010-10-13 18:29:58 -0500 (Wed, 13 Oct 2010) | 20
lines
    
    Deadlock between dahdi_exception() and dahdi_indicate().
    
    There is a deadlock between dahdi_exception() and dahdi_indicate() for
    analog ports.  The call-waiting and three-way-calling feature can
    experience deadlock if these features are trying to do something and
an
    event from the bridged channel happens at the same time.
    
    Deadlock avoidance code added to obtain necessary channel locks before
    attemting an operation with call-waiting and three-way-calling.
    
    (closes issue https://issues.asterisk.org/view.php?id=16847)
    Reported by: shin-shoryuken
    Patches:
          issue_16847_v1.4.patch uploaded by rmudgett (license 664)
          issue_16847_v1.6.2.patch uploaded by rmudgett (license 664)
          issue_16847_v1.8_v2.patch uploaded by rmudgett (license 664)
    Tested by: alecdavis, rmudgett
    
    Review: https://reviewboard.asterisk.org/r/971/
  ........
................

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-10-13 18:45 svnbot         Checkin                                      
2010-10-13 18:45 svnbot         Note Added: 0128000                          
======================================================================