Re: [sr-dev] [kamailio/kamailio] Kamailio memory leak - versions 5.1.4 and 5.1.7 (#1895)

2019-07-01 Thread Jay J
(gdb) bt n
#0  0x7fab34880860 in __errno_location@plt () from 
/usr/lib/x86_64-linux-gnu/libmysqlclient.so.20
#1  0x7fab348bb589 in ?? () from 
/usr/lib/x86_64-linux-gnu/libmysqlclient.so.20
#2  0x7fab348bb849 in ?? () from 
/usr/lib/x86_64-linux-gnu/libmysqlclient.so.20
#3  0x7fab34895bc0 in ?? () from 
/usr/lib/x86_64-linux-gnu/libmysqlclient.so.20
#4  0x7fab34896c2c in ?? () from 
/usr/lib/x86_64-linux-gnu/libmysqlclient.so.20
#5  0x7fab3488aedc in ?? () from 
/usr/lib/x86_64-linux-gnu/libmysqlclient.so.20
#6  0x7fab3488bfa8 in ?? () from 
/usr/lib/x86_64-linux-gnu/libmysqlclient.so.20
#7  0x7fab3488dd61 in mysql_real_query () from 
/usr/lib/x86_64-linux-gnu/libmysqlclient.so.20
#8  0x7fab34e902ae in db_mysql_submit_query (_h=0x7fab35237ee0, 
_s=0x7fab346502d0 ) at km_dbase.c:116
#9  0x7fab3443ff6d in db_do_submit_query (_h=0x7fab35237ee0, 
_query=0x7fab346502d0 , submit_query=0x7fab34e8fb36 
) at db_query.c:53
#10 0x7fab3444310d in db_do_delete (_h=0x7fab35237ee0, _k=0x7ffdad2ea400, 
_o=0x0, _v=0x7ffdad2ea420, _n=2, val2str=0x7fab34e8e136 ,
submit_query=0x7fab34e8fb36 ) at db_query.c:291
#11 0x7fab34e94d64 in db_mysql_delete (_h=0x7fab35237ee0, 
_k=0x7ffdad2ea400, _o=0x0, _v=0x7ffdad2ea420, _n=2) at km_dbase.c:515
#12 0x7fab3285f714 in remove_dialog_from_db (cell=0x7faaae3e6fd0) at 
dlg_db_handler.c:714
#13 0x7fab3283b375 in destroy_dlg (dlg=0x7faaae3e6fd0) at dlg_hash.c:371
#14 0x7fab32842ca0 in dlg_unref_helper (dlg=0x7faaae3e6fd0, cnt=2, 
fname=0x7fab328d353f "dlg_handlers.c", fline=410) at dlg_hash.c:961
#15 0x7fab32867fb5 in dlg_ontdestroy (t=0x7faaae56ea00, type=131072, 
param=0x7ffdad2ea710) at dlg_handlers.c:410
#16 0x7fab32bd5b05 in run_trans_callbacks_internal (cb_lst=0x7faaae56ea78, 
type=131072, trans=0x7faaae56ea00, params=0x7ffdad2ea710) at t_hooks.c:260
#17 0x7fab32bd5c2c in run_trans_callbacks (type=131072, 
trans=0x7faaae56ea00, req=0x0, rpl=0x0, code=0) at t_hooks.c:287
#18 0x7fab32b0bf45 in free_cell_helper (dead_cell=0x7faaae56ea00, silent=0, 
fname=0x7fab32c1e882 "timer.c", fline=689) at h_table.c:166
#19 0x7fab32bc3bf6 in wait_handler (ti=1826413493, wait_tl=0x7faaae56ea88, 
data=0x7faaae56ea00) at timer.c:689
#20 0x556a3d1b32c5 in timer_list_expire (t=1826413493, h=0x7faaad841200, 
slow_l=0x7faaad8418b8, slow_mark=72) at core/timer.c:874
#21 0x556a3d1b376d in timer_handler () at core/timer.c:939
#22 0x556a3d1b3c36 in timer_main () at core/timer.c:978
#23 0x556a3d105322 in main_loop () at main.c:1727
#24 0x556a3d10ca1b in main (argc=7, argv=0x7ffdad2ead98) at main.c:2696


-- 
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/1895#issuecomment-507458520___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] Kamailio memory leak - versions 5.1.4 and 5.1.7 (#1895)

2019-07-01 Thread Jay J
 strings /usr/lib/x86_64-linux-gnu/libcrypto.so | grep "^OpenSSL"
OpenSSLDie
OpenSSL_add_all_ciphers
OpenSSL_add_all_digests
OpenSSL 1.0.2n  7 Dec 2017
OpenSSL HMAC method
OpenSSL EC algorithm
OpenSSL RSA method
OpenSSL DSA method
OpenSSL ECDSA method
OpenSSL DH Method
OpenSSL X9.42 DH method
OpenSSL PKCS#3 DH method
OpenSSL ECDH method
OpenSSL 'dlfcn' shared library method
OpenSSL default
OpenSSL default user interface
OpenSSL CMAC method


