Summary:	ASTERISK-11229: VoiceMailMain with res_adsi.so unloaded crashes asterisk with no error logged
Reporter:	non-poster (non-poster)	Labels:
Date Opened:	2008-01-13 17:14:30.000-0600	Date Closed:	2008-02-12 18:29:59.000-0600
Priority:	Critical	Regression?	No
Status:	Closed/Complete	Components:	Applications/app_voicemail
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) 20080114__bug11760.diff.txt
Description:	app_voicemail has an unenforced dependency on res_adsi.  If res_adsi is not loaded when VoiceMailMain is called, asterisk crashes and there is no indication at any debug level that the crash is related to adsi. ****** ADDITIONAL INFORMATION ****** Remove adsi dependency from voicemail?  Allow voicemail to fail gracefully if adsi not loaded?  Warn/error if voicemail loaded without adsi?
Comments:	By: Michiel van Baak (mvanbaak) 2008-01-14 00:44:11.000-0600 Can you attach a backtrace ? see doc/backtrace.txt By: Tilghman Lesher (tilghman) 2008-01-14 00:46:49.000-0600 Please test and verify this patch works for you. By: Joshua C. Colp (jcolp) 2008-01-14 09:08:41.000-0600 This would only be applicable to 1.4, as in trunk we have a stub in place for adsi so that stuff that uses it can be loaded even if it isn't loaded. By: non-poster (non-poster) 2008-01-14 22:04:40.000-0600 20080114__bug11760.diff.txt seems to be adequate.  Thanks. By: James Golovich (jamesgolovich) 2008-02-04 11:25:04.000-0600 I've tested this out and it works as expected By: Jason Parker (jparker) 2008-02-12 14:02:52.000-0600 Is there any reason not to just backport the adsistubs work that was added to trunk? By: Digium Subversion (svnbot) 2008-02-12 18:24:29.000-0600 Repository: asterisk Revision: 103556 U   branches/1.4/apps/app_voicemail.c ------------------------------------------------------------------------ r103556 | tilghman | 2008-02-12 18:24:28 -0600 (Tue, 12 Feb 2008) | 7 lines Refuse to load app_voicemail if res_adsi is not loaded (which is a symbol dependency) (closes issue ASTERISK-11229) Reported by: non-poster Patches:       20080114__bug11760.diff.txt uploaded by Corydon76 (license 14) Tested by: Corydon76, non-poster, jamesgolovich ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=103556 By: Digium Subversion (svnbot) 2008-02-12 18:28:22.000-0600 Repository: asterisk Revision: 103557 _U  trunk/ ------------------------------------------------------------------------ r103557 | tilghman | 2008-02-12 18:28:22 -0600 (Tue, 12 Feb 2008) | 14 lines Blocked revisions 103556 via svnmerge ........ r103556 | tilghman | 2008-02-12 18:26:57 -0600 (Tue, 12 Feb 2008) | 7 lines Refuse to load app_voicemail if res_adsi is not loaded (which is a symbol dependency) (closes issue ASTERISK-11229) Reported by: non-poster Patches:       20080114__bug11760.diff.txt uploaded by Corydon76 (license 14) Tested by: Corydon76, non-poster, jamesgolovich ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=103557