diff options
author | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-02-19 19:05:34 +0000 |
---|---|---|
committer | tilghman <tilghman@f38db490-d61c-443f-a65b-d21fe96a405b> | 2010-02-19 19:05:34 +0000 |
commit | 10ba3de332116a3584e4b4df75bc5abe36bafb46 (patch) | |
tree | 30975937daa5bbb35bee8669b8dd8882f3eccb9a | |
parent | 3a8db806e617d9fe566b9a3dabb159dbe4616797 (diff) |
Merged revisions 228798 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk
(closes issue #16470)
Reported by: kjotte
........
r228798 | tilghman | 2009-11-09 01:37:52 -0600 (Mon, 09 Nov 2009) | 14 lines
Fix various problems detected with Valgrind.
* chan_console accessed pvts after deallocation.
* The module loader did not check usecount on shutdown, which led to chan_iax2
reading a timer that was already unloaded.
(closes issue #16062)
Reported by: alexanderheinz
Patches:
20091109__issue16062.diff.txt uploaded by tilghman (license 14)
Tested by: tilghman
........
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.6.1@248009 f38db490-d61c-443f-a65b-d21fe96a405b
-rw-r--r-- | channels/chan_console.c | 1 | ||||
-rw-r--r-- | main/loader.c | 47 |
2 files changed, 31 insertions, 17 deletions
diff --git a/channels/chan_console.c b/channels/chan_console.c index 4c9b91a91..e80327d77 100644 --- a/channels/chan_console.c +++ b/channels/chan_console.c @@ -1502,6 +1502,7 @@ return_error_pa_init: return_error: if (pvts) ao2_ref(pvts, -1); + pvts = NULL; pvt_destructor(&globals); return AST_MODULE_LOAD_DECLINE; diff --git a/main/loader.c b/main/loader.c index fb57189b5..268cc80c8 100644 --- a/main/loader.c +++ b/main/loader.c @@ -443,26 +443,39 @@ static struct ast_module *load_dynamic_module(const char *resource_in, unsigned void ast_module_shutdown(void) { struct ast_module *mod; - AST_LIST_HEAD_NOLOCK_STATIC(local_module_list, ast_module); - - /* We have to call the unload() callbacks in reverse order that the modules - * exist in the module list so it is the reverse order of how they were - * loaded. */ + int somethingchanged = 1, final = 0; AST_LIST_LOCK(&module_list); - while ((mod = AST_LIST_REMOVE_HEAD(&module_list, entry))) - AST_LIST_INSERT_HEAD(&local_module_list, mod, entry); - AST_LIST_UNLOCK(&module_list); - while ((mod = AST_LIST_REMOVE_HEAD(&local_module_list, entry))) { - if (mod->info->unload) - mod->info->unload(); - /* Since this should only be called when shutting down "gracefully", - * all channels should be down before we get to this point, meaning - * there will be no module users left. */ - AST_LIST_HEAD_DESTROY(&mod->users); - free(mod); - } + /*!\note Some resources, like timers, are started up dynamically, and thus + * may be still in use, even if all channels are dead. We must therefore + * check the usecount before asking modules to unload. */ + do { + if (!somethingchanged) { + /*!\note If we go through the entire list without changing + * anything, ignore the usecounts and unload, then exit. */ + final = 1; + } + + /* Reset flag before traversing the list */ + somethingchanged = 0; + + AST_LIST_TRAVERSE_SAFE_BEGIN(&module_list, mod, entry) { + if (!final && mod->usecount) { + continue; + } + AST_LIST_REMOVE_CURRENT(entry); + if (mod->info->unload) { + mod->info->unload(); + } + AST_LIST_HEAD_DESTROY(&mod->users); + free(mod); + somethingchanged = 1; + } + AST_LIST_TRAVERSE_SAFE_END; + } while (somethingchanged && !final); + + AST_LIST_UNLOCK(&module_list); } int ast_unload_resource(const char *resource_name, enum ast_module_unload_mode force) |