sr-dev July 2016

sr-dev@lists.kamailio.org

29 participants
297 discussions

git:master:8bec45f3: tls: more detailed log message when tls module cannot set libssl memory manager

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 8bec45f3d0581747da4b807229b4de64511aa5e2 URL: https://github.com/kamailio/kamailio/commit/8bec45f3d0581747da4b807229b4de6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-07-14T12:34:58+02:00 tls: more detailed log message when tls module cannot set libssl memory manager --- Modified: modules/tls/tls_init.c --- Diff: https://github.com/kamailio/kamailio/commit/8bec45f3d0581747da4b807229b4de6… Patch: https://github.com/kamailio/kamailio/commit/8bec45f3d0581747da4b807229b4de6… --- diff --git a/modules/tls/tls_init.c b/modules/tls/tls_init.c index e0bbb0b..118f0ab 100644 --- a/modules/tls/tls_init.c +++ b/modules/tls/tls_init.c @@ -488,6 +488,8 @@ int tls_pre_init(void) if (!CRYPTO_set_mem_functions(ser_malloc, ser_realloc, ser_free)) { #endif ERR("Unable to set the memory allocation functions\n"); + ERR("Be sure tls module is loaded before any other module using libssl" + " (can be loaded first to be safe)\n"); return -1; } @@ -509,7 +511,7 @@ int tls_mod_pre_init_h(void) LM_DBG("already mod pre-initialized\n"); return 0; } - DBG("============= :preparing tls env for modules initialization\n"); + DBG("preparing tls env for modules initialization\n"); SSL_library_init(); SSL_load_error_strings(); tls_mod_preinitialized=1;

8 years

git:master:80f7ac2b: Merge pull request #709 from athonet-open/fix_vendor_id_on_auth_session

by GitHub

Module: kamailio Branch: master Commit: 80f7ac2b85bb10f72ce088dd406b9011866d9fef URL: https://github.com/kamailio/kamailio/commit/80f7ac2b85bb10f72ce088dd406b901… Author: jaybeepee <jason.penton(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2016-07-14T09:15:19+02:00 Merge pull request #709 from athonet-open/fix_vendor_id_on_auth_session ims_qos: Store vendor_id on auth session --- Modified: modules/ims_qos/mod.c --- Diff: https://github.com/kamailio/kamailio/commit/80f7ac2b85bb10f72ce088dd406b901… Patch: https://github.com/kamailio/kamailio/commit/80f7ac2b85bb10f72ce088dd406b901… --- diff --git a/modules/ims_qos/mod.c b/modules/ims_qos/mod.c index 1021672..6cf9ae3 100644 --- a/modules/ims_qos/mod.c +++ b/modules/ims_qos/mod.c @@ -1055,6 +1055,7 @@ static int w_rx_aar(struct sip_msg *msg, char *route, char* dir, char *c_id, int dlgb.set_dlg_var(&callid, &ftag, &ttag, &term_session_key, &auth_session->id); } + auth_session->vendor_id = IMS_vendor_id_3GPP; LM_DBG("Attached CDP auth session [%.*s] for Rx to dialog in %s mode\n", auth_session->id.len, auth_session->id.s, direction); } else { LM_DBG("Update AAR session for this dialog in mode %s\n", direction); @@ -1333,6 +1334,7 @@ static int w_rx_aar_register(struct sip_msg *msg, char* route, char* str1, char* goto error; } auth->u.auth.class = AUTH_CLASS_RXREG; + auth->vendor_id = IMS_vendor_id_3GPP; } //we are ready to send the AAR async. lets save the local data data

8 years

git:master:692aac44: ims_qos: Store vendor_id on auth session

by Federico Favaro

