| Summary: | ASTERISK-00164: app_voicemail2 load/unload fix | ||
| Reporter: | thansen (thansen) | Labels: | |
| Date Opened: | 2003-08-25 10:10:11 | Date Closed: | 2008-01-15 14:33:13.000-0600 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Core/General |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) asterisk-cxnet-voicemail2-fixconfigfailed-patch | |
| Description: | If app_voicemail2 fails to load voicemail.conf it'll fail and never register voicemail and voicemailmain. Asterisk will then call unload on the module which also fails, because it tries to unregister the two apps that were never registered. The result is that app_voicemail2 will be stuck and unusable in Asterisk and the only remedy will be to restart Asterisk. ****** ADDITIONAL INFORMATION ****** The patch included fixes this race by registering the apps before it loads the config, but it would probably be better to have a global tell the module if it had registered the apps or an app lookup function in unload (pbx_findapp perhaps) to only unregister what has been registered. (patch disclaimed) | ||
| Comments: | By: Mark Spencer (markster) 2003-08-25 10:39:58 Fixed in CVS By: Digium Subversion (svnbot) 2008-01-15 14:33:13.000-0600 Repository: asterisk Revision: 1427 U trunk/apps/app_voicemail2.c ------------------------------------------------------------------------ r1427 | markster | 2008-01-15 14:33:13 -0600 (Tue, 15 Jan 2008) | 2 lines Make app_voicemail2 work when there is no config file (bug ASTERISK-164) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=1427 | ||