Yet the issue is still reproducible



-- 
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/1895#issuecomment-507456247___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] db_redis: add support for native redis cluster (#1997)

2019-07-01 Thread Andreas Granig
Hey, unfortunately it doesn’t support it yet, and it’s not clear yet when I can 
address this, as I don’t have immediate needs for it.

-- 
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/1997#issuecomment-507402841___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] uac module documentation should be honor s: prefix for kamcmd (#1988)

2019-07-01 Thread Daniel-Constantin Mierla
Added such examples in docs.

-- 
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/1988#issuecomment-507314688___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] uac module documentation should be honor s: prefix for kamcmd (#1988)

2019-07-01 Thread Daniel-Constantin Mierla
Closed #1988.

-- 
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/1988#event-2451632538___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] git:master:ec42a0b3: uac: docs - examples with rpc commands having params numbers converted to string

2019-07-01 Thread Daniel-Constantin Mierla
Module: kamailio
Branch: master
Commit: ec42a0b36a10dec0852bd1ef2bf3ad7ab9ad8dbc
URL: 
https://github.com/kamailio/kamailio/commit/ec42a0b36a10dec0852bd1ef2bf3ad7ab9ad8dbc

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2019-07-01T17:31:09+02:00

uac: docs - examples with rpc commands having params numbers converted to string

- GH #1988

---

Modified: src/modules/uac/doc/uac_admin.xml

---

Diff:  
https://github.com/kamailio/kamailio/commit/ec42a0b36a10dec0852bd1ef2bf3ad7ab9ad8dbc.diff
Patch: 
https://github.com/kamailio/kamailio/commit/ec42a0b36a10dec0852bd1ef2bf3ad7ab9ad8dbc.patch

---

diff --git a/src/modules/uac/doc/uac_admin.xml 
b/src/modules/uac/doc/uac_admin.xml
index 6e514eeea1..e4971191a8 100644
--- a/src/modules/uac/doc/uac_admin.xml
+++ b/src/modules/uac/doc/uac_admin.xml
@@ -1113,6 +1113,7 @@ event_route[uac:reply] {

 ...
kamcmd uac.reg_info l_uuid account123
+   kamcmd uac.reg_info l_uuid s:12345678
 ...


@@ -1134,6 +1135,7 @@ event_route[uac:reply] {

 ...
kamcmd uac.reg_enable l_uuid account123
+   kamcmd uac.reg_enable l_uuid s:12345678
 ...


@@ -1155,6 +1157,7 @@ event_route[uac:reply] {

 ...
kamcmd uac.reg_disable l_uuid account123
+   kamcmd uac.reg_disable l_uuid s:12345678
 ...


@@ -1191,6 +1194,7 @@ event_route[uac:reply] {

 ...
kamcmd uac.reg_refresh account123
+   kamcmd uac.reg_refresh s:12345678
 ...




___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] uri.param and param.value error when last character is delimiter (#1991)

2019-07-01 Thread Daniel-Constantin Mierla
Closed #1991.

-- 
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/1991#event-2451613284___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] uri.param and param.value error when last character is delimiter (#1991)

2019-07-01 Thread Daniel-Constantin Mierla
I pushed a commit to relax the behaviour of the transformation not to be SIP 
spec strict in such cases -  1c4af823c39d28b694c1dec607d8d72c5b726456 , hoping 
to help by being more flexible with values loaded from various sources.

-- 
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/1991#issuecomment-507312165___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] db_redis: add support for native redis cluster (#1997)

2019-07-01 Thread Daniel-Constantin Mierla
The two modules were developed independently. Maybe @agranig can comment if 
`db_redis` already supports or not processing MOVED replies.

-- 
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/1997#issuecomment-507310610___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


Re: [sr-dev] [kamailio/kamailio] After commit ce8d36f latency_limit_db requires us instead of ms (#1956)

2019-07-01 Thread Marco Capetta
Just tested the upgrade to kamailio 5.2.3 and in fact now both the parameters 
(latency_limit_db and latency_limit_action) expect a micro-seconds value.
I suggest to leave the code as is and instead fix the documentation, currently 
it suggesting to use milli-seconds. 

Thanks

-- 
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/1956#issuecomment-507253817___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev


[sr-dev] [kamailio/kamailio] db_redis: add support for native redis cluster (#1997)

2019-07-01 Thread George Diamantopoulos
The db_redis module seems to only support connecting with a single redis 
instance. It is not evident from the documentation if it implements support for 
native redis cluster by honouring 'MOVED' replies.

ndb_redis seems to have support for native redis clusters, and supports both 
automatic (see allow_dynamic_nodes parameter of ndb_redis) and manual 
configuration of redis nodes.

Since both modules are based on the hiredis library, is it safe to assume this 
support could be easily implemented, with some code sharing between the two 
modules perhaps?

If db_redis supports clusters with automatic configuration only (by processing 
MOVED replies), perhaps the module's documentation should be updated to reflect 
that.

-- 
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/1997___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev