sr-dev February 2020

sr-dev@lists.kamailio.org

17 participants
325 discussions

[kamailio/kamailio] Kamailio crash with invalid statement in configuration (#2227)
by Henning Westerholt 05 Mar '20

05 Mar '20

Kamailio crash with invalid statement in configuration. I already investigated several probable causes to some extend, but did not managed to find a fix yet. ``` $ cat ../etc/crash.cfg (note the capital "I" at the if-case, memdbg was just added for debugging) #!KAMAILIO memdbg=3 loadmodule "textops.so" request_route { Iif (is_method("CANCEL")) { exit; } } ``` henning@static:~/repositories/kamailio/src$ ./kamailio -L modules -Y /tmp -T -S -n 1 -D -m 16 -M 4 -f ../etc/crash.cfg -E 0(24666) ERROR: <core> [core/cfg.y:3403]: yyparse(): cfg. parser: failed to find command Iif (params 2) 0(24666) CRITICAL: <core> [core/cfg.y:3544]: yyerror_at(): parse error in config file /home/henning/repositories/kamailio/src/../etc/crash.cfg, line 7, column 26: unknown command, missing loadmodule? 0(24666) ERROR: <core> [core/cfg.y:3407]: yyparse(): free function 0x7fa138bc1d60 Segmentation fault ``` output from GDB, current git master: Program received signal SIGSEGV, Segmentation fault. 0x0000555555843e4e in rve_destroy (rve=0xc0c0c0c0) at core/rvalue.c:147 147 if (rve->op==RVE_RVAL_OP){ (gdb) bt #0 0x0000555555843e4e in rve_destroy (rve=0xc0c0c0c0) at core/rvalue.c:147 #1 0x00005555558447e1 in rve_destroy (rve=0x7ffff6c98738) at core/rvalue.c:168 #2 0x000055555599b031 in free_mod_func_action (a=0x7ffff6c97d60) at core/cfg.y:3997 #3 0x000055555599665b in yyparse () at core/cfg.y:3408 #4 0x000055555558fbd1 in main (argc=18, argv=0x7fffffffe378) at main.c:2309 (gdb) p rve->op Cannot access memory at address 0xc0c0c0c0 (gdb) p rve $1 = (struct rval_expr *) 0xc0c0c0c0 (gdb) f 1 #1 0x00005555558447e1 in rve_destroy (rve=0x7ffff6c98738) at core/rvalue.c:168 168 rve_destroy(rve->left.rve); (gdb) p rve->left $2 = {rve = 0xc0c0c0c0, rval = {type = 3233857728, refcnt = 0, v = {p = 0xabcdefed, l = 2882400237, s = {s = 0xabcdefed <error: Cannot access memory at address 0xabcdefed>, len = 0}, avps = {type = 2882400237, name = {n = 0, s = {s = 0x0, len = 0}, re = 0x0}, index = 8}, sel = {f = {0xabcdefed, 0x0, 0x0, 0x8}, param_offset = {0, 0, 3598392, 0, -154930224}, params = {{type = 1436524721, v = {i = 1436573777, s = {s = 0x555555a06051 "fragm. from qm_malloc", len = 1436524587}, p = 0x555555a06051}}, {type = 1260, v = { i = -252645136, s = {s = 0xf0f0f0f0 <error: Cannot access memory at address 0xf0f0f0f0>, len = 0}, p = 0xf0f0f0f0}}, {type = SEL_PARAM_INT, v = {i = 0, s = {s = 0x0, len = 0}, p = 0x0}} <repeats 30 times>}, n = 0}, pvs = {type = 2882400237, getf = 0x0, setf = 0x0, pvp = {pvn = {type = 8, nfree = 0x0, u = {isname = {type = 3598392, name = {n = -154930224, s = {s = 0x7ffff6c3f3d0 "", len = 1436524721}, re = 0x7ffff6c3f3d0}}, dname = 0x36e838}}, pvi = {type = 1436573777, u = {ival = 1436524587, dval = 0x5555559fa02b}}}, trans = 0x4ec}, action = 0xabcdefed, bexpr = 0xabcdefed, re = {s = {s = 0xabcdefed <error: Cannot access memory at address 0xabcdefed>, len = 0}, regex = 0x0}}, bsize = 0, flags = 0, buf = ""}} (gdb) p rve $3 = (struct rval_expr *) 0x7ffff6c98738 (gdb) p *rve $4 = {op = 1129201987, left = {rve = 0xc0c0c0c0, rval = {type = 3233857728, refcnt = 0, v = {p = 0xabcdefed, l = 2882400237, s = {s = 0xabcdefed <error: Cannot access memory at address 0xabcdefed>, len = 0}, avps = {type = 2882400237, name = {n = 0, s = {s = 0x0, len = 0}, re = 0x0}, index = 8}, sel = {f = {0xabcdefed, 0x0, 0x0, 0x8}, param_offset = {0, 0, 3598392, 0, -154930224}, params = {{type = 1436524721, v = {i = 1436573777, s = {s = 0x555555a06051 "fragm. from qm_malloc", len = 1436524587}, p = 0x555555a06051}}, {type = 1260, v = {i = -252645136, s = {s = 0xf0f0f0f0 <error: Cannot access memory at address 0xf0f0f0f0>, len = 0}, p = 0xf0f0f0f0}}, {type = SEL_PARAM_INT, v = {i = 0, s = {s = 0x0, len = 0}, p = 0x0}} <repeats 30 times>}, n = 0}, pvs = {type = 2882400237, getf = 0x0, setf = 0x0, pvp = {pvn = {type = 8, nfree = 0x0, u = {isname = {type = 3598392, name = {n = -154930224, s = {s = 0x7ffff6c3f3d0 "", len = 1436524721}, re = 0x7ffff6c3f3d0}}, dname = 0x36e838}}, pvi = { type = 1436573777, u = {ival = 1436524587, dval = 0x5555559fa02b}}}, trans = 0x4ec}, action = 0xabcdefed, bexpr = 0xabcdefed, re = {s = {s = 0xabcdefed <error: Cannot access memory at address 0xabcdefed>, len = 0}, regex = 0x0}}, bsize = 0, flags = 0, buf = ""}}, right = {rve = 0x0, rval = {type = RV_NONE, refcnt = 0, v = {p = 0x0, l = 0, s = {s = 0x0, len = 0}, avps = {type = 0, name = {n = 0, s = {s = 0x0, len = 0}, re = 0x0}, index = 0}, sel = {f = {0x0, 0x0, 0x0, 0x0}, param_offset = {0, 0, 0, 0, 0}, params = {{type = SEL_PARAM_INT, v = {i = 0, s = {s = 0x0, len = 0}, p = 0x0}} <repeats 32 times>}, n = 0}, pvs = {type = PVT_NONE, getf = 0x0, setf = 0x0, pvp = {pvn = {type = 0, nfree = 0x0, u = {isname = {type = 0, name = {n = 0, s = {s = 0x0, len = 0}, re = 0x0}}, dname = 0x0}}, pvi = {type = 0, u = {ival = 0, dval = 0x0}}}, trans = 0x0}, action = 0x0, bexpr = 0x0, re = {s = {s = 0x0, len = 0}, regex = 0x0}}, bsize = 0, flags = 0, buf = ""}}, fpos = {s_line = 0, e_line = 0, s_col = 0, e_col = 0, fname = 0x0, rname = 0x0}} (gdb) ``` -- 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/2227

2 6

MRCP kamailio module (for use with rtpengine)
by Dmitriy Borisov 03 Mar '20

03 Mar '20

Hello! I want to implement kamailio module for MRCP interaction. I need it for automatic machine detection (AMD) system. I will be grateful for any help and have some initial questions: 1. Is it interesting for anyone? 2. Now I reading http://www.asipto.com/pub/kamailio-devel-guide/ guide. Is it the newest one or I can find something more actual? I want to implement such scheme: https://unimrcp.org/ea/index.htm?goto=2:1:30 Where kamailio will be on place of PBX, rtp-sender will be on place of IP Phone (and will be connected to RTP stream via proc-subsystem to rtpengine (like a rtpengine-recorder daemon). I want to use same scheme as HTTP async module (define a route, which must be called on event receive). -- With best regards Dmitry Borisov

2 1

[kamailio/kamailio] New branch in DNS SRV failover is using wrong source socket (#2152)
by Thomas Lobker 03 Mar '20

03 Mar '20

### Description When relaying an `INVITE` from a Kamailio proxy to an interconnect we are using DNS SRV records for load-balancing and failover. The proxy is listening on both a private interface and a public interface with an rfc1918 (ie. `10.0.0.14`) and a public IP address (ie. `185.0.0.34`) respectively. The first branch (before DNS SRV failover) is working as expected. The message will be relayed from the received socket (`185.0.0.34`) to the interconnect. When this branch results in a timeout, the proxy will try to do DNS SRV failover. This new branch and any subsequent branches will no longer use the initial received socket as source. In our case we see that the private address (`10.0.0.14`) is now being used as source address. ### Troubleshooting #### Reproduction DNS SRV: ``` _sip._udp.transit.net. SRV 10 10 5060 transit1.net. _sip._udp.transit.net. SRV 20 10 5060 transit2.net. transit1.net. A 185.10.20.30 transit2.net. A 185.10.20.31 ``` Kamailio: ``` $du = "sip:transit.net;transport=udp"; xinfo("Relaying [$rm] request: [$ru] with Call-ID [$ci]"); t_set_fr(0, 1000); if (not t_relay()) { sl_reply_error(); } ``` Network flow: ``` 12:30:00 INVITE udp:10.0.0.18:5060 => udp:185.0.0.34:5060 (internal request to proxy) 12:30:00 INVITE udp:185.0.0.34:5060 => udp:185.10.20.30:5060 (relaying from proxy to interconnect) ^^^^^^^^^^ (request times out after 1 second, proxy will do a failover to the next endpoint) 12:30:01 INVITE udp:10.0.0.14:5060 => udp:185.10.20.31:5060 (relaying to next interconnect address) ^^^^^^^^^ ``` #### Log Messages Attempt to see where it goes wrong: ``` onsend_route { xinfo("[$RAut] [$Rut] [$sas]\n"); xinfo("$snd(buf)\n"); } ``` ``` INFO: [sip:185.0.0.34:5060;transport=udp] [sip:185.0.0.34:5060;transport=udp] [udp:10.0.0.18:5060] INFO: INVITE sip:+1234567890@transit.net;user=phone SIP/2.0#015#012Record-Route: <sip:185.0.0.34;lr;ftag=tDr7m6erX1N3D>#015#012Via: SIP/2.0/UDP 10.0.0.14;branch=z9hG4bKafe7.7fb590e263fa44677514193a6a1156ce.1#015#012Via: SIP/2.0/UDP 10.0.0.18;received=10.0.0.18;rport=5060;branch=z9hG4bK6t59a17N60FcB ... ``` So the `Record-Route` seems to be correct, but the top most `Via` header shows the private IP address. The message is being sent from the private IP address as well and never reaches the second address from the interconnect. ### Possible Solutions A workaround to add `$fs = "udp:185.0.0.34:5060"` in the `onsend_route` seems to be effective. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.2.3 (x86_64/linux) 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, 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_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 7.4.0 ``` * **Operating System**: ``` Ubuntu 18.04 LTS Linux proxy4 4.15.0-64-generic #73-Ubuntu SMP Thu Sep 12 13:16:13 UTC 2019 x86_64 x86_64 x86_64 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/2152

2 11

[kamailio/kamailio] [kamailio 5.2.5] Wrong send socket on a multi homed proxy (#2181)
by vitalikvoip 02 Mar '20

02 Mar '20

### Description Kamailio fails to forward in-dialog request over TCP/TLS/WS/WSS connection. ### Troubleshooting kamailio 5.2 branch has a commit 09ac3e47ef79dbd599b7dec5b84ae3b792f025ed cherry picked from master branch. It makes sense for master because get_send_socket2() behavior has changed and function became smarter and also there were some patches to propagate connection it to the uac structure. After applying this patch to 5.2.x kamailio searches for an outgoing socket, receives the first one due to limitations of get_send_socket2() and SND_F_FORCE_SOCKET flag. Then it fails to find an existing TCP connection by destination ip:port since it searches for a connection associated with a wrong socket. #### Log Messages  ``` Dec 11 21:33:49 kamailio[17999]: WARNING: <core> [core/msg_translator.c:2832]: via_builder(): TCP/TLS connection (id: 0) for WebSocket could not be found Dec 11 21:33:49 kamailio[17999]: ERROR: tm [t_msgbuilder.c:1372]: assemble_via(): via building failed Dec 11 21:33:49 kamailio[17999]: ERROR: tm [t_msgbuilder.c:1540]: build_uac_req(): error while assembling Via Dec 11 21:33:49 kamailio[17999]: ERROR: tm [uac.c:517]: t_uac_prepare(): Error while building message Dec 11 21:33:49 kamailio[17999]: ERROR: presence [notify.c:1798]: send_notify_request(): in function tmb.t_request_within Dec 11 21:33:49 kamailio[17999]: ERROR: presence [notify.c:1897]: notify(): sending Notify not successful Dec 11 21:33:49 kamailio[17999]: ERROR: presence [notify.c:1491]: publ_notify(): Could not send notify for message-summary ``` ### Possible Solutions Delete mentioned commit from the 5.2 branch.  ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` 5.2.5 ``` * **Operating System**:  ``` 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/2181

3 6

[kamailio/kamailio] Fedora 32/Rawhide build error: modules/siputils: multiple definition of `contact_flds_separator' (#2235)
by Anthony Messina 28 Feb '20

28 Feb '20

Just FYI/notice when preparing for Fedora 32 builds that for `src/modules/siputils`, `contact_flds_separator` is defined both in `contact_ops.h` and `siputils.c` ``` make[2]: Entering directory '/builddir/build/BUILD/kamailio-5.3.2/src/modules/siputils' gcc -shared -Wl,-O2 -Wl,-E -pthread -rdynamic -ldl -Wl,-Bsymbolic-functions sipops.o config.o siputils.o rpid.o ring.o contact_ops.o chargingvector.o options.o utils.o checks.o -L../../lib/srdb1/ -lsrdb1 -Wl,-rpath,/builddir/build/BUILD/kamailio-5.3.2/src/lib/srdb1 -o siputils.so make[2]: Leaving directory '/builddir/build/BUILD/kamailio-5.3.2/src/modules/siputils' /usr/bin/ld: contact_ops.o:/builddir/build/BUILD/kamailio-5.3.2/src/modules/siputils/contact_ops.h:51: multiple definition of `contact_flds_separator'; siputils.o:/builddir/build/BUILD/kamailio-5.3.2/src/modules/siputils/contact_ops.h:51: first defined here collect2: error: ld returned 1 exit status make[2]: *** [../../Makefile.rules:191: siputils.so] Error 1 make[1]: *** [Makefile:511: modules] Error 1 make: *** [Makefile:34: every-module] Error 2 ``` -- 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/2235

2 3

git:5.2:e3e93791: siputils: declare extern contact_flds_separator in header file
by Daniel-Constantin Mierla 28 Feb '20

28 Feb '20

Module: kamailio Branch: 5.2 Commit: e3e937914b30173f83fe91ad4d18f6fe6dbd2eb1 URL: https://github.com/kamailio/kamailio/commit/e3e937914b30173f83fe91ad4d18f6f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-02-28T18:01:31+01:00 siputils: declare extern contact_flds_separator in header file - GH #2235 (cherry picked from commit 6e9b0fad487aadb22df10e90f3db6a104ca99cfc) (cherry picked from commit 66e33a0e3c7af9c51dc665556a022b6598e86843) --- Modified: src/modules/siputils/contact_ops.h --- Diff: https://github.com/kamailio/kamailio/commit/e3e937914b30173f83fe91ad4d18f6f… Patch: https://github.com/kamailio/kamailio/commit/e3e937914b30173f83fe91ad4d18f6f… --- diff --git a/src/modules/siputils/contact_ops.h b/src/modules/siputils/contact_ops.h index cb3c698ed3..ef925c761b 100644 --- a/src/modules/siputils/contact_ops.h +++ b/src/modules/siputils/contact_ops.h @@ -48,7 +48,7 @@ #define DEFAULT_SEPARATOR "*" -char *contact_flds_separator; +extern char *contact_flds_separator; struct uri_format

1 0

git:5.3:66e33a0e: siputils: declare extern contact_flds_separator in header file
by Daniel-Constantin Mierla 28 Feb '20

28 Feb '20

Module: kamailio Branch: 5.3 Commit: 66e33a0e3c7af9c51dc665556a022b6598e86843 URL: https://github.com/kamailio/kamailio/commit/66e33a0e3c7af9c51dc665556a022b6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-02-28T18:00:41+01:00 siputils: declare extern contact_flds_separator in header file - GH #2235 (cherry picked from commit 6e9b0fad487aadb22df10e90f3db6a104ca99cfc) --- Modified: src/modules/siputils/contact_ops.h --- Diff: https://github.com/kamailio/kamailio/commit/66e33a0e3c7af9c51dc665556a022b6… Patch: https://github.com/kamailio/kamailio/commit/66e33a0e3c7af9c51dc665556a022b6… --- diff --git a/src/modules/siputils/contact_ops.h b/src/modules/siputils/contact_ops.h index cb3c698ed3..ef925c761b 100644 --- a/src/modules/siputils/contact_ops.h +++ b/src/modules/siputils/contact_ops.h @@ -48,7 +48,7 @@ #define DEFAULT_SEPARATOR "*" -char *contact_flds_separator; +extern char *contact_flds_separator; struct uri_format

1 0

git:master:7f95cd20: pv: debug messages when setting shv via rpc
by Daniel-Constantin Mierla 28 Feb '20

28 Feb '20

Module: kamailio Branch: master Commit: 7f95cd207b5dd56ace92837ad989f234e64a2471 URL: https://github.com/kamailio/kamailio/commit/7f95cd207b5dd56ace92837ad989f23… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-02-28T17:57:41+01:00 pv: debug messages when setting shv via rpc --- Modified: src/modules/pv/pv_shv.c --- Diff: https://github.com/kamailio/kamailio/commit/7f95cd207b5dd56ace92837ad989f23… Patch: https://github.com/kamailio/kamailio/commit/7f95cd207b5dd56ace92837ad989f23… --- diff --git a/src/modules/pv/pv_shv.c b/src/modules/pv/pv_shv.c index 1f472664d7..fb4de51114 100644 --- a/src/modules/pv/pv_shv.c +++ b/src/modules/pv/pv_shv.c @@ -546,8 +546,9 @@ void rpc_shv_set(rpc_t* rpc, void* c) if (rpc->scan(c, "d", &ival) != 1) { rpc->fault(c, 500, "Missing integer parameter value (Parameters: varname type value)"); return; - } + } isv.n = ival; + LM_DBG("setting %.*s to int %d\n", varname.len, varname.s, ival); } else if (strcasecmp(type.s, "str") == 0 ) { /* String value */ if (rpc->scan(c, "S", &value) != 1) { @@ -556,6 +557,8 @@ void rpc_shv_set(rpc_t* rpc, void* c) } isv.s = value; flags = VAR_VAL_STR; + LM_DBG("setting %.*s to str %.*s\n", varname.len, varname.s, + value.len, value.s); } else { rpc->fault(c, 500, "Unknown parameter type (Types: int or str)"); return;

1 0

git:master:6e9b0fad: siputils: declare extern contact_flds_separator in header file
by Daniel-Constantin Mierla 28 Feb '20

28 Feb '20

Module: kamailio Branch: master Commit: 6e9b0fad487aadb22df10e90f3db6a104ca99cfc URL: https://github.com/kamailio/kamailio/commit/6e9b0fad487aadb22df10e90f3db6a1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-02-28T17:57:41+01:00 siputils: declare extern contact_flds_separator in header file - GH #2235 --- Modified: src/modules/siputils/contact_ops.h --- Diff: https://github.com/kamailio/kamailio/commit/6e9b0fad487aadb22df10e90f3db6a1… Patch: https://github.com/kamailio/kamailio/commit/6e9b0fad487aadb22df10e90f3db6a1… --- diff --git a/src/modules/siputils/contact_ops.h b/src/modules/siputils/contact_ops.h index cb3c698ed3..ef925c761b 100644 --- a/src/modules/siputils/contact_ops.h +++ b/src/modules/siputils/contact_ops.h @@ -48,7 +48,7 @@ #define DEFAULT_SEPARATOR "*" -char *contact_flds_separator; +extern char *contact_flds_separator; struct uri_format

1 0

git:5.3:9539be33: dmq: small spelling fix in error message
by Henning Westerholt 28 Feb '20

28 Feb '20

Module: kamailio Branch: 5.3 Commit: 9539be33fa2026ace2090bc51427eb80346ffb4d URL: https://github.com/kamailio/kamailio/commit/9539be33fa2026ace2090bc51427eb8… Author: Henning Westerholt <hw(a)skalatan.de> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-02-28T14:54:19+01:00 dmq: small spelling fix in error message (cherry picked from commit 97e3de1046cf5247f7a462385778409b02680149) --- Modified: src/modules/dmq/dmq.c --- Diff: https://github.com/kamailio/kamailio/commit/9539be33fa2026ace2090bc51427eb8… Patch: https://github.com/kamailio/kamailio/commit/9539be33fa2026ace2090bc51427eb8… --- diff --git a/src/modules/dmq/dmq.c b/src/modules/dmq/dmq.c index 9414e94e84..2b64d0e907 100644 --- a/src/modules/dmq/dmq.c +++ b/src/modules/dmq/dmq.c @@ -282,7 +282,7 @@ static int child_init(int rank) LM_DBG("starting worker process %d\n", i); newpid = fork_process(PROC_RPC, "DMQ WORKER", 0); if(newpid < 0) { - LM_ERR("failed to form process\n"); + LM_ERR("failed to fork process\n"); return -1; } else if(newpid == 0) { /* child - this will loop forever */

1 0

← Newer
1
...
8
9
10
11
12
13
14
...
33
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev February 2020