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
Versions:Frequency of
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
     2008080100_bug13215.diff.txt uploaded by mvanbaak (license 7)
Tested by: mvanbaak