mirror of
https://github.com/asterisk/asterisk.git
synced 2025-10-12 15:45:18 +00:00
Merged revisions 90967 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r90967 | russell | 2007-12-04 13:57:39 -0600 (Tue, 04 Dec 2007) | 7 lines Make some changes to some additions I made recently for doing channel autoservice when looking up extensions. This code was added to handle the case where a dialplan switch was in use that could block for a long time. However, the way that I added it, it did this for all extension lookups. However, lookups in the in-memory tree of extensions should _not_ take long enough to matter. So, move the autoservice stuff to be only around executing a switch. ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@91011 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
28
main/pbx.c
28
main/pbx.c
@@ -1809,7 +1809,15 @@ struct ast_exten *pbx_find_extension(struct ast_channel *chan,
|
||||
else /* action == E_MATCH */
|
||||
aswf = asw->exists;
|
||||
datap = sw->eval ? sw->tmpdata : sw->data;
|
||||
res = !aswf ? 0 : aswf(chan, context, exten, priority, callerid, datap);
|
||||
if (!aswf)
|
||||
res = 0;
|
||||
else {
|
||||
if (chan)
|
||||
ast_autoservice_start(chan);
|
||||
res = aswf(chan, context, exten, priority, callerid, datap);
|
||||
if (chan)
|
||||
ast_autoservice_stop(chan);
|
||||
}
|
||||
if (res) { /* Got a match */
|
||||
q->swo = asw;
|
||||
q->data = datap;
|
||||
@@ -2609,9 +2617,6 @@ static int pbx_extension_helper(struct ast_channel *c, struct ast_context *con,
|
||||
|
||||
int matching_action = (action == E_MATCH || action == E_CANMATCH || action == E_MATCHMORE);
|
||||
|
||||
if (c)
|
||||
ast_autoservice_start(c);
|
||||
|
||||
ast_rdlock_contexts();
|
||||
if (found)
|
||||
*found = 0;
|
||||
@@ -2622,14 +2627,10 @@ static int pbx_extension_helper(struct ast_channel *c, struct ast_context *con,
|
||||
*found = 1;
|
||||
if (matching_action) {
|
||||
ast_unlock_contexts();
|
||||
if (c)
|
||||
ast_autoservice_stop(c);
|
||||
return -1; /* success, we found it */
|
||||
} else if (action == E_FINDLABEL) { /* map the label to a priority */
|
||||
res = e->priority;
|
||||
ast_unlock_contexts();
|
||||
if (c)
|
||||
ast_autoservice_stop(c);
|
||||
return res; /* the priority we were looking for */
|
||||
} else { /* spawn */
|
||||
if (!e->cached_app)
|
||||
@@ -2638,8 +2639,6 @@ static int pbx_extension_helper(struct ast_channel *c, struct ast_context *con,
|
||||
ast_unlock_contexts();
|
||||
if (!app) {
|
||||
ast_log(LOG_WARNING, "No application '%s' for extension (%s, %s, %d)\n", e->app, context, exten, priority);
|
||||
if (c)
|
||||
ast_autoservice_stop(c);
|
||||
return -1;
|
||||
}
|
||||
if (c->context != context)
|
||||
@@ -2667,23 +2666,17 @@ static int pbx_extension_helper(struct ast_channel *c, struct ast_context *con,
|
||||
"AppData: %s\r\n"
|
||||
"Uniqueid: %s\r\n",
|
||||
c->name, c->context, c->exten, c->priority, app->name, passdata, c->uniqueid);
|
||||
if (c)
|
||||
ast_autoservice_stop(c);
|
||||
return pbx_exec(c, app, passdata); /* 0 on success, -1 on failure */
|
||||
}
|
||||
} else if (q.swo) { /* not found here, but in another switch */
|
||||
ast_unlock_contexts();
|
||||
if (matching_action) {
|
||||
if (c)
|
||||
ast_autoservice_stop(c);
|
||||
return -1;
|
||||
} else {
|
||||
if (!q.swo->exec) {
|
||||
ast_log(LOG_WARNING, "No execution engine for switch %s\n", q.swo->name);
|
||||
res = -1;
|
||||
}
|
||||
if (c)
|
||||
ast_autoservice_stop(c);
|
||||
return q.swo->exec(c, q.foundcontext ? q.foundcontext : context, exten, priority, callerid, q.data);
|
||||
}
|
||||
} else { /* not found anywhere, see what happened */
|
||||
@@ -2709,9 +2702,6 @@ static int pbx_extension_helper(struct ast_channel *c, struct ast_context *con,
|
||||
ast_debug(1, "Shouldn't happen!\n");
|
||||
}
|
||||
|
||||
if (c)
|
||||
ast_autoservice_stop(c);
|
||||
|
||||
return (matching_action) ? 0 : -1;
|
||||
}
|
||||
}
|
||||
|
Reference in New Issue
Block a user