mirror of
https://github.com/asterisk/asterisk.git
synced 2025-09-18 18:58:22 +00:00
Merged revisions 58164 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.2 ........ r58164 | russell | 2007-03-06 18:20:13 -0600 (Tue, 06 Mar 2007) | 4 lines If the channels acquired using the manager Redirect action are not up, then don't attempt to do anything with them. It could lead to weird behavior, including crashes. (issue #8977) ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@58165 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
@@ -1558,8 +1558,19 @@ static int action_redirect(struct mansession *s, const struct message *m)
|
|||||||
astman_send_error(s, m, buf);
|
astman_send_error(s, m, buf);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
if (chan->_state != AST_STATE_UP) {
|
||||||
|
astman_send_error(s, m, "Redirect failed, channel not up.\n");
|
||||||
|
ast_channel_unlock(chan);
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
if (!ast_strlen_zero(name2))
|
if (!ast_strlen_zero(name2))
|
||||||
chan2 = ast_get_channel_by_name_locked(name2);
|
chan2 = ast_get_channel_by_name_locked(name2);
|
||||||
|
if (chan2 && chan2->_state != AST_STATE_UP) {
|
||||||
|
astman_send_error(s, m, "Redirect failed, extra channel not up.\n");
|
||||||
|
ast_channel_unlock(chan);
|
||||||
|
ast_channel_unlock(chan2);
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
res = ast_async_goto(chan, context, exten, pi);
|
res = ast_async_goto(chan, context, exten, pi);
|
||||||
if (!res) {
|
if (!res) {
|
||||||
if (!ast_strlen_zero(name2)) {
|
if (!ast_strlen_zero(name2)) {
|
||||||
|
Reference in New Issue
Block a user