[Asterisk-code-review] manager channels: Fix throwing of HangupHandler manager events (asterisk[13])

[Gerald Schnabel]

Gerald Schnabel has uploaded this change for review. ( https://gerrit.asterisk.org/10910


Change subject: manager_channels: Fix throwing of HangupHandler manager events
......................................................................

manager_channels: Fix throwing of HangupHandler manager events

The type value extracted from stasis message data in channel_hangup_handler_cb
isn't compared against the valid values "run", "pop" and "push". Thus the
manager events HangupHandlerPush, HangupHandlerPop and HangupHandlerRun are
never thrown.

This regression was introduced by ASTERISK-21462.

ASTERISK-28252

Change-Id: I9956e35e18da1873113644df1ddc3c7cd37bf524
---
M main/manager_channels.c
1 file changed, 3 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/10/10910/1

diff --git a/main/manager_channels.c b/main/manager_channels.c
index 6938c6e..eefcaa7 100644
--- a/main/manager_channels.c
+++ b/main/manager_channels.c
@@ -963,11 +963,11 @@
 		return;
 	}
 
-	if (!strcmp(action, "type")) {
+	if (!strcmp(action, "run")) {
 		event = "HangupHandlerRun";
-	} else if (!strcmp(action, "type")) {
+	} else if (!strcmp(action, "pop")) {
 		event = "HangupHandlerPop";
-	} else if (!strcmp(action, "type")) {
+	} else if (!strcmp(action, "push")) {
 		event = "HangupHandlerPush";
 	} else {
 		return;

-- 
To view, visit https://gerrit.asterisk.org/10910
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9956e35e18da1873113644df1ddc3c7cd37bf524
Gerrit-Change-Number: 10910
Gerrit-PatchSet: 1
Gerrit-Owner: Gerald Schnabel <gs at starface.de>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190122/4c2d2f83/attachment.html>