kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-02 02:18:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

app_dial: Immediately exit dial if the caller is already hung up.

Browse Source 
If a caller hangs up before dial is executed within an AGI then the AGI
has likely eaten all queued frames before executing the dial in DeadAGI
mode.  With the caller hung up and no pending frames from the caller's
read queue, dial would not know that the call has hung up until a called
channel answers.  It is rather annoying to whoever just answered the
non-existent call.

Dial should not continue execution in DeadAGI mode, hangup handlers, or
the h exten.

* Added a check early in dial to abort dialing if the caller has hungup.

ASTERISK-25307 #close
Reported by: David Cunningham

Change-Id: Icd1bc0764726ef8c809f76743ca008d0f102f418
This commit is contained in:
Richard Mudgett
2015-12-04 17:22:29 -06:00
parent f42036bf6b
commit 80a8b2a4cd
1 changed files with 18 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
apps/app_dial.c
View File

@@ -2151,6 +2151,24 @@ static int dial_exec_full(struct ast_channel *chan, const char *data, struct ast
return -1; return -1;
} }
if (ast_check_hangup_locked(chan)) {
/*
* Caller hung up before we could dial. If dial is executed
* within an AGI then the AGI has likely eaten all queued
* frames before executing the dial in DeadAGI mode. With
* the caller hung up and no pending frames from the caller's
* read queue, dial would not know that the call has hung up
* until a called channel answers. It is rather annoying to
* whoever just answered the non-existent call.
*
* Dial should not continue execution in DeadAGI mode, hangup
* handlers, or the h exten.
*/
ast_verb(3, "Caller hung up before dial.\n");
pbx_builtin_setvar_helper(chan, "DIALSTATUS", "CANCEL");
return -1;
}
parse = ast_strdupa(data); parse = ast_strdupa(data);
AST_STANDARD_APP_ARGS(args, parse); AST_STANDARD_APP_ARGS(args, parse);