[asterisk-commits] jrose: trunk r411870 - in /trunk: ./ main/manager_channels.c

[SVN commits to the Asterisk project]

Author: jrose
Date: Mon Apr  7 11:15:34 2014
New Revision: 411870

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=411870
Log:
AGI/Manager: Prevent multiple NewExten events during AGI application changes

AGI applications would trigger NewExten events every time the state of the AGI
application changed. This has historically not been the behavior and this
behavior was introduced with a CDR patch. This patch corrects that.

(closes issue ASTERISK-23390)
Reported by: Benjamin Keith Ford
Review: https://reviewboard.asterisk.org/r/3406/
........

Merged revisions 411868 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    trunk/   (props changed)
    trunk/main/manager_channels.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-12-merged' - no diff available.

Modified: trunk/main/manager_channels.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/manager_channels.c?view=diff&rev=411870&r1=411869&r2=411870
==============================================================================
--- trunk/main/manager_channels.c (original)
+++ trunk/main/manager_channels.c Mon Apr  7 11:15:34 2014
@@ -530,8 +530,8 @@
 		return NULL;
 	}
 
-	if (old_snapshot && ast_channel_snapshot_cep_equal(old_snapshot, new_snapshot)
-		&& !strcmp(old_snapshot->appl, new_snapshot->appl)) {
+	/* Ignore updates if the CEP is unchanged */
+	if (old_snapshot && ast_channel_snapshot_cep_equal(old_snapshot, new_snapshot)) {
 		return NULL;
 	}