From f4a30ad32e7e7ce12ef843fa6cdd90624c65ca79 Mon Sep 17 00:00:00 2001 From: Corey Farrell Date: Sun, 13 Jul 2014 16:48:48 +0000 Subject: Fix minor reference leaks in app_skel and TEST_FRAMEWORK * Cleanup games object in app_skel. * Cleanup stasis subscription to TEST_FRAMEWORK in manager.c (12+). Review: https://reviewboard.asterisk.org/r/3757/ ........ Merged revisions 418465 from http://svn.asterisk.org/svn/asterisk/branches/11 ........ Merged revisions 418466 from http://svn.asterisk.org/svn/asterisk/branches/12 git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@418467 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- apps/app_skel.c | 1 + main/manager.c | 10 +++++++++- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/apps/app_skel.c b/apps/app_skel.c index 8a5b5bc75..249b1b675 100644 --- a/apps/app_skel.c +++ b/apps/app_skel.c @@ -702,6 +702,7 @@ static int unload_module(void) ast_cli_unregister_multiple(skel_cli, ARRAY_LEN(skel_cli)); aco_info_destroy(&cfg_info); ao2_global_obj_release(globals); + ao2_cleanup(games); return ast_unregister_application(app); } diff --git a/main/manager.c b/main/manager.c index 31d970942..63e08631a 100644 --- a/main/manager.c +++ b/main/manager.c @@ -1150,6 +1150,10 @@ static struct stasis_forward *rtp_topic_forwarder; /*! \brief The \ref stasis_subscription for forwarding the Security topic to the AMI topic */ static struct stasis_forward *security_topic_forwarder; +#ifdef TEST_FRAMEWORK +struct stasis_subscription *test_suite_sub; +#endif + #define MGR_SHOW_TERMINAL_WIDTH 80 #define MAX_VARS 128 @@ -7987,6 +7991,10 @@ static void manager_shutdown(void) ao2_t_global_obj_release(event_docs, "Dispose of event_docs"); #endif +#ifdef TEST_FRAMEWORK + stasis_unsubscribe(test_suite_sub); +#endif + if (stasis_router) { stasis_message_router_unsubscribe_and_join(stasis_router); stasis_router = NULL; @@ -8179,7 +8187,7 @@ static int __init_manager(int reload, int by_external_config) ast_manager_register_xml_core("BlindTransfer", EVENT_FLAG_CALL, action_blind_transfer); #ifdef TEST_FRAMEWORK - stasis_subscribe(ast_test_suite_topic(), test_suite_event_cb, NULL); + test_suite_sub = stasis_subscribe(ast_test_suite_topic(), test_suite_event_cb, NULL); #endif ast_cli_register_multiple(cli_manager, ARRAY_LEN(cli_manager)); -- cgit v1.2.3