[asterisk-commits] jrose: branch 12 r411868 - /branches/12/main/manager_channels.c

[SVN commits to the Asterisk project]

Author: jrose
Date: Mon Apr  7 11:02:44 2014
New Revision: 411868

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=411868
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/

Modified:
    branches/12/main/manager_channels.c

Modified: branches/12/main/manager_channels.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/main/manager_channels.c?view=diff&rev=411868&r1=411867&r2=411868
==============================================================================
--- branches/12/main/manager_channels.c (original)
+++ branches/12/main/manager_channels.c Mon Apr  7 11:02:44 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;
 	}