diff options
| author | Matthew Nicholson <mnicholson@digium.com> | 2011-01-24 18:59:22 +0000 |
|---|---|---|
| committer | Matthew Nicholson <mnicholson@digium.com> | 2011-01-24 18:59:22 +0000 |
| commit | e706b5706e1ed503569c28c7a374a31063bf4244 (patch) | |
| tree | 9e6c6ce3c4de14d886a8c131a7621a792d457728 /res/res_agi.c | |
| parent | 54f6c31a27319d768da73afa966423e1083bb486 (diff) | |
According to section 19.1.2 of RFC 3261:
For each component, the set of valid BNF expansions defines exactly
which characters may appear unescaped. All other characters MUST be
escaped.
This patch modifies ast_uri_encode() to encode strings in line with this recommendation. This patch also adds an ast_escape_quoted() function which escapes '"' and '\' characters in quoted strings in accordance with section 25.1 of RFC 3261. The ast_uri_encode() function has also been modified to take an ast_flags struct describing the set of rules it should use when escaping characters to allow for it to escape SIP URIs in addition to HTTP URIs and other types of URIs or variations of those two URI types in the future.
The ast_uri_decode() function has also been modified to accept an ast_flags struct describing the set of rules to use when decoding to enable decoding '+' as ' ' in legacy http URLs.
The unit tests for these functions have also been updated.
ABE-2705
Review: https://reviewboard.asterisk.org/r/1081/
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@303509 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/res_agi.c')
| -rw-r--r-- | res/res_agi.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/res/res_agi.c b/res/res_agi.c index cd4760b41..7dc2d20c9 100644 --- a/res/res_agi.c +++ b/res/res_agi.c @@ -1254,7 +1254,7 @@ static enum agi_result launch_asyncagi(struct ast_channel *chan, char *argv[], i /* encode it and send it thru the manager so whoever is going to take care of AGI commands on this channel can decide which AGI commands to execute based on the setup info */ - ast_uri_encode(agi_buffer, ami_buffer, AMI_BUF_SIZE, 1); + ast_uri_encode(agi_buffer, ami_buffer, AMI_BUF_SIZE, ast_uri_http); manager_event(EVENT_FLAG_AGI, "AsyncAGI", "SubEvent: Start\r\nChannel: %s\r\nEnv: %s\r\n", chan->name, ami_buffer); while (1) { /* bail out if we need to hangup */ @@ -1286,7 +1286,7 @@ static enum agi_result launch_asyncagi(struct ast_channel *chan, char *argv[], i manager. Include the CommandID if it was specified when the command was added */ agi_buffer[res] = '\0'; - ast_uri_encode(agi_buffer, ami_buffer, AMI_BUF_SIZE, 1); + ast_uri_encode(agi_buffer, ami_buffer, AMI_BUF_SIZE, ast_uri_http); if (ast_strlen_zero(cmd->cmd_id)) manager_event(EVENT_FLAG_AGI, "AsyncAGI", "SubEvent: Exec\r\nChannel: %s\r\nResult: %s\r\n", chan->name, ami_buffer); else |