From 49e63a191dd11e498e0c76e931b6823619f84c33 Mon Sep 17 00:00:00 2001 From: Joshua Colp Date: Sat, 15 Nov 2014 19:01:21 +0000 Subject: chan_motif / chan_pjsip: Fix incorrect "No such module" messages when reloading. For chan_motif the direct return value of the underlying config options framework was passed back. This can relay various states which the module loader would not interpet as success. It has been changed so only on errors will it report back an error. For chan_pjsip the code implemented a dummy reload function which always returned an error. This has been removed as all configuration is held within res_pjsip instead. ASTERISK-23651 #close Reported by: Rusty Newton ........ Merged revisions 427981 from http://svn.asterisk.org/svn/asterisk/branches/12 ........ Merged revisions 427982 from http://svn.asterisk.org/svn/asterisk/branches/13 git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@427983 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- channels/chan_motif.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'channels/chan_motif.c') diff --git a/channels/chan_motif.c b/channels/chan_motif.c index e5e72283a..48f1b1fc7 100644 --- a/channels/chan_motif.c +++ b/channels/chan_motif.c @@ -2801,7 +2801,11 @@ end: /*! \brief Reload module */ static int reload(void) { - return aco_process_config(&cfg_info, 1); + if (aco_process_config(&cfg_info, 1) == ACO_PROCESS_ERROR) { + return -1; + } + + return 0; } /*! \brief Unload the jingle channel from Asterisk */ -- cgit v1.2.3