kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-31 02:37:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use built-in parsing functions for Contact and Record-Route headers.

Browse Source 
If a Contact or a Record-Route header had a quoted string with an
item in angle brackets, then we would mis-parse it. For instance,
"Bob <1234>" <1234@example.org>
would be misparsed as having the URI "1234"
The fix for this is to use parsing functions from reqresp_parser.h
since they are heavily tested and are awesome.

(issue ASTERISK-18990)



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@351284 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Mark Michelson
2012-01-17 16:41:23 +00:00
parent ec47280520
commit eae2207967
1 changed files with 13 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
channels/chan_sip.c
View File

@@ -14032,7 +14032,7 @@ static void build_route(struct sip_pvt *p, struct sip_request *req, int backward
struct sip_route *thishop, *head, *tail;
int start = 0;
int len;
const char *rr, *contact, *c;
const char *rr, *c;
/* Once a persistent route is set, don't fool with it */
if (p->route && p->route_persistent) {
@@ -14058,17 +14058,22 @@ static void build_route(struct sip_pvt *p, struct sip_request *req, int backward
/* 1st we pass through all the hops in any Record-Route headers */
for (;;) {
/* Each Record-Route header */
char rr_copy[256];
char *rr_copy_ptr;
char *rr_iter;
rr = __get_header(req, "Record-Route", &start);
if (*rr == '\0') {
break;
}
for (; (rr = strchr(rr, '<')) ; rr += len) { /* Each route entry */
++rr;
len = strcspn(rr, ">") + 1;
ast_copy_string(rr_copy, rr, sizeof(rr_copy));
rr_copy_ptr = rr_copy;
while ((rr_iter = strsep(&rr_copy_ptr, ","))) { /* Each route entry */
char *uri = get_in_brackets(rr_iter);
len = strlen(uri) + 1;
/* Make a struct route */
if ((thishop = ast_malloc(sizeof(*thishop) + len))) {
/* ast_calloc is not needed because all fields are initialized in this block */
ast_copy_string(thishop->hop, rr, len);
ast_copy_string(thishop->hop, uri, len);
ast_debug(2, "build_route: Record-Route hop: <%s>\n", thishop->hop);
/* Link in */
if (backwards) {
@@ -14097,20 +14102,12 @@ static void build_route(struct sip_pvt *p, struct sip_request *req, int backward
if (!head || (!ast_strlen_zero(head->hop) && strstr(head->hop, ";lr") == NULL) ) {
/* 2nd append the Contact: if there is one */
/* Can be multiple Contact headers, comma separated values - we just take the first */
contact = get_header(req, "Contact");
char *contact = ast_strdupa(get_header(req, "Contact"));
if (!ast_strlen_zero(contact)) {
ast_debug(2, "build_route: Contact hop: %s\n", contact);
/* Look for <: delimited address */
c = strchr(contact, '<');
if (c) {
/* Take to > */
++c;
len = strcspn(c, ">") + 1;
} else {
/* No <> - just take the lot */
c = contact;
len = strlen(contact) + 1;
}
c = get_in_brackets(contact);
len = strlen(c) + 1;
if ((thishop = ast_malloc(sizeof(*thishop) + len))) {
/* ast_calloc is not needed because all fields are initialized in this block */
ast_copy_string(thishop->hop, c, len);