[kamailio/kamailio] SMSOPS: body < 3 characters not transmitted (empty) (Issue #3211) - sr-dev

gmaruzz

9 Aug 2022 9 Aug '22

8:15 a.m.

Hello,

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, SRSRAN)

-- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3211 You are receiving this because you are subscribed to this thread. Message ID: kamailio/kamailio/issues/3211@github.com

Attachments:

attachment.html (text/html — 1.7 KB)

Show replies by date

gmaruzz

9 Aug 9 Aug

8:18 a.m.

I'm using Kamailio 5.3

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?

-- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3211#issuecomment-1209308097 You are receiving this because you are subscribed to this thread. Message ID: kamailio/kamailio/issues/3211/1209308097@github.com

Henning Westerholt

8:27 a.m.

What about just giving a newer version (e.g. 5.6.x) a try? :-)

-- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3211#issuecomment-1209316853 You are receiving this because you are subscribed to this thread. Message ID: kamailio/kamailio/issues/3211/1209316853@github.com

gmaruzz

8:32 a.m.

[sms.zip](https://github.com/kamailio/kamailio/files/9290354/sms.zip) in the zip file two SMSs, one with payload "A" and one with payload "ABC"

-- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3211#issuecomment-1209321514 You are receiving this because you are subscribed to this thread. Message ID: kamailio/kamailio/issues/3211/1209321514@github.com

gmaruzz

8:33 a.m.

(I'll check with newer version, if fixed would be nice to backport as bug)

-- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3211#issuecomment-1209323021 You are receiving this because you are subscribed to this thread. Message ID: kamailio/kamailio/issues/3211/1209323021@github.com

gmaruzz

9:37 a.m.

I confirm, is fixed in 6.1 would be nice to backport the fix [sms_one_character_fixed_in_6.1.zip](https://github.com/kamailio/kamailio/files/9291084/sms_one_character_fixed_i...)

-- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3211#issuecomment-1209393045 You are receiving this because you are subscribed to this thread. Message ID: kamailio/kamailio/issues/3211/1209393045@github.com

Henning Westerholt

2:21 p.m.

You probably mean 5.6.x? Thanks for the confirmation. Regarding backporting the fix, please note that 5.3.x is end of life since about a year. Please consider using a newer release.

-- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3211#issuecomment-1209720204 You are receiving this because you are subscribed to this thread. Message ID: kamailio/kamailio/issues/3211/1209720204@github.com

Henning Westerholt

2:21 p.m.

Closed #3211 as completed.

-- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3211#event-7157853579 You are receiving this because you are subscribed to this thread. Message ID: kamailio/kamailio/issue/3211/issue_event/7157853579@github.com

946

Age (days ago)

946

Last active (days ago)

sr-dev@lists.kamailio.org

7 comments

2 participants

tags (0)

participants (2)

gmaruzz
Henning Westerholt