Skip to content

Commit

Permalink
(closes issue #12469)
Browse files Browse the repository at this point in the history
Reported by: triccyx

I had a bit a problem reproducing this in my setup (trying not to disturb my other stuff)
but finally, I got it. The problem appears to be that the extension is being added in
replace mode, which kinda assumes that the pattern trie has been formed, when in fact,
in this case, it was not. The checks being done are not nec. when the tree is not yet
formed, as changes like this will be summarized when the trie is formed in the future.

I tested the fix, and the crash no longer happens. Feel free to open the bug again if
this fix doesn't cure the problem.
  • Loading branch information
murf committed Apr 22, 2008
1 parent 1060203 commit 6ecbd78
Showing 1 changed file with 12 additions and 9 deletions.
21 changes: 12 additions & 9 deletions main/pbx.c
Expand Up @@ -6467,11 +6467,12 @@ static int add_pri(struct ast_context *con, struct ast_exten *tmp,
el->next = tmp;
/* The pattern trie points to this exten; replace the pointer,
and all will be well */

if (x->exten) { /* this test for safety purposes */
x->exten = tmp; /* replace what would become a bad pointer */
} else {
ast_log(LOG_ERROR,"Trying to delete an exten from a context, but the pattern tree node returned isn't an extension\n");
if (x) { /* if the trie isn't formed yet, don't sweat this */
if (x->exten) { /* this test for safety purposes */
x->exten = tmp; /* replace what would become a bad pointer */
} else {
ast_log(LOG_ERROR,"Trying to delete an exten from a context, but the pattern tree node returned isn't an extension\n");
}
}
} else { /* We're the very first extension. */
struct match_char *x = add_exten_to_pattern_tree(con, e, 1);
Expand All @@ -6490,10 +6491,12 @@ static int add_pri(struct ast_context *con, struct ast_exten *tmp,
con->root = tmp;
/* The pattern trie points to this exten; replace the pointer,
and all will be well */
if (x->exten) { /* this test for safety purposes */
x->exten = tmp; /* replace what would become a bad pointer */
} else {
ast_log(LOG_ERROR,"Trying to delete an exten from a context, but the pattern tree node returned isn't an extension\n");
if (x) { /* if the trie isn't formed yet; no problem */
if (x->exten) { /* this test for safety purposes */
x->exten = tmp; /* replace what would become a bad pointer */
} else {
ast_log(LOG_ERROR,"Trying to delete an exten from a context, but the pattern tree node returned isn't an extension\n");
}
}
}
if (tmp->priority == PRIORITY_HINT)
Expand Down

0 comments on commit 6ecbd78

Please sign in to comment.