docker_open5gs icon indicating copy to clipboard operation
docker_open5gs copied to clipboard
verified publisher icon herlesupreeth

Kamailio bug in SMSOPS: body < 3 characters not transmitted (empty)

Open gmaruzz opened this issue 3 years ago • 4 comments

as per: https://github.com/kamailio/kamailio/issues/3211

seems to me that if cellphone sends an SMS shorter than 3 characters (eg: "aa") it is sent as empty string

this maybe because of some snafu in PDU "payload" management in smsops_impl.c , pointer arithmetic between different characters' coding (ASCII, UTF8, UCS2, etc)

(tested with OnePlus 5t, 6t, Open5GS, SRSLTE, USRP201)

sms.zip in the zip file two SMSs, one with payload "A" and one with payload "ABC"

I see after 5.3 there are at least 3 new commits related to payload management... maybe is already fixed? can you check it out?

https://github.com/kamailio/kamailio/commits/master/src/modules/smsops

gmaruzz avatar Aug 09 '22 12:08 gmaruzz

I confirm, is fixed in Kamailio 5.6.1 would be nice to backport the fix

sms_one_character_fixed_in_6.1.zip

gmaruzz avatar Aug 09 '22 13:08 gmaruzz

@gmaruzz thank you very much for pointing out..will try to backport the fix onto 5.3 in coming days

herlesupreeth avatar Aug 09 '22 15:08 herlesupreeth

thanks to you!

gmaruzz avatar Aug 09 '22 21:08 gmaruzz

It's done. Can you please re-compile kamailio base image and give it a try? Thanks a lot

herlesupreeth avatar Aug 10 '22 06:08 herlesupreeth

works very well, thanks!

gmaruzz avatar Aug 24 '22 15:08 gmaruzz