diff -uNr asterisk-1.6.2.2.ORIG/main/event.c asterisk-1.6.2.2/main/event.c --- asterisk-1.6.2.2.ORIG/main/event.c 2010-02-10 21:05:55.278139356 +0000 +++ asterisk-1.6.2.2/main/event.c 2010-02-10 21:06:58.008198202 +0000 @@ -841,6 +841,11 @@ struct ast_event_iterator iterator; int res = 0; + /* Event has no IEs allocated */ + if (event->event_len < sizeof(*event) + sizeof(ie_type)) { + return NULL; + } + for (ast_event_iterator_init(&iterator, event); !res; res = ast_event_iterator_next(&iterator)) { if (ast_event_iterator_get_ie_type(&iterator) == ie_type) return ast_event_iterator_get_ie_raw(&iterator); diff -uNr asterisk-1.6.2.2.ORIG/main/loader.c asterisk-1.6.2.2/main/loader.c --- asterisk-1.6.2.2.ORIG/main/loader.c 2010-02-10 21:05:55.273223462 +0000 +++ asterisk-1.6.2.2/main/loader.c 2010-02-10 21:06:58.006203410 +0000 @@ -444,26 +444,39 @@ 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)