<!-- Kamailio Project uses GitHub Issues only for bugs in the code or feature requests.
If you have questions about using Kamailio or related to its configuration file, ask on sr-users mailing list:
* http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
If you have questions about developing extensions to Kamailio or its existing C code, ask on sr-dev mailing list
* http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev
Please try to fill this template as much as possible for any issue. It helps the developers to troubleshoot the issue.
If you submit a feature request (or enhancement), you can delete the text of the template and only add the description of what you would like to be added.
If there is no content to be filled in a section, the entire section can be removed.
You can delete the comments from the template sections when filling.
You can delete next line and everything above before submitting (it is a comment). -->
### Description
<!-- Explain what you did, what you expected to happen, and what actually happened. --> When registering a soft client running on my Android phone (Grandstream Wave), Kamailio crashes every time when the client registers. When setting the debug level below 3 Kamailio does not crash. I have only found the problem with this SIP client. Others are working fine.
### Troubleshooting
#### Reproduction
<!-- If the issue can be reproduced, describe how it can be done. --> Set debug level greather than 2 and register the Grandstream Wave soft client from an Android phone. Kamailio then crashes.
#### Log Messages
<!-- Check the syslog file and if there are relevant log messages printed by Kamailio, add them next, or attach to issue, or provide a link to download them (e.g., to a pastebin site). -->
just before crashing I get the following message: 0(7819) ERROR: <core> [core/udp_server.c:484]: udp_rcv_loop(): print buffer building failed (10/16/4)
### Possible Solutions
<!-- If you found a solution or workaround for the issue, describe it. Ideally, provide a pull request with a fix. --> The problem seems to come from non-printable characters sent by the SIP client causing Kamailio to crash.
Changing line 482 in udp_server.c from l = snprintf(printbuf+j, 6, " %02X ", buf[i]); to l = snprintf(printbuf+j, 6, " %02X ", (unsigned char)buf[i]);
fixes the problem.
The compiler treats buf[i] as a signed integer and prepends 'FFFFFF' to the string making it longer than 6 characters. Casting buf[i] as an unsigned char prevents Kamailio from crashing.
### Additional Information
* **Kamailio Version** - output of `kamailio -v`
version: kamailio 5.1.0 (x86_64/linux) flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLACKLIST, HAVE_RESOLV_RES ADAPTIVE_WAIT_LOOPS=1024, MAX_RECV_BUFFER_SIZE 262144, MAX_LISTEN 16, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. id: unknown compiled with gcc 5.3.1
* **Operating System**:
<!-- Details about the operating system, the type: Linux (e.g.,: Debian 8.4, Ubuntu 16.04, CentOS 7.1, ...), MacOS, xBSD, Solaris, ...; Kernel details (output of `uname -a`) -->
Ubuntu 16.04 (Xenial)
Casting to unsigned char is good to do (pushed a patch for it), but snprintf() is not writing if it needs more space than the buffer size parameter (6 in this case) -- that's the reason the log message is printed. So there has to be something else.
Did you get a core dump file? If yes, can you grab the backtrace with gdb and paste it here?
Looking a bit more, I found what was -- jumping to error resulted in going out of loop receiving packets, practically terminating kamailio process (so not a crash, just an unwanted shut down). I pushed the patches to master and 5.1 branches. Reopen if still an issue.
Closed #1403.
Thanks Daniel, it works perfectly now.
I believe this is the same as the issue I reported here:
https://lists.kamailio.org/pipermail/sr-dev/2017-November/043920.html
I'll check with the new patches as soon as I have the chance.