[sr-dev] git:master:65336276: topos: skip comma at the end when reinserting rr headers

Module: kamailio Branch: master Commit: 65336276f219454210556b36f3d37a0e957d9c1c URL: https://github.com/kamailio/kamailio/commit/65336276f219454210556b36f3d37a0… Author: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt; Committer: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt; Date: 2016-08-05T08:27:41+02:00 topos: skip comma at the end when reinserting rr headers - reported by GH #716 --- Modified: modules/topos/tps_msg.c --- Diff: https://github.com/kamailio/kamailio/commit/65336276f219454210556b36f3d37a0… Patch: https://github.com/kamailio/kamailio/commit/65336276f219454210556b36f3d37a0… --- diff --git a/modules/topos/tps_msg.c b/modules/topos/tps_msg.c index 2f398e8..ecc67ef 100644 --- a/modules/topos/tps_msg.c +++ b/modules/topos/tps_msg.c @@ -629,6 +629,7 @@ int tps_reappend_route(sip_msg_t *msg, tps_data_t *ptsd, str *hbody, int rev) c = 1; if(sb.len>0) { sb.s = hbody->s + i + 1; + if(sb.s[sb.len-1]==',') sb.len--; if(tps_add_headers(msg, &hname, &sb, 0)<0) { return -1; } @@ -640,6 +641,7 @@ int tps_reappend_route(sip_msg_t *msg, tps_data_t *ptsd, str *hbody, int rev) if(c==1) { if(sb.len>0) { sb.s = hbody->s; + if(sb.s[sb.len-1]==',') sb.len--; if(tps_add_headers(msg, &hname, &sb, 0)<0) { return -1; } @@ -648,7 +650,9 @@ int tps_reappend_route(sip_msg_t *msg, tps_data_t *ptsd, str *hbody, int rev) } } - if(tps_add_headers(msg, &hname, hbody, 0)<0) { + sb = *hbody; + if(sb.s[sb.len-1]==',') sb.len--; + if(tps_add_headers(msg, &hname, &sb, 0)<0) { return -1; }