diff options
author | Tilghman Lesher <tilghman@meg.abyt.es> | 2010-09-21 22:58:10 +0000 |
---|---|---|
committer | Tilghman Lesher <tilghman@meg.abyt.es> | 2010-09-21 22:58:10 +0000 |
commit | 949e81e6e5c2e6a5e535828bf8da30a445676c73 (patch) | |
tree | 6bae398b28ea6576ae7662cb6094576ffee1fdab /channels | |
parent | b287e9310123f2dea6b169d601c460893b8b84f5 (diff) |
Merged revisions 288159 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8
................
r288159 | tilghman | 2010-09-21 17:57:22 -0500 (Tue, 21 Sep 2010) | 29 lines
Merged revisions 288113 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2
................
r288113 | tilghman | 2010-09-21 16:59:46 -0500 (Tue, 21 Sep 2010) | 22 lines
Merged revisions 288112 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r288112 | tilghman | 2010-09-21 16:58:13 -0500 (Tue, 21 Sep 2010) | 15 lines
Try both the encoded and unencoded subscription URI for a match in hints.
When a phone sends an encoded URI for a subscription, the URI is not matched
with the actual hint that is in decoded format. For example, if we have an
extension with a hint that is named: "#5601" or "*5601", the subscription will
work fine if the phone subscribes with an already decoded URI, but when it's
decoded like "%255601" or "%2A5601", Asterisk is unable to match it with the
correct hint.
(closes issue #17785)
Reported by: ramonpeek
Patches:
20100831__issue17785.diff.txt uploaded by tilghman (license 14)
Tested by: ramonpeek
........
................
................
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@288160 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'channels')
-rw-r--r-- | channels/chan_sip.c | 27 |
1 files changed, 18 insertions, 9 deletions
diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 120dd1d66..3e4524db6 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -14364,26 +14364,35 @@ static enum sip_get_dest_result get_destination(struct sip_pvt *p, struct sip_re ast_verbose("Looking for %s in %s (domain %s)\n", uri, p->context, p->domain); } + /* Since extensions.conf can have unescaped characters, try matching a + * decoded uri in addition to the non-decoded uri. */ + decoded_uri = ast_strdupa(uri); + ast_uri_decode(decoded_uri); + /* If this is a subscription we actually just need to see if a hint exists for the extension */ if (req->method == SIP_SUBSCRIBE) { char hint[AST_MAX_EXTENSION]; - return (ast_get_hint(hint, sizeof(hint), NULL, 0, NULL, p->context, p->exten) ? - SIP_GET_DEST_EXTEN_FOUND : - SIP_GET_DEST_EXTEN_NOT_FOUND); + int which = 0; + if (ast_get_hint(hint, sizeof(hint), NULL, 0, NULL, p->context, uri) || + (ast_get_hint(hint, sizeof(hint), NULL, 0, NULL, p->context, decoded_uri) && (which = 1))) { + if (!oreq) { + ast_string_field_set(p, exten, which ? decoded_uri : uri); + } + return SIP_GET_DEST_EXTEN_FOUND; + } else { + return SIP_GET_DEST_EXTEN_NOT_FOUND; + } } else { struct ast_cc_agent *agent; - decoded_uri = ast_strdupa(uri); - ast_uri_decode(decoded_uri); + int which = 0; /* Check the dialplan for the username part of the request URI, the domain will be stored in the SIPDOMAIN variable - Since extensions.conf can have unescaped characters, try matching a decoded - uri in addition to the non-decoded uri Return 0 if we have a matching extension */ if (ast_exists_extension(NULL, p->context, uri, 1, S_OR(p->cid_num, from)) || - ast_exists_extension(NULL, p->context, decoded_uri, 1, S_OR(p->cid_num, from)) || + (ast_exists_extension(NULL, p->context, decoded_uri, 1, S_OR(p->cid_num, from)) && (which = 1)) || !strcmp(decoded_uri, ast_pickup_ext())) { if (!oreq) { - ast_string_field_set(p, exten, decoded_uri); + ast_string_field_set(p, exten, which ? decoded_uri : uri); } return SIP_GET_DEST_EXTEN_FOUND; } else if ((agent = find_sip_cc_agent_by_notify_uri(tmp))) { |