From 1f051c7e2d172ec6dca2dae6be2a978ccec6191c Mon Sep 17 00:00:00 2001 From: Richard Mudgett Date: Fri, 1 Nov 2013 23:52:45 +0000 Subject: [PATCH] config: Allow ConfBridge DTMF menus to have '#' as the first digit. ConfBridge allows custom DTMF menus to be created in the confbridge.conf file by assigning a DTMF key sequence to a sequence of actions as follows: DTMF-sequence = action,action... Unfortunately, the normal config file processing code interprets an initial '#' character as starting a directive such as #include. * Add the ability to escape the first non-blank character in a config line so the '#' character can be used without triggering the directive processing code. (closes issue AFS-2) (closes issue ASTERISK-22478) Reported by: Nicolas Tanski Patches: jira_asterisk_22478_v11.patch (license #5621) patch uploaded by rmudgett (modified) Review: https://reviewboard.asterisk.org/r/2969/ git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/11@402407 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- configs/confbridge.conf.sample | 9 +++++++-- main/config.c | 21 ++++++++++++++++++--- 2 files changed, 25 insertions(+), 5 deletions(-) diff --git a/configs/confbridge.conf.sample b/configs/confbridge.conf.sample index a4f829aa2c..94739a6fe5 100644 --- a/configs/confbridge.conf.sample +++ b/configs/confbridge.conf.sample @@ -62,7 +62,7 @@ type=user ; loose the user will hear themselves briefly each ; time they begin talking until the dsp has time to ; establish that they are in fact talking. - ; 2. When talk detection AMI events are enabled, this value + ; 2. When talk detection AMI events are enabled, this value ; determines when talking has begun which results in ; an AMI event to fire. If this value is set too tight ; AMI events may be falsely triggered by variants in @@ -248,6 +248,11 @@ type=bridge ; Below is a list of menu actions that can be assigned ; to a DTMF sequence. ; +; To have the first DTMF digit in a sequence be the '#' character, you need to +; escape it. If it is not escaped then normal config file processing will +; think it is a directive like #include. For example: +; \#1=toggle_mute ; Pressing #1 will toggle the mute setting. +; ; A single DTMF sequence can have multiple actions associated with it. This is ; accomplished by stringing the actions together and using a ',' as the delimiter. ; Example: Both listening and talking volume is reset when '5' is pressed. @@ -270,7 +275,7 @@ type=bridge ; using the '&' character as a delimiter. ; toggle_mute ; Toggle turning on and off mute. Mute will make the user silent ; to everyone else, but the user will still be able to listen in. - ; continue to collect the dtmf sequence. + ; no_op ; This action does nothing (No Operation). Its only real purpose exists for ; being able to reserve a sequence in the config as a menu exit sequence. ; decrease_listening_volume ; Decreases the channel's listening volume. diff --git a/main/config.c b/main/config.c index 6b50b70223..25d483c2c6 100644 --- a/main/config.c +++ b/main/config.c @@ -1419,14 +1419,26 @@ static int process_text_line(struct ast_config *cfg, struct ast_category **cat, } else { /* Just a line (variable = value) */ int object = 0; + int is_escaped; + if (!(*cat)) { ast_log(LOG_WARNING, "parse error: No category context for line %d of %s\n", lineno, configfile); return -1; } - c = strchr(cur, '='); - if (c && c > cur && (*(c - 1) == '+')) { + is_escaped = cur[0] == '\\'; + if (is_escaped) { + /* First character is escaped. */ + ++cur; + if (cur[0] < 33) { + ast_log(LOG_ERROR, "Invalid escape in line %d of %s\n", lineno, configfile); + return -1; + } + } + c = strchr(cur + is_escaped, '='); + + if (c && c > cur + is_escaped && (*(c - 1) == '+')) { struct ast_variable *var, *replace = NULL; struct ast_str **str = ast_threadstorage_get(&appendbuf, sizeof(*str)); @@ -1462,8 +1474,11 @@ static int process_text_line(struct ast_config *cfg, struct ast_category **cat, object = 1; c++; } + cur = ast_strip(cur); set_new_variable: - if ((v = ast_variable_new(ast_strip(cur), ast_strip(c), S_OR(suggested_include_file, cfg->include_level == 1 ? "" : configfile)))) { + if (ast_strlen_zero(cur)) { + ast_log(LOG_WARNING, "No variable name in line %d of %s\n", lineno, configfile); + } else if ((v = ast_variable_new(cur, ast_strip(c), S_OR(suggested_include_file, cfg->include_level == 1 ? "" : configfile)))) { v->lineno = lineno; v->object = object; *last_cat = 0; -- 2.47.2