sr-dev January 2015

sr-dev@lists.kamailio.org

38 participants
530 discussions

[kamailio] permissions: Core dump (#41)
by Olle E. Johansson 23 Jan '15

23 Jan '15

NO traffic, just starting Kamailio and letting it run for a while with a lot of modules loaded. git head. (gdb) bt full #0 0x00007fff8a92e866 in __pthread_kill () No symbol table info available. #1 0x00007fff86b1235c in pthread_kill () No symbol table info available. #2 0x00007fff869e9b1a in abort () No symbol table info available. #3 0x00000001081ebc67 in fm_free (qm=0x10a78a000, p=0x5c, file=0x109ee9295 "permissions: hash.c", func=0x109eea05c "free_subnet_table", line=845) at mem/f_malloc.c:588 f = (struct fm_frag *) 0x10aa0a698 #4 0x0000000109eb9990 in free_subnet_table (table=0x10aa0c340) at hash.c:845 i = 0 #5 0x0000000109ea8cfc in clean_addresses () at address.c:442 No locals. #6 0x0000000109ec5c7e in mod_exit () at permissions.c:682 i = 1 #7 0x00000001080fc089 in destroy_modules () at sr_module.c:788 t = (struct sr_module *) 0x108553540 foo = (struct sr_module *) 0x1085531b8 #8 0x0000000107f533cb in cleanup (show_status=1) at main.c:512 memlog = 0 #9 0x0000000107f5ad2b in shutdown_children (sig=15, show_status=1) at main.c:654 No locals. #10 0x0000000107f56394 in handle_sigs () at main.c:684 chld = 0 chld_status = 0 memlog = 0 #11 0x0000000107f5adb7 in sig_usr (signo=2) at main.c:781 memlog = 0 #12 <signal handler called> --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/41

2 4

git:master:165a3fc2: permissions: init global pointer variables to null for safe shutdown at start
by Daniel-Constantin Mierla 23 Jan '15

23 Jan '15

Module: kamailio Branch: master Commit: 165a3fc2e4dfed09bd9f46e43270d49cf5f72a87 URL: https://github.com/kamailio/kamailio/commit/165a3fc2e4dfed09bd9f46e43270d49… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-01-23T11:07:26+01:00 permissions: init global pointer variables to null for safe shutdown at start - reported by Olle E. Johansson, GH#41 --- Modified: modules/permissions/address.c --- Diff: https://github.com/kamailio/kamailio/commit/165a3fc2e4dfed09bd9f46e43270d49… Patch: https://github.com/kamailio/kamailio/commit/165a3fc2e4dfed09bd9f46e43270d49… --- diff --git a/modules/permissions/address.c b/modules/permissions/address.c index 45eecf6..7ad0fb5 100644 --- a/modules/permissions/address.c +++ b/modules/permissions/address.c @@ -46,17 +46,17 @@ #define TABLE_VERSION 6 -struct addr_list ***addr_hash_table; /* Ptr to current hash table ptr */ -struct addr_list **addr_hash_table_1; /* Pointer to hash table 1 */ -struct addr_list **addr_hash_table_2; /* Pointer to hash table 2 */ +struct addr_list ***addr_hash_table = NULL; /* Ptr to current hash table ptr */ +struct addr_list **addr_hash_table_1 = NULL; /* Pointer to hash table 1 */ +struct addr_list **addr_hash_table_2 = NULL; /* Pointer to hash table 2 */ -struct subnet **subnet_table; /* Ptr to current subnet table */ -struct subnet *subnet_table_1; /* Ptr to subnet table 1 */ -struct subnet *subnet_table_2; /* Ptr to subnet table 2 */ +struct subnet **subnet_table = NULL; /* Ptr to current subnet table */ +struct subnet *subnet_table_1 = NULL; /* Ptr to subnet table 1 */ +struct subnet *subnet_table_2 = NULL; /* Ptr to subnet table 2 */ -struct domain_name_list ***domain_list_table; /* Ptr to current domain name table */ -static struct domain_name_list **domain_list_table_1; /* Ptr to domain name table 1 */ -static struct domain_name_list **domain_list_table_2; /* Ptr to domain name table 2 */ +struct domain_name_list ***domain_list_table = NULL; /* Ptr to current domain name table */ +static struct domain_name_list **domain_list_table_1 = NULL; /* Ptr to domain name table 1 */ +static struct domain_name_list **domain_list_table_2 = NULL; /* Ptr to domain name table 2 */ static db1_con_t* db_handle = 0;

1 0

[kamailio] pike: Core dump (#39)
by Olle E. Johansson 23 Jan '15

23 Jan '15

Just loading pike (with a lot of other modules) running without any traffic. git head. (gdb) bt full #0 0x000000010b83e045 in clean_routine (ticks=10264832, param=0x0) at pike_funcs.c:175 mask = '\0' <repeats 31 times> head = { next = 0x62642e6f696c6961, prev = 0x0 } ll = (struct list_link *) 0x6d616b2f62642f73 dad = (struct ip_node *) 0x0 node = (struct ip_node *) 0x0 i = 1936026741 #1 0x0000000109aedc61 in compat_old_handler (ti=164237316, tl=0x10c365db8, data=0x10c365db8) at timer.c:996 t = (struct sr_timer *) 0x10c365db8 #2 0x0000000109aee43f in slow_timer_main () at timer.c:1130 n = 0 ret = 4294967295 tl = (struct timer_ln *) 0x10c365db8 i = 0 sig = 31 #3 0x000000010989af00 in main_loop () at main.c:1336 i = 166504192 pid = 0 si = (struct socket_info *) 0x109e9dff8 si_desc = "\b\000\000\0000\000\000\000??EV?\000\000\000\000\000\000\000\000\000\000???\t\001\000\000\000\001", '\0' <repeats 47 times>, "?EV?\000\000?\024?\t\001\000\000\000\020j?\t\001\000\000\000\000\000\000\000\006\000\000\000?9?\t\001\000\000\000\001\000\000\000\000\000\000" nrprocs = 1 #4 0x00000001098a722d in main (argc=8, argv=0x7fff5645ba88) at main.c:2508 --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/39

2 4

git:master:c87a75c5: pike: check that timer var is initialized before accessing the fields
by Daniel-Constantin Mierla 23 Jan '15

23 Jan '15

Module: kamailio Branch: master Commit: c87a75c5fb7aa5d940093fbbca8d9557cc9a0239 URL: https://github.com/kamailio/kamailio/commit/c87a75c5fb7aa5d940093fbbca8d955… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-01-23T10:55:15+01:00 pike: check that timer var is initialized before accessing the fields - safe start with empty config, reported by Olle E. Johansson, GH#39 --- Modified: modules/pike/pike_funcs.c --- Diff: https://github.com/kamailio/kamailio/commit/c87a75c5fb7aa5d940093fbbca8d955… Patch: https://github.com/kamailio/kamailio/commit/c87a75c5fb7aa5d940093fbbca8d955… --- diff --git a/modules/pike/pike_funcs.c b/modules/pike/pike_funcs.c index 83b565a..a0b964f 100644 --- a/modules/pike/pike_funcs.c +++ b/modules/pike/pike_funcs.c @@ -172,7 +172,7 @@ void clean_routine(unsigned int ticks , void *param) /* LM_DBG("entering (%d)\n",ticks); */ /* before locking check first if the list is not empty and if can * be at least one element removed */ - if ( is_list_empty( timer )) return; /* quick exit */ + if (timer==0 || is_list_empty( timer )) return; /* quick exit */ /* get the expired elements */ lock_get( timer_lock );

1 0

[kamailio] jsonrpc-c (#33)
by Olle E. Johansson 23 Jan '15

23 Jan '15

The jsonrpc-c module doesn't compile on OS/X CC (gcc) [M jsonrpc-c.so] jsonrpc_io.o jsonrpc_io.c:31:10: fatal error: 'sys/timerfd.h' file not found #include <sys/timerfd.h> ^ 1 error generated. --- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/33

2 5

git:master:33570d9f: jsonrpc-c: added note that it depends on Linux specific API
by Daniel-Constantin Mierla 23 Jan '15

23 Jan '15

Module: kamailio Branch: master Commit: 33570d9f575e3b0566ac5f0b9904940420be9ae2 URL: https://github.com/kamailio/kamailio/commit/33570d9f575e3b0566ac5f0b9904940… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-01-23T10:47:45+01:00 jsonrpc-c: added note that it depends on Linux specific API - reported by Olle E. Johansson, GH#33 --- Modified: modules/jsonrpc-c/README Modified: modules/jsonrpc-c/doc/jsonrpc-c_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/33570d9f575e3b0566ac5f0b9904940… Patch: https://github.com/kamailio/kamailio/commit/33570d9f575e3b0566ac5f0b9904940… --- diff --git a/modules/jsonrpc-c/README b/modules/jsonrpc-c/README index bfa8ea3..fcddab0 100644 --- a/modules/jsonrpc-c/README +++ b/modules/jsonrpc-c/README @@ -10,7 +10,7 @@ Jordan Levy <jordan(a)flowroute.com> - Copyright Â© 2011 Flowroute LLC (flowroute.com) + Copyright © 2011 Flowroute LLC (flowroute.com) __________________________________________________________________ Table of Contents @@ -77,6 +77,9 @@ Chapter 1. Admin Guide 2.1. Kamailio Modules 2.2. External Libraries or Applications + This module uses Linux specific API, it might not compile on other + types of operating systems. + 2.1. Kamailio Modules The following modules must be loaded before this module: @@ -112,7 +115,7 @@ modparam("jsonrpc", "servers", "localhost:9999,2 10.10.0.1:9999,2 backup.server: 4.2. jsonrpc_request(method, parameters, return_route, error_route, result_var) -4.1. jsonrpc_notification(method, parameters) +4.1. jsonrpc_notification(method, parameters) Invokes the remote 'method' with the given 'parameters' as a notification. Unlike jsonrpc_request (below), notifications do not @@ -127,7 +130,7 @@ modparam("jsonrpc", "servers", "localhost:9999,2 10.10.0.1:9999,2 backup.server: jsonrpc_notification("update_user", "{'id': 1234, 'name': 'Petros'}") ... -4.2. jsonrpc_request(method, parameters, return_route, error_route, +4.2. jsonrpc_request(method, parameters, return_route, error_route, result_var) Invokes the remote 'method' with the given 'parameters'. When the diff --git a/modules/jsonrpc-c/doc/jsonrpc-c_admin.xml b/modules/jsonrpc-c/doc/jsonrpc-c_admin.xml index ef5dcae..c27508b 100644 --- a/modules/jsonrpc-c/doc/jsonrpc-c_admin.xml +++ b/modules/jsonrpc-c/doc/jsonrpc-c_admin.xml @@ -32,6 +32,10 @@ <section> <title>Dependencies</title> + <para> + This module uses Linux specific API, it might not compile on other + types of operating systems. + </para> <section> <title>&kamailio; Modules</title> <para>

1 0

git:master:6ba1d8ad: acc: rebuilt of readme
by Daniel-Constantin Mierla 23 Jan '15

23 Jan '15

Module: kamailio Branch: master Commit: 6ba1d8ad7f1fb8139202f12a4269c6f4f1e000b7 URL: https://github.com/kamailio/kamailio/commit/6ba1d8ad7f1fb8139202f12a4269c6f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-01-23T09:51:07+01:00 acc: rebuilt of readme --- Modified: modules/acc/README --- Diff: https://github.com/kamailio/kamailio/commit/6ba1d8ad7f1fb8139202f12a4269c6f… Patch: https://github.com/kamailio/kamailio/commit/6ba1d8ad7f1fb8139202f12a4269c6f… --- diff --git a/modules/acc/README b/modules/acc/README index bbf84fe..59d9b33 100644 --- a/modules/acc/README +++ b/modules/acc/README @@ -564,11 +564,6 @@ Note information given will not be stored in the CDR as they cannot be accessed by the end of the call when the CDR is logged. - Note that CDR generation does not involve any kind of database storage - (yet). In order to persist the CDRs into a database you will have to - set up an exterior process (i.e., a script living outside of Kamailio) - and implement the storage task yourself. - Sometimes, dialogs expire because the UA has a problem and a final message is never transmitted. You can toggle on/off the generation of CDR-based logging in such cases with only the dlg_vars showing by using

1 0

git:master:75db07b5: acc: removed note in the docs about no-db support for cdrs
by Daniel-Constantin Mierla 23 Jan '15

23 Jan '15

Module: kamailio Branch: master Commit: 75db07b5d7794b0f67d5798917d1833a21382bb7 URL: https://github.com/kamailio/kamailio/commit/75db07b5d7794b0f67d5798917d1833… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-01-23T09:50:17+01:00 acc: removed note in the docs about no-db support for cdrs --- Modified: modules/acc/doc/acc_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/75db07b5d7794b0f67d5798917d1833… Patch: https://github.com/kamailio/kamailio/commit/75db07b5d7794b0f67d5798917d1833… --- diff --git a/modules/acc/doc/acc_admin.xml b/modules/acc/doc/acc_admin.xml index 9d0403c..eb7acdc 100644 --- a/modules/acc/doc/acc_admin.xml +++ b/modules/acc/doc/acc_admin.xml @@ -352,12 +352,6 @@ if (uri=~"sip:+40") /* calls to Romania */ { </para> <para> - Note that CDR generation does not involve any kind of database storage (yet). In order - to persist the CDRs into a database you will have to set up an exterior process (i.e., - a script living outside of &kamailio;) and implement the storage task yourself. - </para> - - <para> Sometimes, dialogs expire because the UA has a problem and a final message is never transmitted. You can toggle on/off the generation of CDR-based logging in such cases with only the dlg_vars showing by using the cdr_expired_dlg_enable parameter

1 0

git:master:814c08f3: nathelper: enclode IPv6 in square branckets for fix_nated_contact()
by Daniel-Constantin Mierla 22 Jan '15

22 Jan '15

Module: kamailio Branch: master Commit: 814c08f36f13ace21e52181da3039402f951b0a8 URL: https://github.com/kamailio/kamailio/commit/814c08f36f13ace21e52181da303940… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2015-01-22T22:37:16+01:00 nathelper: enclode IPv6 in square branckets for fix_nated_contact() - even there is no nat in ipv6 --- Modified: modules/nathelper/nathelper.c --- Diff: https://github.com/kamailio/kamailio/commit/814c08f36f13ace21e52181da303940… Patch: https://github.com/kamailio/kamailio/commit/814c08f36f13ace21e52181da303940… --- diff --git a/modules/nathelper/nathelper.c b/modules/nathelper/nathelper.c index 3123751..20500d2 100644 --- a/modules/nathelper/nathelper.c +++ b/modules/nathelper/nathelper.c @@ -803,6 +803,8 @@ fix_nated_contact_f(struct sip_msg* msg, char* str1, char* str2) cp = ip_addr2a(&msg->rcv.src_ip); len = c->uri.len + strlen(cp) + 6 /* :port */ - hostport.len + 1; + if(msg->rcv.src_ip.af==AF_INET6) + len += 2; buf = pkg_malloc(len); if (buf == NULL) { LM_ERR("out of pkg memory\n"); @@ -812,8 +814,13 @@ fix_nated_contact_f(struct sip_msg* msg, char* str1, char* str2) temp[1] = c->uri.s[c->uri.len]; c->uri.s[c->uri.len] = hostport.s[0] = '\0'; if(uri.maddr.len<=0) { - len1 = snprintf(buf, len, "%s%s:%d%s", c->uri.s, cp, msg->rcv.src_port, - hostport.s + hostport.len); + if(msg->rcv.src_ip.af==AF_INET6) { + len1 = snprintf(buf, len, "%s[%s]:%d%s", c->uri.s, cp, + msg->rcv.src_port, hostport.s + hostport.len); + } else { + len1 = snprintf(buf, len, "%s%s:%d%s", c->uri.s, cp, + msg->rcv.src_port, hostport.s + hostport.len); + } } else { /* skip maddr parameter - makes no sense anymore */ LM_DBG("removing maddr parameter from contact uri: [%.*s]\n", @@ -827,8 +834,15 @@ fix_nated_contact_f(struct sip_msg* msg, char* str1, char* str2) params1.len--; params2.s = uri.maddr.s + uri.maddr.len; params2.len = c->uri.s + c->uri.len - params2.s; - len1 = snprintf(buf, len, "%s%s:%d%.*s%.*s", c->uri.s, cp, msg->rcv.src_port, - params1.len, params1.s, params2.len, params2.s); + if(msg->rcv.src_ip.af==AF_INET6) { + len1 = snprintf(buf, len, "%s[%s]:%d%.*s%.*s", c->uri.s, cp, + msg->rcv.src_port, params1.len, params1.s, + params2.len, params2.s); + } else { + len1 = snprintf(buf, len, "%s%s:%d%.*s%.*s", c->uri.s, cp, + msg->rcv.src_port, params1.len, params1.s, + params2.len, params2.s); + } } if (len1 < len) len = len1;

1 0

git:master:a0fee0d8: ims_registrar_pcscf: Only initialize child process, if subscribe to reg is enabled.
by Carsten Bock 22 Jan '15

22 Jan '15

Module: kamailio Branch: master Commit: a0fee0d8a0a23934f9832b50aa753d03adb694a3 URL: https://github.com/kamailio/kamailio/commit/a0fee0d8a0a23934f9832b50aa753d0… Author: Carsten Bock <carsten(a)ng-voice.com> Committer: Carsten Bock <carsten(a)ng-voice.com> Date: 2015-01-22T14:16:33+02:00 ims_registrar_pcscf: Only initialize child process, if subscribe to reg is enabled. --- Modified: modules/ims_registrar_pcscf/reg_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/a0fee0d8a0a23934f9832b50aa753d0… Patch: https://github.com/kamailio/kamailio/commit/a0fee0d8a0a23934f9832b50aa753d0… --- diff --git a/modules/ims_registrar_pcscf/reg_mod.c b/modules/ims_registrar_pcscf/reg_mod.c index 9d50862..8eede00 100644 --- a/modules/ims_registrar_pcscf/reg_mod.c +++ b/modules/ims_registrar_pcscf/reg_mod.c @@ -318,7 +318,7 @@ static int child_init(int rank) { LM_DBG("Initialization of module in child [%d] \n", rank); - if (rank == PROC_MAIN) { + if ((subscribe_to_reginfo == 1) && (rank == PROC_MAIN)) { LM_DBG("Creating RegInfo Event Processor process\n"); int pid = fork_process(PROC_SIPINIT, "RegInfo Event Processor", 1); if (pid < 0)

1 0

← Newer
1
...
11
12
13
14
15
16
17
...
53
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev January 2015