sysmocom
/
asterisk
11
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

app_queue: Fix hint updates for included contexts 

Previously, if custom hints were used with the hint:
format in app_queue, when device state changes occured,
app_queue would only do a literal string comparison of
the context used for the hint in app_queue and the context
of the hint which just changed state. This caused hints
to not update and become stale if the context associated
with the agent included the context which actually changes
state, essentially completely breaking device state for
any such agents defined in this manner.

This fix adds an additional check to ensure that included
contexts are also compared against the context which changed
state, so that the behavior is correct no matter whether the
context is specified to app_queue directly or indirectly.

ASTERISK-29578 #close

Change-Id: I8caf2f8da8157ef3d9ea71a8568c1eec95592b78
This commit is contained in:
Naveen Albert 2021-09-05 18:11:57 +00:00 committed by Friendly Automation
parent b2c834e349
commit cfd0246d11
1 changed files with 47 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

49
apps/app_queue.c
View File

@ -2630,6 +2630,46 @@ static int extensionstate2devicestate(int state)
return state;
}
/*!
* \brief Returns if one context includes another context
*
* \param parent Parent context to search for child
* \param child Context to check for inclusion in parent
*
* This function recrusively checks if the context child is included in the context parent.
*
* \return 1 if child is included in parent, 0 if not
*/
static int context_included(const char *parent, const char *child);
static int context_included(const char *parent, const char *child)
{
struct ast_context *c = NULL;
c = ast_context_find(parent);
if (!c) {
/* well, if parent doesn't exist, how can the child be included in it? */
return 0;
}
if (!strcmp(ast_get_context_name(c), parent)) {
/* found the context of the hint app_queue is using. Now, see
if that context includes the one that just changed state */
struct ast_include *inc = NULL;
while ((inc = (struct ast_include*) ast_walk_context_includes(c, inc))) {
const char *includename = ast_get_include_name(inc);
if (!strcasecmp(child, includename)) {
return 1;
}
/* recurse on this context, for nested includes. The
PBX extension parser will prevent infinite recursion. */
if (context_included(includename, child)) {
return 1;
}
}
}
return 0;
}
static int extension_state_cb(const char *context, const char *exten, struct ast_state_cb_info *info, void *data)
{
struct ao2_iterator miter, qiter;
@ -2650,9 +2690,14 @@ static int extension_state_cb(const char *context, const char *exten, struct ast
miter = ao2_iterator_init(q->members, 0);
for (; (m = ao2_iterator_next(&miter)); ao2_ref(m, -1)) {
if (!strcmp(m->state_context, context) && !strcmp(m->state_exten, exten)) {
found = 1;
} else if (!strcmp(m->state_exten, exten) && context_included(m->state_context, context)) {
/* context could be included in m->state_context. We need to check. */
found = 1;
}
if (found) {
update_status(q, m, device_state);
ao2_ref(m, -1);
found = 1;
break;
}
}
@ -2663,7 +2708,7 @@ static int extension_state_cb(const char *context, const char *exten, struct ast
}
ao2_iterator_destroy(&qiter);
if (found) {
if (found) {
ast_debug(1, "Extension '%s@%s' changed to state '%d' (%s)\n", exten, context, device_state, ast_devstate2str(device_state));
} else {
ast_debug(3, "Extension '%s@%s' changed to state '%d' (%s) but we don't care because they're not a member of any queue.\n",