[sr-dev] [kamailio/kamailio] 4.4.x TOPOS - parse_rr() error when using topos (#716)

[darencrew]

After more debugging, it seems that the Route header is not well regenerated from topos stored values.

There is a wrong additional comma at the end of the `Route` header:

`<sip:A.A.35.90;r2=on;lr;ftag=SD9pnva01-8c51e3a9-0001-0313-0000-0000>,`

Instead of

`<sip:A.A.35.90;r2=on;lr;ftag=SD9pnva01-8c51e3a9-0001-0313-0000-0000>`

I tried to reduce sb size by replacing last character with \0, that makes it work

`*(sb.s-1)='\0';`

between these two lines
	
```
638		sb.len++;
639	}
```

 but leads, sometimes, to segmentation faults...

There is probably another way to achieve a correct header regeneration, but i don't know how.

Please let me know if i can give you more informations

---
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/716#issuecomment-236642671
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-dev/attachments/20160801/6f35761b/attachment.html>