[asterisk-commits] mjordan: branch 11 r374301 - in /branches/11: ./ main/ccss.c

[SVN commits to the Asterisk project]

Author: mjordan
Date: Wed Oct  3 12:27:05 2012
New Revision: 374301

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=374301
Log:
Destroy the generic_monitors container after the core_instances in ccss

For each item in core_instances disposed of in the shutdown of ccss, any
generic monitor instances referenced by the objecs will be removed from
generic_monitors during their destruction.  Hilarity ensues if
generic_monitors no longer exists.

Thanks to the Asterisk Test Suite's generic_ccss for complaining loudly when
it ran into this.
........

Merged revisions 374300 from http://svn.asterisk.org/svn/asterisk/branches/10

Modified:
    branches/11/   (props changed)
    branches/11/main/ccss.c

Propchange: branches/11/
------------------------------------------------------------------------------
Binary property 'branch-10-merged' - no diff available.

Modified: branches/11/main/ccss.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/main/ccss.c?view=diff&rev=374301&r1=374300&r2=374301
==============================================================================
--- branches/11/main/ccss.c (original)
+++ branches/11/main/ccss.c Wed Oct  3 12:27:05 2012
@@ -4533,13 +4533,14 @@
 	if (cc_core_taskprocessor) {
 		cc_core_taskprocessor = ast_taskprocessor_unreference(cc_core_taskprocessor);
 	}
+	/* Note that core instances must be destroyed prior to the generic_monitors */
+	if (cc_core_instances) {
+		ao2_t_ref(cc_core_instances, -1, "Unref cc_core_instances container in cc_shutdown");
+		cc_core_instances = NULL;
+	}
 	if (generic_monitors) {
 		ao2_t_ref(generic_monitors, -1, "Unref generic_monitor container in cc_shutdown");
 		generic_monitors = NULL;
-	}
-	if (cc_core_instances) {
-		ao2_t_ref(cc_core_instances, -1, "Unref cc_core_instances container in cc_shutdown");
-		cc_core_instances = NULL;
 	}
 }