[sr-dev] git:master:632f5439: core/dset: Create Contacts header with path vector as Route header

Module: kamailio Branch: master Commit: 632f54393fe315be3aa2291d4f229fad645a5165 URL: https://github.com/kamailio/kamailio/commit/632f54393fe315be3aa2291d4f229fa… Author: Alex Hermann &lt;alex(a)hexla.nl&gt; Committer: Henning Westerholt &lt;henningw(a)users.noreply.github.com&gt; Date: 2019-02-11T21:42:26+01:00 core/dset: Create Contacts header with path vector as Route header When Kamailio is functioning as a registrar sends a 302 redirect, the registered contacts are sent inside the Contact header. If a contact as a path vector, that path should become a Route header in the INVITE (rfc 3327). This patch adds the path vector as a Route header parameter to the Contacts in the redirect so the receiving entity of the 302 can add the proper Route headers. This is a standard way to specify headers to be added to a request, see rfc3261 section 19.1.1. --- Modified: src/core/dset.c Modified: src/core/dset.h --- Diff: https://github.com/kamailio/kamailio/commit/632f54393fe315be3aa2291d4f229fa… Patch: https://github.com/kamailio/kamailio/commit/632f54393fe315be3aa2291d4f229fa… --- diff --git a/src/core/dset.c b/src/core/dset.c index 239a4f2ebc..d911515e0b 100644 --- a/src/core/dset.c +++ b/src/core/dset.c @@ -39,6 +39,7 @@ #include "dset.h" #include "mem/mem.h" #include "ip_addr.h" +#include "strutils.h" #define CONTACT "Contact: " #define CONTACT_LEN (sizeof(CONTACT) - 1) @@ -49,6 +50,8 @@ #define Q_PARAM ";q=" #define Q_PARAM_LEN (sizeof(Q_PARAM) - 1) +#define ROUTE_PARAM "?Route=" +#define ROUTE_PARAM_LEN (sizeof(ROUTE_PARAM) - 1) /* * Where we store URIs of additional transaction branches @@ -465,6 +468,7 @@ int append_branch(struct sip_msg* msg, str* uri, str* dst_uri, str* path, return 1; } + /*! \brief * Combines the given elements into a Contact header field * dest = target buffer, will be updated to new position after the printed contact @@ -472,7 +476,7 @@ int append_branch(struct sip_msg* msg, str* uri, str* dst_uri, str* path, * end = end of target buffer * Returns 0 on success or -1 on error (buffer is too short) */ -static int print_contact_str(char **dest, str *uri, qvalue_t q, char *end, int options) +static int print_contact_str(char **dest, str *uri, qvalue_t q, str *path, char *end, int options) { char *p = *dest; str buf; @@ -484,6 +488,25 @@ static int print_contact_str(char **dest, str *uri, qvalue_t q, char *end, int o *p++ = '<'; memcpy(p, uri->s, uri->len); p += uri->len; + + /* uri parameters */ + /* path vector as route header parameter */ + if ((options & DS_PATH) && path->len > 0) { + if (p + ROUTE_PARAM_LEN + path->len > end) { + return -1; + } + memcpy(p, ROUTE_PARAM, ROUTE_PARAM_LEN); + p += ROUTE_PARAM_LEN; + /* copy escaped path into dest */ + buf.s = p; + buf.len = end - p; + if (escape_param(path, &buf) < 0) { + return -1; + } + p += buf.len; + } + + /* end of uri parameters */ *p++ = '>'; /* header parameters */ @@ -511,7 +534,7 @@ char* print_dset(struct sip_msg* msg, int* len, int options) { int cnt = 0; qvalue_t q; - str uri; + str uri, path; char *p; int crt_branch; static char dset[MAX_REDIRECTION_LEN]; @@ -529,7 +552,7 @@ char* print_dset(struct sip_msg* msg, int* len, int options) /* current uri */ if (msg->new_uri.s) { - if (print_contact_str(&p, &msg->new_uri, ruri_q, end, options) < 0) { + if (print_contact_str(&p, &msg->new_uri, ruri_q, &msg->path_vec, end, options) < 0) { goto memfail; } cnt++; @@ -537,7 +560,7 @@ char* print_dset(struct sip_msg* msg, int* len, int options) /* branches */ init_branch_iterator(); - while ((uri.s = next_branch(&uri.len, &q, 0, 0, 0, 0, 0, 0, 0))) { + while ((uri.s = next_branch(&uri.len, &q, 0, &path, 0, 0, 0, 0, 0))) { if (cnt > 0) { if (p + CONTACT_DELIM_LEN > end) { goto memfail; @@ -546,7 +569,7 @@ char* print_dset(struct sip_msg* msg, int* len, int options) p += CONTACT_DELIM_LEN; } - if (print_contact_str(&p, &uri, q, end, options) < 0) { + if (print_contact_str(&p, &uri, q, &path, end, options) < 0) { goto memfail; } diff --git a/src/core/dset.h b/src/core/dset.h index b786cee053..e09262c834 100644 --- a/src/core/dset.h +++ b/src/core/dset.h @@ -37,6 +37,8 @@ extern unsigned int nr_branches; extern int ruri_is_new; +#define DS_PATH 2 + /*! \brief * Structure for storing branch attributes */