From: Kinsey Moore Date: Thu, 29 May 2014 18:33:34 +0000 (+0000) Subject: PBX: Prevent incorrect hint parsing X-Git-Tag: 11.11.0-rc1~45 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=b4042d348f31ae439e08372e6e0661c3a2892648;p=thirdparty%2Fasterisk.git PBX: Prevent incorrect hint parsing Dynamic and pattern matching hints should not be checked for their last known state until they are instantiated by subscribers. (closes issue AFS-56) Reported by: John Hardin Patch AFS-56-pbx.diff submitted by Matt Jordan (license 6283) ........ Merged revisions 414813 from http://svn.asterisk.org/svn/asterisk/branches/1.8 git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@414859 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- diff --git a/main/pbx.c b/main/pbx.c index a8b58de165..117e3102f9 100644 --- a/main/pbx.c +++ b/main/pbx.c @@ -5883,12 +5883,18 @@ static int ast_add_hint(struct ast_exten *e) return -1; } hint_new->exten = e; - hint_new->laststate = ast_extension_state2(e, NULL); - if ((presence_state = extension_presence_state_helper(e, &subtype, &message)) > 0) { - hint_new->last_presence_state = presence_state; - hint_new->last_presence_subtype = subtype; - hint_new->last_presence_message = message; - message = subtype = NULL; + if (strstr(e->app, "${") && e->exten[0] == '_') { + /* The hint is dynamic and hasn't been evaluted yet */ + hint_new->laststate = AST_DEVICE_INVALID; + hint_new->last_presence_state = AST_PRESENCE_INVALID; + } else { + hint_new->laststate = ast_extension_state2(e, NULL); + if ((presence_state = extension_presence_state_helper(e, &subtype, &message)) > 0) { + hint_new->last_presence_state = presence_state; + hint_new->last_presence_subtype = subtype; + hint_new->last_presence_message = message; + message = subtype = NULL; + } } /* Prevent multiple add hints from adding the same hint at the same time. */ @@ -7882,7 +7888,11 @@ static int show_dialplan_helper(int fd, const char *context, const char *exten, struct ast_exten *e; struct ast_include *i; struct ast_ignorepat *ip; +#ifndef LOW_MEMORY + char buf[1024], buf2[1024]; +#else char buf[256], buf2[256]; +#endif int context_info_printed = 0; if (context && strcmp(ast_get_context_name(c), context))