sr-dev March 2024

sr-dev@lists.kamailio.org

14 participants
238 discussions

git:master:8a2894cd: core: main - init local variables used for cli param parsing

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 8a2894cd6180f904dd37acb4985db4e4935aa434 URL: https://github.com/kamailio/kamailio/commit/8a2894cd6180f904dd37acb4985db4e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-04T15:31:47+01:00 core: main - init local variables used for cli param parsing --- Modified: src/main.c --- Diff: https://github.com/kamailio/kamailio/commit/8a2894cd6180f904dd37acb4985db4e… Patch: https://github.com/kamailio/kamailio/commit/8a2894cd6180f904dd37acb4985db4e… --- diff --git a/src/main.c b/src/main.c index 1856bf7bc03..c7e98e1b433 100644 --- a/src/main.c +++ b/src/main.c @@ -2083,9 +2083,9 @@ int main(int argc, char **argv) int c, r; char *tmp; int tmp_len; - int port; - int proto; - int aproto; + int port = 5060; + int proto = PROTO_NONE; + int aproto = PROTO_NONE; char *ahost = NULL; int aport = 0; char *options;

4 months, 1 week

git:5.8:0ba1ff1d: tls: new option tls_threads_mode = 2

by S-P Chan

Module: kamailio Branch: 5.8 Commit: 0ba1ff1d430e5e51c7745d7eb1242dcae2557db9 URL: https://github.com/kamailio/kamailio/commit/0ba1ff1d430e5e51c7745d7eb1242dc… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-03-04T22:27:36+08:00 tls: new option tls_threads_mode = 2 - use pthread_atfork to force all thread-locals to 0x0 after fork() (cherry picked from commit 464299c202f3ba963aed821b777075397e843856) --- Modified: src/modules/tls/tls_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/0ba1ff1d430e5e51c7745d7eb1242dc… Patch: https://github.com/kamailio/kamailio/commit/0ba1ff1d430e5e51c7745d7eb1242dc… --- diff --git a/src/modules/tls/tls_mod.c b/src/modules/tls/tls_mod.c index 1dc9357f56a..6dfde00c93a 100644 --- a/src/modules/tls/tls_mod.c +++ b/src/modules/tls/tls_mod.c @@ -337,6 +337,20 @@ static tls_domains_cfg_t* tls_use_modparams(void) } #endif +/* global config tls_threads_mode = 2 + * - force all thread-locals to be 0x0 after fork() + * - with OpenSSL loaded the largest value observed + * is < 10 + * + */ +static void fork_child(void) +{ + for(int k = 0; k < 16; k++) { + if(pthread_getspecific(k) != 0) + pthread_setspecific(k, 0x0); + } +} + static int mod_init(void) { int method; @@ -446,6 +460,9 @@ static int mod_init(void) ksr_module_set_flag(KSRMOD_FLAG_POSTCHILDINIT); } #endif + if(ksr_tls_threads_mode == 2) { + pthread_atfork(NULL, NULL, &fork_child); + } return 0; error: tls_h_mod_destroy_f();

4 months, 1 week

git:5.8:0a565d34: core/rthreads.h: add new option tls_threads_mode = 2

by S-P Chan

Module: kamailio Branch: 5.8 Commit: 0a565d34b98b7ec4de0524a3e216e8d907661181 URL: https://github.com/kamailio/kamailio/commit/0a565d34b98b7ec4de0524a3e216e8d… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-03-04T22:27:15+08:00 core/rthreads.h: add new option tls_threads_mode = 2 - add global handling of thread-locals with tls_threads_mode = 2 - this will run a pthread_atfork handler to reset all thread-locals to 0x0 - alternative solution to running functions in thread executors - requires tls.so to be loaded to be effective (cherry picked from commit e7f040f219b46592081a6053b4ed1ae0d0552b1a) --- Modified: src/core/rthreads.h --- Diff: https://github.com/kamailio/kamailio/commit/0a565d34b98b7ec4de0524a3e216e8d… Patch: https://github.com/kamailio/kamailio/commit/0a565d34b98b7ec4de0524a3e216e8d… --- diff --git a/src/core/rthreads.h b/src/core/rthreads.h index 0f4f0cf8b8a..6fee4d0b407 100644 --- a/src/core/rthreads.h +++ b/src/core/rthreads.h @@ -40,7 +40,7 @@ static void *run_threadP(_thread_proto fn, void *arg) pthread_t tid; void *ret; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg); } @@ -77,7 +77,7 @@ static void *run_threadPI(_thread_protoPI fn, void *arg1, int arg2) pthread_t tid; void *ret; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg1, arg2); } @@ -113,7 +113,7 @@ static void run_threadV(_thread_protoV fn) #ifdef USE_TLS pthread_t tid; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { fn(); return; @@ -152,7 +152,7 @@ static int run_thread4PP(_thread_proto4PP fn, void *arg1, void *arg2) pthread_t tid; int ret; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg1, arg2); } @@ -188,7 +188,7 @@ static void run_thread0P(_thread_proto0P fn, void *arg1) #ifdef USE_TLS pthread_t tid; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { fn(arg1); return; @@ -240,7 +240,7 @@ static int run_thread4P5I2P2(_thread_proto4P5I2P2 fn, void *arg1, void *arg2, pthread_t tid; int ret; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9); } @@ -278,7 +278,7 @@ static int run_thread4L(_thread_proto4L fn, long arg1) pthread_t tid; int ret; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg1); }

4 months, 1 week

git:master:464299c2: tls: new option tls_threads_mode = 2

by S-P Chan

Module: kamailio Branch: master Commit: 464299c202f3ba963aed821b777075397e843856 URL: https://github.com/kamailio/kamailio/commit/464299c202f3ba963aed821b7770753… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-03-04T22:00:38+08:00 tls: new option tls_threads_mode = 2 - use pthread_atfork to force all thread-locals to 0x0 after fork() --- Modified: src/modules/tls/tls_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/464299c202f3ba963aed821b7770753… Patch: https://github.com/kamailio/kamailio/commit/464299c202f3ba963aed821b7770753… --- diff --git a/src/modules/tls/tls_mod.c b/src/modules/tls/tls_mod.c index 1dc9357f56a..6dfde00c93a 100644 --- a/src/modules/tls/tls_mod.c +++ b/src/modules/tls/tls_mod.c @@ -337,6 +337,20 @@ static tls_domains_cfg_t* tls_use_modparams(void) } #endif +/* global config tls_threads_mode = 2 + * - force all thread-locals to be 0x0 after fork() + * - with OpenSSL loaded the largest value observed + * is < 10 + * + */ +static void fork_child(void) +{ + for(int k = 0; k < 16; k++) { + if(pthread_getspecific(k) != 0) + pthread_setspecific(k, 0x0); + } +} + static int mod_init(void) { int method; @@ -446,6 +460,9 @@ static int mod_init(void) ksr_module_set_flag(KSRMOD_FLAG_POSTCHILDINIT); } #endif + if(ksr_tls_threads_mode == 2) { + pthread_atfork(NULL, NULL, &fork_child); + } return 0; error: tls_h_mod_destroy_f();

4 months, 1 week

git:master:e7f040f2: core/rthreads.h: add new option tls_threads_mode = 2

by S-P Chan

Module: kamailio Branch: master Commit: e7f040f219b46592081a6053b4ed1ae0d0552b1a URL: https://github.com/kamailio/kamailio/commit/e7f040f219b46592081a6053b4ed1ae… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-03-04T22:00:38+08:00 core/rthreads.h: add new option tls_threads_mode = 2 - add global handling of thread-locals with tls_threads_mode = 2 - this will run a pthread_atfork handler to reset all thread-locals to 0x0 - alternative solution to running functions in thread executors - requires tls.so to be loaded to be effective --- Modified: src/core/rthreads.h --- Diff: https://github.com/kamailio/kamailio/commit/e7f040f219b46592081a6053b4ed1ae… Patch: https://github.com/kamailio/kamailio/commit/e7f040f219b46592081a6053b4ed1ae… --- diff --git a/src/core/rthreads.h b/src/core/rthreads.h index 1a04f9982c7..56d68cef276 100644 --- a/src/core/rthreads.h +++ b/src/core/rthreads.h @@ -40,7 +40,7 @@ static void *run_threadP(_thread_proto fn, void *arg) pthread_t tid; void *ret = NULL; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg); } @@ -77,7 +77,7 @@ static void *run_threadPI(_thread_protoPI fn, void *arg1, int arg2) pthread_t tid; void *ret = NULL; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg1, arg2); } @@ -113,7 +113,7 @@ static void run_threadV(_thread_protoV fn) #ifdef USE_TLS pthread_t tid; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { fn(); return; @@ -152,7 +152,7 @@ static int run_thread4PP(_thread_proto4PP fn, void *arg1, void *arg2) pthread_t tid; int ret = 0; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg1, arg2); } @@ -188,7 +188,7 @@ static void run_thread0P(_thread_proto0P fn, void *arg1) #ifdef USE_TLS pthread_t tid; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { fn(arg1); return; @@ -240,7 +240,7 @@ static int run_thread4P5I2P2(_thread_proto4P5I2P2 fn, void *arg1, void *arg2, pthread_t tid; int ret = 0; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg1, arg2, arg3, arg4, arg5, arg6, arg7, arg8, arg9); } @@ -278,7 +278,7 @@ static int run_thread4L(_thread_proto4L fn, long arg1) pthread_t tid; int ret = 0; - if(likely(ksr_tls_threads_mode == 0 + if(likely(ksr_tls_threads_mode == 0 || ksr_tls_threads_mode == 2 || (ksr_tls_threads_mode == 1 && process_no > 0))) { return fn(arg1); }

4 months, 1 week

git:master:5b8b2717: ctl: reset log prefix on reading ctl traffic

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 5b8b2717ee2f57da932132683b27eb33cea3fd59 URL: https://github.com/kamailio/kamailio/commit/5b8b2717ee2f57da932132683b27eb3… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-04T13:58:35+01:00 ctl: reset log prefix on reading ctl traffic - it may be previously set in process by some event route --- Modified: src/modules/ctl/fifo_server.c Modified: src/modules/ctl/io_listener.c --- Diff: https://github.com/kamailio/kamailio/commit/5b8b2717ee2f57da932132683b27eb3… Patch: https://github.com/kamailio/kamailio/commit/5b8b2717ee2f57da932132683b27eb3… --- diff --git a/src/modules/ctl/fifo_server.c b/src/modules/ctl/fifo_server.c index 772441d31e4..94a700cf50e 100644 --- a/src/modules/ctl/fifo_server.c +++ b/src/modules/ctl/fifo_server.c @@ -736,6 +736,7 @@ int fifo_process(char *msg_buf, int size, int *bytes_needed, void *sh, static rpc_ctx_t context; unsigned int rdata; + log_prefix_set(NULL); DBG("process_fifo: called with %d bytes, offset %d: %.*s\n", size, (int)(long)*saved_state, size, msg_buf); /* search for the end of the request (\n\r) */ diff --git a/src/modules/ctl/io_listener.c b/src/modules/ctl/io_listener.c index ed584e4c7f9..3634b63acff 100644 --- a/src/modules/ctl/io_listener.c +++ b/src/modules/ctl/io_listener.c @@ -26,6 +26,7 @@ #include "../../core/mem/mem.h" #include "../../core/rpc.h" /* who & ls rpcs */ #include "../../core/ut.h" +#include "../../core/action.h" #include "../../core/cfg/cfg_struct.h" #include "ctrl_socks.h" @@ -346,6 +347,7 @@ static int handle_ctrl_dgram(struct ctrl_socket *cs) struct send_handle sh; void *saved_state; + log_prefix_set(NULL); saved_state = 0; /* we get always a new datagram */ sh.fd = cs->fd; sh.type = S_DISCONNECTED; @@ -486,6 +488,7 @@ static int handle_stream_read(struct stream_connection *s_c, int idx) sh.from_len = 0; r = &s_c->req; bytes_free = STREAM_BUF_SIZE - (int)(r->end - r->buf); + log_prefix_set(NULL); if(bytes_free == 0) { LOG(L_ERR, "ERROR: handle_stream_read: buffer overrun\n"); goto close_connection;

4 months, 1 week

git:master:0fc0cdd7: acc: cdr - do not free static empty string in case of error

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 0fc0cdd791ce5157232dd2139238708403cdde1d URL: https://github.com/kamailio/kamailio/commit/0fc0cdd791ce5157232dd2139238708… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-04T09:07:22+01:00 acc: cdr - do not free static empty string in case of error --- Modified: src/modules/acc/acc_cdr.c --- Diff: https://github.com/kamailio/kamailio/commit/0fc0cdd791ce5157232dd2139238708… Patch: https://github.com/kamailio/kamailio/commit/0fc0cdd791ce5157232dd2139238708… --- diff --git a/src/modules/acc/acc_cdr.c b/src/modules/acc/acc_cdr.c index 4ec019f1f8d..4c003e0ecab 100644 --- a/src/modules/acc/acc_cdr.c +++ b/src/modules/acc/acc_cdr.c @@ -113,10 +113,10 @@ int cdr_core2strar(struct dlg_cell *dlg, str *values, int *unused, char *types) /* cleanup already allocated memory and * return that we didn't do anything */ for(i = i - 1; i >= 0; i--) { - if(NULL != values[i].s) { + if(NULL != values[i].s && types[i] != TYPE_NULL) { pkg_free(values[i].s); - values[i].s = NULL; } + values[i].s = NULL; } return 0; }

4 months, 1 week

git:master:6ee26a3a: core: parser rr - check for rr value before serializing

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 6ee26a3ae3293aaeb4507d5d0b16ad31173cc39a URL: https://github.com/kamailio/kamailio/commit/6ee26a3ae3293aaeb4507d5d0b16ad3… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-04T08:57:47+01:00 core: parser rr - check for rr value before serializing --- Modified: src/core/parser/parse_rr.c --- Diff: https://github.com/kamailio/kamailio/commit/6ee26a3ae3293aaeb4507d5d0b16ad3… Patch: https://github.com/kamailio/kamailio/commit/6ee26a3ae3293aaeb4507d5d0b16ad3… --- diff --git a/src/core/parser/parse_rr.c b/src/core/parser/parse_rr.c index f6fce793faf..362f30b15ed 100644 --- a/src/core/parser/parse_rr.c +++ b/src/core/parser/parse_rr.c @@ -444,20 +444,28 @@ int print_rr_body( i = (nb_recs == NULL) ? 0 : *nb_recs; while(i < n) { - memcpy(cp, route[i].s, route[i].len); - cp += route[i].len; - if(++i < n) - *(cp++) = ','; + if(route[i].s != NULL) { + memcpy(cp, route[i].s, route[i].len); + cp += route[i].len; + if(++i < n) + *(cp++) = ','; + } else { + i++; + } } } else { i = (nb_recs == NULL) ? n - 1 : (n - *nb_recs - 1); while(i >= 0) { - memcpy(cp, route[i].s, route[i].len); - cp += route[i].len; - if(i-- > 0) - *(cp++) = ','; + if(route[i].s != NULL) { + memcpy(cp, route[i].s, route[i].len); + cp += route[i].len; + if(i-- > 0) + *(cp++) = ','; + } else { + i--; + } } } oroute->len = cp - start;

4 months, 1 week

Heads-up: slightly larger tls commit for the 5.8 cycle

by Richard Chan

Hi sr-dev Just a heads-up that I have included a (slightly larger than normal) commit to 5.8. This offers provider support (v3) in parallel to ENGINE(v1.1.1). - provides basic infrastructure for OpenSSL 3 providers - it can be guarded with -DOPENSSL_NO_PROVIDER (which turns this commit off completely. Thanks for your patience with all the tls.so churn. Regards Richard

4 months, 2 weeks

git:5.8:3d80bc81: tls: basic OpenSSL 3 support of provider keys (replaces ENGINE)

by S-P Chan

Module: kamailio Branch: 5.8 Commit: 3d80bc811c75eb0cbcf0f4aee8929a5082b4de6a URL: https://github.com/kamailio/kamailio/commit/3d80bc811c75eb0cbcf0f4aee8929a5… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-03-03T14:38:56+08:00 tls: basic OpenSSL 3 support of provider keys (replaces ENGINE) - initial support for v3 provider keys (replaces ENGINE from v1.1.1) - can be disabled behind build flag -DOPENSSL_NO_PROVIDER - provider keys start with /uri: e.g private_key = /uri:pkcs11:token=NSS%20Certificate%20DB;type=private;object=Fork-Test-c67cc0e0 - global config: provider_quirks: 0 | 1 - 0 - default - 1 - create a new OSS_LIB_CTX* in the child This integration does not load any providers itself and depends on the usual export OPENSSL_CONF=my-openssl.cnf to configure providers. (cherry picked from commit 69883dd381368ca219cc52140e71d571775f95d5) --- Modified: src/modules/tls/tls_domain.c Modified: src/modules/tls/tls_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/3d80bc811c75eb0cbcf0f4aee8929a5… Patch: https://github.com/kamailio/kamailio/commit/3d80bc811c75eb0cbcf0f4aee8929a5…

4 months, 2 weeks

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev March 2024