Skip to content

Commit 96c8e2d

Browse files
authored
Merge pull request #3448 from vladpaiu/fix_compare_uri
Fix comparision of SIP URIs, add GR and PN related params checks
2 parents de50249 + 8fa7bcb commit 96c8e2d

File tree

2 files changed

+8
-0
lines changed

2 files changed

+8
-0
lines changed

parser/msg_parser.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -208,6 +208,9 @@ struct sip_uri {
208208
str pn_prid_val;
209209
str pn_param_val;
210210
str pn_purr_val;
211+
/* XXX - in the future when adding params as special links
212+
* in the list above, make sure to also update compare_uris() function
213+
* to explicitly compare these here */
211214

212215
/* unknown params */
213216
str u_name[URI_MAX_U_PARAMS]; /* Unknown param names */

parser/parse_uri.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1818,6 +1818,11 @@ int compare_uris(str *raw_uri_a,struct sip_uri* parsed_uri_a,
18181818
compare_uri_val(method_val,strncasecmp);
18191819
compare_uri_val(lr_val,strncasecmp);
18201820
compare_uri_val(r2_val,strncasecmp);
1821+
compare_uri_val(gr_val,strncasecmp);
1822+
compare_uri_val(pn_provider_val,strncasecmp);
1823+
compare_uri_val(pn_prid_val,strncasecmp);
1824+
compare_uri_val(pn_param_val,strncasecmp);
1825+
compare_uri_val(pn_purr_val,strncasecmp);
18211826

18221827
if (first.u_params_no == 0 || second.u_params_no == 0)
18231828
/* one URI doesn't have other params,

0 commit comments

Comments
 (0)