[Home]

Summary:ASTERISK-12499: Several deadlocks in skinny
Reporter:Jason Parker (jparker)Labels:
Date Opened:2008-07-31 18:18:50Date Closed:2008-08-01 05:47:08
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_skinny
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 2008080100_bug13215.diff.txt
Description:mvanbaak brought up a few places where a deadlock is certain to happen, and I found one more where a lock is just locked again instead of being unlocked.

I'm not sure how many of these are in 1.4/1.6.0
Comments:By: Jason Parker (jparker) 2008-07-31 18:20:36

handle_skinny_show_lines is the one I found, and is trunk only.
By: Michiel van Baak (mvanbaak) 2008-07-31 18:26:03

patch for trunk attached, as I cannot commit right now with this public wifi on my boat.
Will have a look at 1.4 tomorrow when I'm home and resolve these nasty issues.

Thanks qwell.
By: Michiel van Baak (mvanbaak) 2008-08-01 05:45:47

Patch for 1.4 is almost the same, minus the handle_skinny_show_lines part.
By: Digium Subversion (svnbot) 2008-08-01 05:47:06

Repository: asterisk
Revision: 135055

U   branches/1.4/channels/chan_skinny.c

------------------------------------------------------------------------
r135055 | mvanbaak | 2008-08-01 05:47:04 -0500 (Fri, 01 Aug 2008) | 8 lines

fix some potential deadlocks in chan_skinny

(closes issue ASTERISK-12499)
Reported by: qwell
Patches:
     2008080100_bug13215.diff.txt uploaded by mvanbaak (license 7)
Tested by: mvanbaak

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

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