[Home]

Summary:ASTERISK-12757: [patch] Fixes skinny unload
Reporter:Damien Wedhorn (wedhorn)Labels:
Date Opened:2008-09-19 19:16:16Date Closed:2008-09-25 20:43:15
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Channels/chan_skinny
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 2008092000_skinnyunload.diff.txt
( 1) unload.diff
Description:allows module unload chan_skinny.so
Comments:By: Michiel van Baak (mvanbaak) 2008-09-20 02:56:44

When you have your phones registering a noop line in the dialplan, and you unload chan_skinny.so, and after that do a 'dialplan show <regcontext>' asterisk dies.

The BT is totally useless.

Looking into it now and will provide some more info when I have that.
By: Michiel van Baak (mvanbaak) 2008-09-20 03:16:51

Found it.

When you add only one regcontext everything works fine.
If you have, like me, several it will only destroy the context that's created the last.

I dont think this is skinny specific. I'm now setting up sip softphones to confirm this issue is there as well

[edit]
Good and bad news. Sip leaves the context around as well. But it does not crash because it removes the inserted extensions correctly


By: Michiel van Baak (mvanbaak) 2008-09-20 04:07:27

new patch uploaded that fixes the crash.
By: Digium Subversion (svnbot) 2008-09-20 05:29:41

Repository: asterisk
Revision: 143799

U   trunk/channels/chan_skinny.c

------------------------------------------------------------------------
r143799 | mvanbaak | 2008-09-20 05:29:39 -0500 (Sat, 20 Sep 2008) | 8 lines

make 'module unload chan_skinny.so' actually work.

(closes issue ASTERISK-12757)
Reported by: wedhorn
Patches:
     unload.diff uploaded by wedhorn (license 30)
 With small tweak by me to prevent a crash

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

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