From: Jonathan Rose <jrose@digium.com>
Date: Mon, 7 Apr 2014 16:15:34 +0000 (+0000)
Subject: AGI/Manager: Prevent multiple NewExten events during AGI application changes
X-Git-Tag: 13.0.0-beta1~374
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c0a812e1432aa0aeb5d913230df5ca3e1a09bdc1;p=thirdparty%2Fasterisk.git

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


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@411870 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---

diff --git a/main/manager_channels.c b/main/manager_channels.c
index 3fdcc9654b..e61f790a6d 100644
--- a/main/manager_channels.c
+++ b/main/manager_channels.c
@@ -530,8 +530,8 @@ static struct ast_manager_event_blob *channel_newexten(
 		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;
 	}