mirror of
https://github.com/asterisk/asterisk.git
synced 2025-09-06 12:36:58 +00:00
Fix up some PBX logic that became broken. The code would exit prematurely when it should have been collecting more digits.
(closes issue #11175) Reported by: pj git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@89072 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
52
main/pbx.c
52
main/pbx.c
@@ -2541,36 +2541,36 @@ static int __ast_pbx_run(struct ast_channel *c)
|
|||||||
pos = 0;
|
pos = 0;
|
||||||
dst_exten[pos++] = digit = res;
|
dst_exten[pos++] = digit = res;
|
||||||
dst_exten[pos] = '\0';
|
dst_exten[pos] = '\0';
|
||||||
}
|
} else if (res == AST_PBX_KEEPALIVE) {
|
||||||
if (res == AST_PBX_KEEPALIVE) {
|
|
||||||
ast_debug(1, "Spawn extension (%s,%s,%d) exited KEEPALIVE on '%s'\n", c->context, c->exten, c->priority, c->name);
|
ast_debug(1, "Spawn extension (%s,%s,%d) exited KEEPALIVE on '%s'\n", c->context, c->exten, c->priority, c->name);
|
||||||
ast_verb(2, "Spawn extension (%s, %s, %d) exited KEEPALIVE on '%s'\n", c->context, c->exten, c->priority, c->name);
|
ast_verb(2, "Spawn extension (%s, %s, %d) exited KEEPALIVE on '%s'\n", c->context, c->exten, c->priority, c->name);
|
||||||
error = 1;
|
error = 1;
|
||||||
}
|
|
||||||
ast_debug(1, "Spawn extension (%s,%s,%d) exited non-zero on '%s'\n", c->context, c->exten, c->priority, c->name);
|
|
||||||
ast_verb(2, "Spawn extension (%s, %s, %d) exited non-zero on '%s'\n", c->context, c->exten, c->priority, c->name);
|
|
||||||
|
|
||||||
if ((res == AST_PBX_ERROR) && ast_exists_extension(c, c->context, "e", 1, c->cid.cid_num)) {
|
|
||||||
/* if we are already on the 'e' exten, don't jump to it again */
|
|
||||||
if (!strcmp(c->exten, "e")) {
|
|
||||||
if (option_verbose > 1)
|
|
||||||
ast_verbose(VERBOSE_PREFIX_2 "Spawn extension (%s, %s, %d) exited ERROR while already on 'e' exten on '%s'\n", c->context, c->exten, c->priority, c->name);
|
|
||||||
error = 1;
|
|
||||||
} else {
|
|
||||||
pbx_builtin_raise_exception(c, "ERROR");
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if (c->_softhangup == AST_SOFTHANGUP_ASYNCGOTO) {
|
|
||||||
c->_softhangup = 0;
|
|
||||||
} else if (c->_softhangup == AST_SOFTHANGUP_TIMEOUT) {
|
|
||||||
/* atimeout, nothing bad */
|
|
||||||
} else {
|
} else {
|
||||||
if (c->cdr)
|
ast_debug(1, "Spawn extension (%s,%s,%d) exited non-zero on '%s'\n", c->context, c->exten, c->priority, c->name);
|
||||||
ast_cdr_update(c);
|
ast_verb(2, "Spawn extension (%s, %s, %d) exited non-zero on '%s'\n", c->context, c->exten, c->priority, c->name);
|
||||||
error = 1;
|
|
||||||
break;
|
if ((res == AST_PBX_ERROR) && ast_exists_extension(c, c->context, "e", 1, c->cid.cid_num)) {
|
||||||
|
/* if we are already on the 'e' exten, don't jump to it again */
|
||||||
|
if (!strcmp(c->exten, "e")) {
|
||||||
|
if (option_verbose > 1)
|
||||||
|
ast_verbose(VERBOSE_PREFIX_2 "Spawn extension (%s, %s, %d) exited ERROR while already on 'e' exten on '%s'\n", c->context, c->exten, c->priority, c->name);
|
||||||
|
error = 1;
|
||||||
|
} else {
|
||||||
|
pbx_builtin_raise_exception(c, "ERROR");
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (c->_softhangup == AST_SOFTHANGUP_ASYNCGOTO) {
|
||||||
|
c->_softhangup = 0;
|
||||||
|
} else if (c->_softhangup == AST_SOFTHANGUP_TIMEOUT) {
|
||||||
|
/* atimeout, nothing bad */
|
||||||
|
} else {
|
||||||
|
if (c->cdr)
|
||||||
|
ast_cdr_update(c);
|
||||||
|
error = 1;
|
||||||
|
break;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if (error)
|
if (error)
|
||||||
|
Reference in New Issue
Block a user