Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SMSOPS: body < 3 characters not transmitted (empty) #3211

Closed
gmaruzz opened this issue Aug 9, 2022 · 6 comments
Closed

SMSOPS: body < 3 characters not transmitted (empty) #3211

gmaruzz opened this issue Aug 9, 2022 · 6 comments

Comments

@gmaruzz
Copy link

gmaruzz commented Aug 9, 2022

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)

@gmaruzz
Copy link
Author

gmaruzz commented Aug 9, 2022

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?

@henningw
Copy link
Contributor

henningw commented Aug 9, 2022

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

@gmaruzz
Copy link
Author

gmaruzz commented Aug 9, 2022

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

@gmaruzz
Copy link
Author

gmaruzz commented Aug 9, 2022

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

@gmaruzz
Copy link
Author

gmaruzz commented Aug 9, 2022

I confirm, is fixed in 6.1
would be nice to backport the fix
sms_one_character_fixed_in_6.1.zip

@henningw
Copy link
Contributor

henningw commented Aug 9, 2022

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.

@henningw henningw closed this as completed Aug 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants