app_if: Fix next priority calculation.

Commit fa3922a4d2 fixed
a branching issue but "overshoots" when calculating
the next priority. This fixes that; accompanying
test suite tests have also been extended.

Resolves: #560
This commit is contained in:
Naveen Albert 2024-01-28 08:57:47 -05:00
parent 103a6dc987
commit 64f9df5e59
1 changed files with 3 additions and 1 deletions

View File

@ -286,9 +286,11 @@ static int if_helper(struct ast_channel *chan, const char *data, int end)
snprintf(end_varname,sizeof(end_varname),"END_%s",varname);
ast_channel_lock(chan);
/* For EndIf, simply go to the next priority.
* We do not add 1 to ast_channel_priority because the dialplan will
* auto-increment the priority when we return, so just keep the priority as is.
* For ExitIf or false If() condition, we need to find the end of the current
* If branch (at same indentation) and branch there. */
endifpri = end == 2 ? ast_channel_priority(chan) + 1 : find_matching_endif(chan, NULL);
endifpri = end == 2 ? ast_channel_priority(chan) : find_matching_endif(chan, NULL);
if ((goto_str = pbx_builtin_getvar_helper(chan, end_varname))) {
ast_parseable_goto(chan, goto_str);
pbx_builtin_setvar_helper(chan, end_varname, NULL);