Module: kamailio Branch: master Commit: 692aac449c73e83b01c1bbe792c089ac60bb89a9 URL: https://github.com/kamailio/kamailio/commit/692aac449c73e83b01c1bbe792c089a… Author: Federico Favaro <federico(a)aereo.lan.athonet.com> Committer: Federico Favaro <federico(a)aereo.lan.athonet.com> Date: 2016-07-13T15:55:35+02:00 ims_qos: Store vendor_id on auth session When creating new Auth Session for both signalling and media, vendor_id was never assigned. If auth timer expires, CDP will fail to route correctly STR based on vendor ID 0 --- Modified: modules/ims_qos/mod.c --- Diff: https://github.com/kamailio/kamailio/commit/692aac449c73e83b01c1bbe792c089a… Patch: https://github.com/kamailio/kamailio/commit/692aac449c73e83b01c1bbe792c089a… --- diff --git a/modules/ims_qos/mod.c b/modules/ims_qos/mod.c index 1021672..6cf9ae3 100644 --- a/modules/ims_qos/mod.c +++ b/modules/ims_qos/mod.c @@ -1055,6 +1055,7 @@ static int w_rx_aar(struct sip_msg *msg, char *route, char* dir, char *c_id, int dlgb.set_dlg_var(&callid, &ftag, &ttag, &term_session_key, &auth_session->id); } + auth_session->vendor_id = IMS_vendor_id_3GPP; LM_DBG("Attached CDP auth session [%.*s] for Rx to dialog in %s mode\n", auth_session->id.len, auth_session->id.s, direction); } else { LM_DBG("Update AAR session for this dialog in mode %s\n", direction); @@ -1333,6 +1334,7 @@ static int w_rx_aar_register(struct sip_msg *msg, char* route, char* str1, char* goto error; } auth->u.auth.class = AUTH_CLASS_RXREG; + auth->vendor_id = IMS_vendor_id_3GPP; } //we are ready to send the AAR async. lets save the local data data

8 years

[kamailio/kamailio] ims_qos: Store vendor_id on auth session (#709)

by Federico Favaro

When creating new Auth Session for both signalling and media, vendor_id was never assigned. If auth timer expires, CDP will fail to route correctly STR based on vendor ID 0 You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/709 -- Commit Summary -- * ims_qos: Store vendor_id on auth session -- File Changes -- M modules/ims_qos/mod.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/709.patch https://github.com/kamailio/kamailio/pull/709.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/709

8 years

git:master:d3bdf14a: tm: wrap uri values in brackets inside error messages

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: d3bdf14a2e0ae113092aa33c89698131c020d209 URL: https://github.com/kamailio/kamailio/commit/d3bdf14a2e0ae113092aa33c8969813… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2016-07-14T08:43:05+02:00 tm: wrap uri values in brackets inside error messages - easier to spot invalid empty chars - useless spaces removed --- Modified: modules/tm/ut.h --- Diff: https://github.com/kamailio/kamailio/commit/d3bdf14a2e0ae113092aa33c8969813… Patch: https://github.com/kamailio/kamailio/commit/d3bdf14a2e0ae113092aa33c8969813…

8 years

git:4.3:ad531a9c: janssonrpc-c: init cfg framework for additional process

by Victor Seva

Module: kamailio Branch: 4.3 Commit: ad531a9c5f817a0319b3c54295f059c8da7216b5 URL: https://github.com/kamailio/kamailio/commit/ad531a9c5f817a0319b3c54295f059c… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2016-07-13T17:32:25+02:00 janssonrpc-c: init cfg framework for additional process - revealed by GH #708 (cherry picked from commit 5d8d565331111b80d72a20d19f2bcb14ea6301a4) --- Modified: modules/janssonrpc-c/janssonrpc_io.c Modified: modules/janssonrpc-c/janssonrpc_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/ad531a9c5f817a0319b3c54295f059c… Patch: https://github.com/kamailio/kamailio/commit/ad531a9c5f817a0319b3c54295f059c… --- diff --git a/modules/janssonrpc-c/janssonrpc_io.c b/modules/janssonrpc-c/janssonrpc_io.c index 81ce22a..0aa2836 100644 --- a/modules/janssonrpc-c/janssonrpc_io.c +++ b/modules/janssonrpc-c/janssonrpc_io.c @@ -39,6 +39,7 @@ #include "../../action.h" #include "../../route_struct.h" #include "../../lvalue.h" +#include "../../cfg/cfg_struct.h" #include "../../rand/fastrand.h" #include "../tm/tm_load.h" #include "../jansson/jansson_utils.h" @@ -57,11 +58,15 @@ void io_shutdown(int sig); int jsonrpc_io_child_process(int cmd_pipe) { + struct event* pipe_ev = NULL; + global_ev_base = event_base_new(); global_evdns_base = evdns_base_new(global_ev_base, 1); set_non_blocking(cmd_pipe); - struct event* pipe_ev = event_new(global_ev_base, cmd_pipe, EV_READ | EV_PERSIST, cmd_pipe_cb, NULL); + pipe_ev = event_new(global_ev_base, cmd_pipe, + EV_READ | EV_PERSIST, cmd_pipe_cb, NULL); + if(!pipe_ev) { ERR("Failed to create pipe event\n"); return -1; @@ -449,6 +454,7 @@ void cmd_pipe_cb(int fd, short event, void *arg) return; } + cfg_update(); switch(cmd->type) { case CMD_CLOSE: diff --git a/modules/janssonrpc-c/janssonrpc_mod.c b/modules/janssonrpc-c/janssonrpc_mod.c index 7db22cc..c69b843 100644 --- a/modules/janssonrpc-c/janssonrpc_mod.c +++ b/modules/janssonrpc-c/janssonrpc_mod.c @@ -29,6 +29,7 @@ #include "../../trim.h" #include "../../sr_module.h" #include "../../timer_proc.h" +#include "../../cfg/cfg_struct.h" #include "../tm/tm_load.h" #include "../jansson/jansson_utils.h" @@ -194,6 +195,7 @@ static int child_init(int rank) return -1; /* error */ if(pid==0){ /* child */ + if (cfg_child_init()) return -1; close(pipe_fds[1]); return jsonrpc_io_child_process(pipe_fds[0]); }

8 years

git:4.3:afa1f484: tm: execut cfg_update() inside t_continue()

by Victor Seva

Module: kamailio Branch: 4.3 Commit: afa1f48417b7837e468285b7455903de9d832461 URL: https://github.com/kamailio/kamailio/commit/afa1f48417b7837e468285b7455903d… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2016-07-13T17:32:25+02:00 tm: execut cfg_update() inside t_continue() - refresh the values changed inside cfg framework - reported by Victor Seva, GH #708 (cherry picked from commit c7bec3320f6907c4edd8416a5e5cfa89b071a411) --- Modified: modules/tm/t_suspend.c --- Diff: https://github.com/kamailio/kamailio/commit/afa1f48417b7837e468285b7455903d… Patch: https://github.com/kamailio/kamailio/commit/afa1f48417b7837e468285b7455903d… --- diff --git a/modules/tm/t_suspend.c b/modules/tm/t_suspend.c index d0d854d..0837562 100644 --- a/modules/tm/t_suspend.c +++ b/modules/tm/t_suspend.c @@ -22,6 +22,7 @@ #include "../../action.h" #include "../../script_cb.h" #include "../../dset.h" +#include "../../cfg/cfg_struct.h" #include "config.h" #include "sip_msg.h" @@ -177,6 +178,8 @@ int t_continue(unsigned int hash_index, unsigned int label, int do_put_on_wait; struct hdr_field *hdr, *prev = 0, *tmp = 0; + cfg_update(); + if (t_lookup_ident(&t, hash_index, label) < 0) { LM_ERR("transaction not found\n"); return -1;

8 years

git:4.4:12777af8: janssonrpc-c: init cfg framework for additional process

by Victor Seva

Module: kamailio Branch: 4.4 Commit: 12777af81509e3f6aa3e88c6aacb8844403d1e0b URL: https://github.com/kamailio/kamailio/commit/12777af81509e3f6aa3e88c6aacb884… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2016-07-13T17:31:52+02:00 janssonrpc-c: init cfg framework for additional process - revealed by GH #708 (cherry picked from commit 5d8d565331111b80d72a20d19f2bcb14ea6301a4) --- Modified: modules/janssonrpc-c/janssonrpc_io.c Modified: modules/janssonrpc-c/janssonrpc_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/12777af81509e3f6aa3e88c6aacb884… Patch: https://github.com/kamailio/kamailio/commit/12777af81509e3f6aa3e88c6aacb884… --- diff --git a/modules/janssonrpc-c/janssonrpc_io.c b/modules/janssonrpc-c/janssonrpc_io.c index 1e1a82b..ed0be6c 100644 --- a/modules/janssonrpc-c/janssonrpc_io.c +++ b/modules/janssonrpc-c/janssonrpc_io.c @@ -39,6 +39,7 @@ #include "../../action.h" #include "../../route_struct.h" #include "../../lvalue.h" +#include "../../cfg/cfg_struct.h" #include "../../rand/fastrand.h" #include "../tm/tm_load.h" #include "../jansson/jansson_utils.h" @@ -57,11 +58,15 @@ void io_shutdown(int sig); int jsonrpc_io_child_process(int cmd_pipe) { + struct event* pipe_ev = NULL; + global_ev_base = event_base_new(); global_evdns_base = evdns_base_new(global_ev_base, 1); set_non_blocking(cmd_pipe); - struct event* pipe_ev = event_new(global_ev_base, cmd_pipe, EV_READ | EV_PERSIST, cmd_pipe_cb, NULL); + pipe_ev = event_new(global_ev_base, cmd_pipe, + EV_READ | EV_PERSIST, cmd_pipe_cb, NULL); + if(!pipe_ev) { ERR("Failed to create pipe event\n"); return -1; @@ -449,6 +454,7 @@ void cmd_pipe_cb(int fd, short event, void *arg) return; } + cfg_update(); switch(cmd->type) { case CMD_CLOSE: diff --git a/modules/janssonrpc-c/janssonrpc_mod.c b/modules/janssonrpc-c/janssonrpc_mod.c index 7db22cc..c69b843 100644 --- a/modules/janssonrpc-c/janssonrpc_mod.c +++ b/modules/janssonrpc-c/janssonrpc_mod.c @@ -29,6 +29,7 @@ #include "../../trim.h" #include "../../sr_module.h" #include "../../timer_proc.h" +#include "../../cfg/cfg_struct.h" #include "../tm/tm_load.h" #include "../jansson/jansson_utils.h" @@ -194,6 +195,7 @@ static int child_init(int rank) return -1; /* error */ if(pid==0){ /* child */ + if (cfg_child_init()) return -1; close(pipe_fds[1]); return jsonrpc_io_child_process(pipe_fds[0]); }

8 years

git:4.4:c8661094: tm: execut cfg_update() inside t_continue()

by Victor Seva

Module: kamailio Branch: 4.4 Commit: c86610943d5c84e0c6ae8fd9e8d40f29e936b4ea URL: https://github.com/kamailio/kamailio/commit/c86610943d5c84e0c6ae8fd9e8d40f2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2016-07-13T17:31:52+02:00 tm: execut cfg_update() inside t_continue() - refresh the values changed inside cfg framework - reported by Victor Seva, GH #708 (cherry picked from commit c7bec3320f6907c4edd8416a5e5cfa89b071a411) --- Modified: modules/tm/t_suspend.c --- Diff: https://github.com/kamailio/kamailio/commit/c86610943d5c84e0c6ae8fd9e8d40f2… Patch: https://github.com/kamailio/kamailio/commit/c86610943d5c84e0c6ae8fd9e8d40f2… --- diff --git a/modules/tm/t_suspend.c b/modules/tm/t_suspend.c index e8096bb..6bb2547 100644 --- a/modules/tm/t_suspend.c +++ b/modules/tm/t_suspend.c @@ -22,6 +22,7 @@ #include "../../action.h" #include "../../script_cb.h" #include "../../dset.h" +#include "../../cfg/cfg_struct.h" #include "config.h" #include "sip_msg.h" @@ -178,6 +179,8 @@ int t_continue(unsigned int hash_index, unsigned int label, struct hdr_field *hdr, *prev = 0, *tmp = 0; int route_type_bk; + cfg_update(); + if (t_lookup_ident(&t, hash_index, label) < 0) { LM_ERR("transaction not found\n"); return -1;

8 years

[kamailio/kamailio] core.debug value changed via fifo/rpc gets lots after t_suspend()/t_continue() (#708)

by Victor Seva

in kamailio.cfg I have `` debug=1 `` setting debug level to 2 in the command line `` kamctl fifo debug 2 `` or `` kamcmd proxy cfg.set_now_int core debug 2 `` works fine until the process does t_suspend() after t_continue() logs are debug level 1 only --- 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/708

8 years

← Newer
1
...
9
10
11
12
13
14
15
...
30
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev July 2016