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
Versions:Frequency of
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

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
     unload.diff uploaded by wedhorn (license 30)
 With small tweak by me to prevent a crash