diff options
author | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-09-08 21:02:36 +0000 |
---|---|---|
committer | russell <russell@f38db490-d61c-443f-a65b-d21fe96a405b> | 2008-09-08 21:02:36 +0000 |
commit | da217c769d5e5193975cbf8f2da03835efbe80e9 (patch) | |
tree | a72c37f6f22243551275296b2e2761c923a05ac0 /main | |
parent | d819ba025e2c068c4de4a5afe3e1d10e681a5fcf (diff) |
When doing an async goto, detect if the channel is already in the middle of a
masquerade. This can happen when chan_local is trying to optimize itself out.
If this happens, fail the async goto instead of bursting into flames.
(closes issue #13435)
Reported by: geoff2010
git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@141806 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'main')
-rw-r--r-- | main/pbx.c | 24 |
1 files changed, 15 insertions, 9 deletions
diff --git a/main/pbx.c b/main/pbx.c index f4dfd0d52..9aa96118c 100644 --- a/main/pbx.c +++ b/main/pbx.c @@ -4609,17 +4609,23 @@ int ast_async_goto(struct ast_channel *chan, const char *context, const char *ex S_OR(context, chan->context), S_OR(exten, chan->exten), priority); /* Masquerade into temp channel */ - ast_channel_masquerade(tmpchan, chan); - - /* Grab the locks and get going */ - ast_channel_lock(tmpchan); - ast_do_masquerade(tmpchan); - ast_channel_unlock(tmpchan); - /* Start the PBX going on our stolen channel */ - if (ast_pbx_start(tmpchan)) { - ast_log(LOG_WARNING, "Unable to start PBX on %s\n", tmpchan->name); + if (ast_channel_masquerade(tmpchan, chan)) { + /* Failed to set up the masquerade. It's probably chan_local + * in the middle of optimizing itself out. Sad. :( */ ast_hangup(tmpchan); + tmpchan = NULL; res = -1; + } else { + /* Grab the locks and get going */ + ast_channel_lock(tmpchan); + ast_do_masquerade(tmpchan); + ast_channel_unlock(tmpchan); + /* Start the PBX going on our stolen channel */ + if (ast_pbx_start(tmpchan)) { + ast_log(LOG_WARNING, "Unable to start PBX on %s\n", tmpchan->name); + ast_hangup(tmpchan); + res = -1; + } } } } |