[asterisk-commits] mjordan: trunk r394530 - /trunk/main/cel.c

[SVN commits to the Asterisk project]

Author: mjordan
Date: Tue Jul 16 17:25:33 2013
New Revision: 394530

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=394530
Log:
Re-order handlers in CEL to ensure that HANGUP events happen after APP_END

When a channel is hungup, both an APP_END event and a HANGUP event can be
fired. To ensure that HANGUP events occur after APP_END events, the method
callbacks for the APP_END event should be processed prior to the callbacks
for the HANGUP event.

Modified:
    trunk/main/cel.c

Modified: trunk/main/cel.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/cel.c?view=diff&rev=394530&r1=394529&r2=394530
==============================================================================
--- trunk/main/cel.c (original)
+++ trunk/main/cel.c Tue Jul 16 17:25:33 2013
@@ -1145,9 +1145,14 @@
 	}
 }
 
+/* \brief Handlers for channel snapshot changes.
+ * \note Order of the handlers matters. Application changes must come before state
+ * changes to ensure that hangup notifications occur after application changes.
+ * Linkedid checking should always come last.
+ */
 cel_channel_snapshot_monitor cel_channel_monitors[] = {
+	cel_channel_app_change,
 	cel_channel_state_change,
-	cel_channel_app_change,
 	cel_channel_linkedid_change,
 };