ben.carrasco/freeswitch
1
0
Fork 0
mirror of https://github.com/signalwire/freeswitch.git synced 2026-10-03 17:53:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

[mod_sofia] Bound the To-user split in the inbound MESSAGE handler (#3134)

Browse Source 
`sofia_presence_handle_sip_i_message()` splits the chat proto selector off a
truncating copy of the To user. Split only a user that fits the buffer; an
over-long one is handled as a plain address.
This commit is contained in:
Dmitry Verenitsin
2026-08-26 16:01:18 +03:00
committed by GitHub
parent cc93a683ca
commit 098018df3d
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/mod/endpoints/mod_sofia/sofia_presence.c
+4 -1
View File
@@ -4871,7 +4871,10 @@ void sofia_presence_handle_sip_i_message(int status,
full_from = sip_header_as_string(nua_handle_get_home(nh), (void *) sip->sip_from);
if ((p = strchr(to_user, '+')) && p != to_user) {
/* Only split a user that fits in proto: the copy below truncates, so a '+' past the
buffer would leave nothing to split on, and a selector or address cut in half must
not be parsed either. An over-long user is treated as a plain address. */
if ((p = strchr(to_user, '+')) && p != to_user && strlen(to_user) < sizeof(proto)) {
switch_copy_string(proto, to_user, sizeof(proto));
p = strchr(proto, '+');
*p++ = '\0';