sr-dev January 2019

sr-dev@lists.kamailio.org

27 participants
301 discussions

[kamailio/kamailio] TLS module: Unable to load tls module (#1797)

by com4bizz

ERROR: tls [tls_init.c:601]: tls_pre_init(): Unable to set the memory allocation functions This error comes with 5.0x - 5.2x and maybe in earlyer versions too. Tried OpenSSL 1.0, and 1.1 with same result... -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1797

5 years, 11 months

[kamailio/kamailio] Segmentation fault in TM module while processing SIP status 408 (#1806)

by Muhammad Shahzad Shafi

### Description We have segfault in Kamailio v5.0.7 rev. 7ab0b1 installed on Debain 7.x 32bit KVM when processing sip reply 408 due to RING Timeout.  ### Troubleshooting No troubleshooting was done, since it happened on a production server. We simply restarted the server. #### Reproduction The problem is random and has happened a couple of times within a month.  #### Debugging Data Here is back trace from core dump generated by kamailio. <pre> Core was generated by `/usr/local/adx-webrtc/sbin/kamailio -f /usr/local/adx-webrtc/etc/kamailio/kamai'. Program terminated with signal 11, Segmentation fault. #0 0xb4f9bcb9 in run_failure_handlers (t=0x92d6111c, rpl=0xffffffff, code=408, extra_flags=96) at t_reply.c:1013 1013 t_reply.c: No such file or directory. (gdb) bt #0 0xb4f9bcb9 in run_failure_handlers (t=0x92d6111c, rpl=0xffffffff, code=408, extra_flags=96) at t_reply.c:1013 #1 0xb4f9ea32 in t_should_relay_response (Trans=0x92d6111c, new_code=408, branch=0, should_store=0xbf90fba4, should_relay=0xbf90fba8, cancel_data=0xbf90fc28, reply=0xffffffff) at t_reply.c:1382 #2 0xb4fa1431 in relay_reply (t=0x92d6111c, p_msg=0xffffffff, branch=0, msg_status=408, cancel_data=0xbf90fc28, do_put_on_wait=0) at t_reply.c:1785 #3 0xb4f4bbca in fake_reply (t=0x92d6111c, branch=0, code=408) at timer.c:340 #4 0xb4f4bfe7 in final_response_handler (r_buf=0x92d61288, t=0x92d6111c) at timer.c:506 #5 0xb4f4c07e in retr_buf_handler (ticks=368965158, tl=0x92d6129c, p=0xfffffffe) at timer.c:562 #6 0x08250eb4 in slow_timer_main () at core/timer.c:1131 #7 0x08069a4e in main_loop () at main.c:1679 #8 0x08070868 in main (argc=13, argv=0xbf9103a4) at main.c:2642 </pre> Here is full back trace.  ``` (gdb) bt full #0 0xb4f9bcb9 in run_failure_handlers (t=0x92d6111c, rpl=0xffffffff, code=408, extra_flags=96) at t_reply.c:1013 faked_req = 0x984311f4 faked_req_len = 4512 shmem_msg = 0x94ed18b8 on_failure = 2 keng = 0x0 __FUNCTION__ = "run_failure_handlers" #1 0xb4f9ea32 in t_should_relay_response (Trans=0x92d6111c, new_code=408, branch=0, should_store=0xbf90fba4, should_relay=0xbf90fba8, cancel_data=0xbf90fc28, reply=0xffffffff) at t_reply.c:1382 branch_cnt = 1 picked_code = 408 new_branch = -1755505652 inv_through = 0 extra_flags = 96 i = 0 replies_dropped = 0 __FUNCTION__ = "t_should_relay_response" #2 0xb4fa1431 in relay_reply (t=0x92d6111c, p_msg=0xffffffff, branch=0, msg_status=408, cancel_data=0xbf90fc28, do_put_on_wait=0) at t_reply.c:1785 relay = -65536 save_clone = 0 buf = 0x0 res_len = 0 relayed_code = 0 relayed_msg = 0x0 reply_bak = 0xb5002368 bm = {to_tag_val = {s = 0xb5a847f7 "ation", len = 10}} totag_retr = 0 reply_status = RPS_ERROR uas_rb = 0x0 to_tag = 0x0 reason = {s = 0x0, len = 1946659428} onsend_params = {req = 0xb5002368, rpl = 0x0, param = 0xbf910234, code = -1081017352, flags = 56659, branch = 46322, t_rbuf = 0xb4fd5a10, dst = 0x2, send_buf = { s = 0xbf90fce8 "\030\375\220\277\034\021֒\210\022֒\240", len = 1946588245}} ip = {af = 0, len = 3213949832, u = {addrl = {4294967295, 0, 3213951540, 3213949832}, addr32 = {4294967295, 0, 3213951540, 3213949832}, addr16 = {65535, 65535, 0, 0, 564, 49041, 64392, 49040}, addr = "\377\377\377\377\000\000\000\000\064\002\221\277\210", <incomplete sequence \373\220\277>}} __FUNCTION__ = "relay_reply" #3 0xb4f4bbca in fake_reply (t=0x92d6111c, branch=0, code=408) at timer.c:340 cancel_data = {cancel_bitmap = 0, reason = {cause = 0, u = {text = {s = 0x0, len = 5}, e2e_cancel = 0x0, packed_hdrs = {s = 0x0, len = 5}}}} do_cancel_branch = 1 reply_status = 29068 #4 0xb4f4bfe7 in final_response_handler (r_buf=0x92d61288, t=0x92d6111c) at timer.c:506 silent = 0 branch_ret = -1258282136 prev_branch = 0 now = 0 #5 0xb4f4c07e in retr_buf_handler (ticks=368965158, tl=0x92d6129c, p=0xfffffffe) at timer.c:562 rbuf = 0x92d61288 fr_remainder = 0 retr_remainder = 12 retr_interval = 1674326491 new_retr_interval_ms = 160 crt_retr_interval_ms = 3213950232 t = 0x92d6111c __FUNCTION__ = "retr_buf_handler" #6 0x08250eb4 in slow_timer_main () at core/timer.c:1131 n = 12 ret = 0 tl = 0x92d6129c i = 516 __FUNCTION__ = "slow_timer_main" #7 0x08069a4e in main_loop () at main.c:1679 i = 4 pid = 0 si = 0x0 si_desc = "udp receiver child=3 sock=xx.xx.xx.xx:5060\000\000\000\000\000\004\000\000\000\030\000\221\277\333\061\314c\001\000\000\000\333\061\314c\230\377\220\277\264\n(\bd\024<t\004\000\000\000\331\332\066\b\260\354\066\bq\000\000\000t\331\066\b\v\020\000\000Y\222\350\264D\221\257\265;\031B\264\\\"C\264\214#\000\000\000\000\000" nrprocs = 4 woneinit = 1 __FUNCTION__ = "main_loop" #8 0x08070868 in main (argc=13, argv=0xbf9103a4) at main.c:2642 cfg_stream = 0x8a4a008 c = -1 r = 0 tmp = 0xbf910903 "" tmp_len = -1218121696 port = 2209 proto = 1 options = 0x8344f9c ":f:cm:M:dVIhEeb:l:L:n:vKrRDTN:W:w:t:u:g:P:G:SQ:O:a:A:x:X:Y:" ret = -1 seed = 3093231387 rfd = 4 debug_save = 0 debug_flag = 0 dont_fork_cnt = 0 n_lst = 0xbf9103a4 p = 0x805d60c "[\201\303\354\253<" st = {st_dev = 14, __pad1 = 0, st_ino = 10259, st_mode = 16832, st_nlink = 2, st_uid = 0, st_gid = 0, st_rdev = 0, __pad2 = 0, st_size = 60, st_blksize = 4096, st_blocks = 0, st_atim = { tv_sec = 1542580403, tv_nsec = 128163439}, st_mtim = {tv_sec = 1542580752, tv_nsec = 236241520}, st_ctim = {tv_sec = 1542580752, tv_nsec = 236241520}, __unused4 = 0, __unused5 = 0} __FUNCTION__ = "main" ``` #### Log Messages No logs available since it happend on a production server.  ``` Jan 10 16:00:53 webrtc-as kernel: [25983771.956320] kamailio[29068]: segfault at 36c ip b4f9bcb9 sp bf90f7a0 error 6 in tm.so[b4eeb000+117000] ``` #### SIP Traffic No SIP traffic available.  ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.0.7 (i386/linux) 7ab0b1 flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, 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, select. id: 7ab0b1 compiled on 22:43:08 Aug 27 2018 with gcc 4.7.2 ``` * **Operating System**:  ``` Linux webrtc-as1 3.16.0-0.bpo.4-686-pae #1 SMP Debian 3.16.36-1+deb8u2~bpo70+1 (2016-10-19) i686 GNU/Linux ``` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/1806

5 years, 11 months

git:5.1:8a21170c: siptrace: don't calculate message source socket if already set

by Federico Cabiddu

Module: kamailio Branch: 5.1 Commit: 8a21170c3a5283b84e0ff3cbc6db8c72413dec05 URL: https://github.com/kamailio/kamailio/commit/8a21170c3a5283b84e0ff3cbc6db8c7… Author: Federico Cabiddu <fcabiddu(a)libon.com> Committer: Federico Cabiddu <fcabiddu(a)libon.com> Date: 2019-01-17T09:43:47+01:00 siptrace: don't calculate message source socket if already set (cherry picked from commit 929a24f08aeef6b72999ac9b188ffeded40759a9) --- Modified: src/modules/siptrace/siptrace.c --- Diff: https://github.com/kamailio/kamailio/commit/8a21170c3a5283b84e0ff3cbc6db8c7… Patch: https://github.com/kamailio/kamailio/commit/8a21170c3a5283b84e0ff3cbc6db8c7… --- diff --git a/src/modules/siptrace/siptrace.c b/src/modules/siptrace/siptrace.c index 035ace455b..1e26bc2131 100644 --- a/src/modules/siptrace/siptrace.c +++ b/src/modules/siptrace/siptrace.c @@ -1520,7 +1520,10 @@ int siptrace_net_data_send(sr_event_param_t *evp) return -1; new_dst = *nd->dst; - new_dst.send_sock = get_send_socket(0, &nd->dst->to, nd->dst->proto); + + if(new_dst.send_sock == 0) { + new_dst.send_sock = get_send_socket(0, &nd->dst->to, nd->dst->proto); + } memset(&sto, 0, sizeof(siptrace_data_t));

5 years, 11 months

git:5.2:b7193895: siptrace: don't calculate message source socket if already set

by Federico Cabiddu

Module: kamailio Branch: 5.2 Commit: b71938952e1fc7543fa5bf7ba2abed6e39167150 URL: https://github.com/kamailio/kamailio/commit/b71938952e1fc7543fa5bf7ba2abed6… Author: Federico Cabiddu <fcabiddu(a)libon.com> Committer: Federico Cabiddu <fcabiddu(a)libon.com> Date: 2019-01-17T09:43:29+01:00 siptrace: don't calculate message source socket if already set (cherry picked from commit 929a24f08aeef6b72999ac9b188ffeded40759a9) --- Modified: src/modules/siptrace/siptrace.c --- Diff: https://github.com/kamailio/kamailio/commit/b71938952e1fc7543fa5bf7ba2abed6… Patch: https://github.com/kamailio/kamailio/commit/b71938952e1fc7543fa5bf7ba2abed6… --- diff --git a/src/modules/siptrace/siptrace.c b/src/modules/siptrace/siptrace.c index 7de96b4b5d..521ff113ab 100644 --- a/src/modules/siptrace/siptrace.c +++ b/src/modules/siptrace/siptrace.c @@ -1492,7 +1492,10 @@ int siptrace_net_data_send(sr_event_param_t *evp) return -1; new_dst = *nd->dst; - new_dst.send_sock = get_send_socket(0, &nd->dst->to, nd->dst->proto); + + if(new_dst.send_sock == 0) { + new_dst.send_sock = get_send_socket(0, &nd->dst->to, nd->dst->proto); + } memset(&sto, 0, sizeof(siptrace_data_t));

5 years, 11 months

git:master:79494842: Merge pull request #1810 from kamailio/grumvalski/siptrace_src_socket

by GitHub

Module: kamailio Branch: master Commit: 7949484246f38dc284d994b148c917f16ec8b880 URL: https://github.com/kamailio/kamailio/commit/7949484246f38dc284d994b148c917f… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2019-01-17T09:41:42+01:00 Merge pull request #1810 from kamailio/grumvalski/siptrace_src_socket siptrace: siptrace: don't calculate message source socket if already set --- Modified: src/modules/siptrace/siptrace.c --- Diff: https://github.com/kamailio/kamailio/commit/7949484246f38dc284d994b148c917f… Patch: https://github.com/kamailio/kamailio/commit/7949484246f38dc284d994b148c917f… --- diff --git a/src/modules/siptrace/siptrace.c b/src/modules/siptrace/siptrace.c index 8957c22fab..05a6310f40 100644 --- a/src/modules/siptrace/siptrace.c +++ b/src/modules/siptrace/siptrace.c @@ -1493,7 +1493,10 @@ int siptrace_net_data_send(sr_event_param_t *evp) return -1; new_dst = *nd->dst; - new_dst.send_sock = get_send_socket(0, &nd->dst->to, nd->dst->proto); + + if(new_dst.send_sock == 0) { + new_dst.send_sock = get_send_socket(0, &nd->dst->to, nd->dst->proto); + } memset(&sto, 0, sizeof(siptrace_data_t));

5 years, 11 months

git:master:929a24f0: siptrace: don't calculate message source socket if already set

by Federico Cabiddu

Module: kamailio Branch: master Commit: 929a24f08aeef6b72999ac9b188ffeded40759a9 URL: https://github.com/kamailio/kamailio/commit/929a24f08aeef6b72999ac9b188ffed… Author: Federico Cabiddu <fcabiddu(a)libon.com> Committer: Federico Cabiddu <fcabiddu(a)libon.com> Date: 2019-01-16T18:09:42+01:00 siptrace: don't calculate message source socket if already set --- Modified: src/modules/siptrace/siptrace.c --- Diff: https://github.com/kamailio/kamailio/commit/929a24f08aeef6b72999ac9b188ffed… Patch: https://github.com/kamailio/kamailio/commit/929a24f08aeef6b72999ac9b188ffed… --- diff --git a/src/modules/siptrace/siptrace.c b/src/modules/siptrace/siptrace.c index 8957c22fab..05a6310f40 100644 --- a/src/modules/siptrace/siptrace.c +++ b/src/modules/siptrace/siptrace.c @@ -1493,7 +1493,10 @@ int siptrace_net_data_send(sr_event_param_t *evp) return -1; new_dst = *nd->dst; - new_dst.send_sock = get_send_socket(0, &nd->dst->to, nd->dst->proto); + + if(new_dst.send_sock == 0) { + new_dst.send_sock = get_send_socket(0, &nd->dst->to, nd->dst->proto); + } memset(&sto, 0, sizeof(siptrace_data_t));

5 years, 11 months

[kamailio/kamailio] siptrace: siptrace: don't calculate message source socket if already set (#1810)

by Federico Cabiddu

When trace_mode = 1, the source socket of an outgoing message can be retrieved from the payload passed in the callback of SREV_NET_DATA_SEND. Otherwise, if the socket has been manually forced (setting $fs, calling force_send_socket..), the information will be overwritten and the actual source socket of the outgoing message lost. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1810 -- Commit Summary -- * siptrace: siptrace: don't calculate message source socket if already set -- File Changes -- M src/modules/siptrace/siptrace.c (5) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1810.patch https://github.com/kamailio/kamailio/pull/1810.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/1810

5 years, 11 months

git:master:4197b52f: presence_xml: clang format the code

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 4197b52f313b75abd32592b78a5faf9b23ddbf1b URL: https://github.com/kamailio/kamailio/commit/4197b52f313b75abd32592b78a5faf9… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-01-17T08:53:18+01:00 presence_xml: clang format the code --- Modified: src/modules/presence_xml/add_events.c Modified: src/modules/presence_xml/add_events.h Modified: src/modules/presence_xml/api.h Modified: src/modules/presence_xml/notify_body.c Modified: src/modules/presence_xml/notify_body.h Modified: src/modules/presence_xml/pidf.c Modified: src/modules/presence_xml/pidf.h Modified: src/modules/presence_xml/pres_check.c Modified: src/modules/presence_xml/pres_check.h Modified: src/modules/presence_xml/presence_xml.c Modified: src/modules/presence_xml/presence_xml.h Modified: src/modules/presence_xml/xcap_auth.c Modified: src/modules/presence_xml/xcap_auth.h --- Diff: https://github.com/kamailio/kamailio/commit/4197b52f313b75abd32592b78a5faf9… Patch: https://github.com/kamailio/kamailio/commit/4197b52f313b75abd32592b78a5faf9…

5 years, 11 months

git:master:a9331b65: presence_xml: include guard for pres_check.h, declare extern variables in .h

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: a9331b659eb767e8ae19f1ae2adb0ecbfbc71c13 URL: https://github.com/kamailio/kamailio/commit/a9331b659eb767e8ae19f1ae2adb0ec… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-01-17T08:45:06+01:00 presence_xml: include guard for pres_check.h, declare extern variables in .h --- Modified: src/modules/presence_xml/add_events.h Modified: src/modules/presence_xml/api.h Modified: src/modules/presence_xml/pres_check.h Modified: src/modules/presence_xml/presence_xml.c --- Diff: https://github.com/kamailio/kamailio/commit/a9331b659eb767e8ae19f1ae2adb0ec… Patch: https://github.com/kamailio/kamailio/commit/a9331b659eb767e8ae19f1ae2adb0ec… --- diff --git a/src/modules/presence_xml/add_events.h b/src/modules/presence_xml/add_events.h index a4acc8dd75..c15c75c368 100644 --- a/src/modules/presence_xml/add_events.h +++ b/src/modules/presence_xml/add_events.h @@ -13,15 +13,15 @@ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA * */ /*! * \file - * \brief Kamailio Presence_XML :: Several event packages, presence, presence.winfo, dialog;sla + * \brief Kamailio Presence_XML :: Several event packages, presence, presence.winfo, dialog;sla * \ingroup presence_xml */ diff --git a/src/modules/presence_xml/api.h b/src/modules/presence_xml/api.h index 73ed2c12d0..36a907f1cd 100644 --- a/src/modules/presence_xml/api.h +++ b/src/modules/presence_xml/api.h @@ -20,6 +20,7 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA * */ + #ifndef PXML_API_H #define PXML_API_H #include "../../core/str.h" diff --git a/src/modules/presence_xml/pres_check.h b/src/modules/presence_xml/pres_check.h index 70d1e88128..40b9409076 100644 --- a/src/modules/presence_xml/pres_check.h +++ b/src/modules/presence_xml/pres_check.h @@ -13,12 +13,16 @@ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#ifndef _PRES_CHECK_H_ +#define _PRES_CHECK_H_ + + #include <stdio.h> #include "../../core/parser/msg_parser.h" #include "../presence/bind_presence.h" @@ -26,6 +30,7 @@ int presxml_check_basic(struct sip_msg* msg, str presentity_uri, str status); int presxml_check_activities(struct sip_msg* msg, str presentity_uri, str activity); -contains_event_t pres_contains_event; -pres_get_presentity_t pres_get_presentity; -pres_free_presentity_t pres_free_presentity; +extern contains_event_t pres_contains_event; +extern pres_get_presentity_t pres_get_presentity; +extern pres_free_presentity_t pres_free_presentity; +#endif diff --git a/src/modules/presence_xml/presence_xml.c b/src/modules/presence_xml/presence_xml.c index 228896c18a..3918903e96 100644 --- a/src/modules/presence_xml/presence_xml.c +++ b/src/modules/presence_xml/presence_xml.c @@ -78,6 +78,9 @@ add_event_t pres_add_event; update_watchers_t pres_update_watchers; pres_get_sphere_t pres_get_sphere; +contains_event_t pres_contains_event; +pres_get_presentity_t pres_get_presentity; +pres_free_presentity_t pres_free_presentity; /* Module parameter variables */ str xcap_table= str_init("xcap");

5 years, 11 months

git:master:516091c0: presence: remove svn id lines and history comments

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 516091c0fcf3554d1778f257bc8b086b5ef30a70 URL: https://github.com/kamailio/kamailio/commit/516091c0fcf3554d1778f257bc8b086… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2019-01-17T08:36:52+01:00 presence: remove svn id lines and history comments --- Modified: src/modules/presence_xml/add_events.c Modified: src/modules/presence_xml/add_events.h Modified: src/modules/presence_xml/api.h Modified: src/modules/presence_xml/notify_body.c Modified: src/modules/presence_xml/notify_body.h Modified: src/modules/presence_xml/pidf.c Modified: src/modules/presence_xml/pidf.h Modified: src/modules/presence_xml/presence_xml.h Modified: src/modules/presence_xml/xcap_auth.c Modified: src/modules/presence_xml/xcap_auth.h --- Diff: https://github.com/kamailio/kamailio/commit/516091c0fcf3554d1778f257bc8b086… Patch: https://github.com/kamailio/kamailio/commit/516091c0fcf3554d1778f257bc8b086…

5 years, 11 months

← Newer
1
...
14
15
16
17
18
19
20
...
31
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2019