sr-dev March 2024

sr-dev@lists.kamailio.org

14 participants
238 discussions

git:master:5933893b: tls: clean-up of ENGINE

by S-P Chan

Module: kamailio Branch: master Commit: 5933893b919bccdc1c35e7bb0505e3a49d72c403 URL: https://github.com/kamailio/kamailio/commit/5933893b919bccdc1c35e7bb0505e3a… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-03-02T08:46:39+08:00 tls: clean-up of ENGINE - remove tls_map.* - not needed anymore - install an ENGINE in each worker SSL_CTX no need to replicate to all processes --- Modified: src/modules/tls/tls_domain.c Modified: src/modules/tls/tls_server.c Removed: src/modules/tls/tls_map.c Removed: src/modules/tls/tls_map.h --- Diff: https://github.com/kamailio/kamailio/commit/5933893b919bccdc1c35e7bb0505e3a… Patch: https://github.com/kamailio/kamailio/commit/5933893b919bccdc1c35e7bb0505e3a…

4 months, 2 weeks

git:master:d8e0942c: core: select - handle ws and wss inside select_ip_port()

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: d8e0942c9e83c8cad7c182bf41c156ba35bf24d2 URL: https://github.com/kamailio/kamailio/commit/d8e0942c9e83c8cad7c182bf41c156b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-01T12:46:17+01:00 core: select - handle ws and wss inside select_ip_port() --- Modified: src/core/select_core.c --- Diff: https://github.com/kamailio/kamailio/commit/d8e0942c9e83c8cad7c182bf41c156b… Patch: https://github.com/kamailio/kamailio/commit/d8e0942c9e83c8cad7c182bf41c156b… --- diff --git a/src/core/select_core.c b/src/core/select_core.c index 12b884ead04..57dad2f78e5 100644 --- a/src/core/select_core.c +++ b/src/core/select_core.c @@ -1387,7 +1387,7 @@ int select_ip_port(str *res, select_t *s, struct sip_msg *msg) if(param & SEL_PROTO) { switch(msg->rcv.proto) { case PROTO_NONE: - proto_str.s = 0; + proto_str.s = ""; proto_str.len = 0; break; @@ -1411,6 +1411,16 @@ int select_ip_port(str *res, select_t *s, struct sip_msg *msg) proto_str.len = 4; break; + case PROTO_WS: + proto_str.s = "ws"; + proto_str.len = 2; + break; + + case PROTO_WSS: + proto_str.s = "wss"; + proto_str.len = 3; + break; + default: LM_ERR("Unknown transport protocol\n"); return -1;

4 months, 2 weeks

git:master:daac0bdb: core: ppcfg - jump to end when defexp eval does not get a str val

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: daac0bdb88f6f732bda7c683af45b90f87fdb7c9 URL: https://github.com/kamailio/kamailio/commit/daac0bdb88f6f732bda7c683af45b90… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-01T12:41:13+01:00 core: ppcfg - jump to end when defexp eval does not get a str val --- Modified: src/core/ppcfg.c --- Diff: https://github.com/kamailio/kamailio/commit/daac0bdb88f6f732bda7c683af45b90… Patch: https://github.com/kamailio/kamailio/commit/daac0bdb88f6f732bda7c683af45b90… --- diff --git a/src/core/ppcfg.c b/src/core/ppcfg.c index 262a7931cd2..d77a237882b 100644 --- a/src/core/ppcfg.c +++ b/src/core/ppcfg.c @@ -510,6 +510,8 @@ char *pp_defexp_eval(char *exval, int exlen, int qmode) LM_DBG("expression string result: [%s]\n", result->param.stz.sval); sval.s = result->param.stz.sval; sval.len = strlen(result->param.stz.sval); + } else { + goto done; } if(qmode == 1) {

4 months, 2 weeks

git:master:19894425: file_out: additional checks for silenting warnings

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 19894425684504e2a19622ae5a9cdfc565f16fd3 URL: https://github.com/kamailio/kamailio/commit/19894425684504e2a19622ae5a9cdfc… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-01T11:42:45+01:00 file_out: additional checks for silenting warnings --- Modified: src/modules/file_out/file_out.c --- Diff: https://github.com/kamailio/kamailio/commit/19894425684504e2a19622ae5a9cdfc… Patch: https://github.com/kamailio/kamailio/commit/19894425684504e2a19622ae5a9cdfc… --- diff --git a/src/modules/file_out/file_out.c b/src/modules/file_out/file_out.c index ad055f1852c..f3dcfb831d9 100644 --- a/src/modules/file_out/file_out.c +++ b/src/modules/file_out/file_out.c @@ -226,28 +226,31 @@ static void fo_log_writer_process(int rank) LM_ERR("deque error\n"); return; } - FILE *out = fo_get_file_handle(log_message.dest_file); - if(out == NULL) { - LM_ERR("file handle is NULL\n"); - return; - } + if(log_message.message != NULL) { + FILE *out = fo_get_file_handle(log_message.dest_file); + if(out == NULL) { + LM_ERR("file handle is NULL\n"); + return; + } - /* Get prefix for the file */ - if(log_message.prefix != NULL && log_message.prefix->len > 0) { - if(fprintf(out, "%.*s", log_message.prefix->len, - log_message.prefix->s) + /* Get prefix for the file */ + if(log_message.prefix != NULL && log_message.prefix->len > 0) { + if(fprintf(out, "%.*s", log_message.prefix->len, + log_message.prefix->s) + < 0) { + LM_ERR("Failed to write prefix to file with err {%s}\n", + strerror(errno)); + } + } + if(fprintf(out, "%.*s\n", log_message.message->len, + log_message.message->s) < 0) { - LM_ERR("Failed to write prefix to file with err {%s}\n", + LM_ERR("Failed to write to file with err {%s}\n", strerror(errno)); } - } - if(fprintf(out, "%.*s\n", log_message.message->len, - log_message.message->s) - < 0) { - LM_ERR("Failed to write to file with err {%s}\n", strerror(errno)); - } - if(fflush(out) < 0) { - LM_ERR("Failed to flush file with err {%s}\n", strerror(errno)); + if(fflush(out) < 0) { + LM_ERR("Failed to flush file with err {%s}\n", strerror(errno)); + } } if(log_message.prefix != NULL) { @@ -337,7 +340,11 @@ static int fo_fixup_free_int_pvar(void **param, int param_no) static int fo_add_filename(modparam_t type, void *val) { - if(val != NULL && strlen((char *)val) == 0) { + if(val == NULL) { + LM_ERR("modparam value is null\n"); + return -1; + } + if(strlen((char *)val) == 0) { LM_ERR("modparam value is empty\n"); return -1; }

4 months, 2 weeks

git:master:73c66150: influxdbc: copy with buffer size limit

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 73c66150c604326c07d454002c12cb221289936a URL: https://github.com/kamailio/kamailio/commit/73c66150c604326c07d454002c12cb2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-01T09:55:16+01:00 influxdbc: copy with buffer size limit --- Modified: src/modules/influxdbc/ic.c --- Diff: https://github.com/kamailio/kamailio/commit/73c66150c604326c07d454002c12cb2… Patch: https://github.com/kamailio/kamailio/commit/73c66150c604326c07d454002c12cb2… --- diff --git a/src/modules/influxdbc/ic.c b/src/modules/influxdbc/ic.c index 09ab462ffcb..dc493619f28 100644 --- a/src/modules/influxdbc/ic.c +++ b/src/modules/influxdbc/ic.c @@ -107,7 +107,7 @@ void ic_influx_database(char *host, long port, he = gethostbyname(host); if(he == NULL) { - sprintf(errorbuf, + snprintf(errorbuf, 1024, "influx host=%s to ip address convertion failed " "gethostbyname(), bailing out\n", host); @@ -115,22 +115,24 @@ void ic_influx_database(char *host, long port, } /* this could return multiple ip addresses but we assume its the first one */ if(he->h_addr_list[0] != NULL) { - strcpy(influx_ip, - inet_ntoa(*(struct in_addr *)(he->h_addr_list[0]))); + strncpy(influx_ip, + inet_ntoa(*(struct in_addr *)(he->h_addr_list[0])), 16); + influx_ip[16] = '\0'; DEBUG fprintf(stderr, "ic_influx_by_hostname hostname=%s converted to ip " "address %s))\n", host, influx_ip); } else { - sprintf(errorbuf, + snprintf(errorbuf, 1024, "influx host=%s to ip address convertion failed (empty " "list), bailing out\n", host); error(errorbuf); } } else { - strcpy(influx_ip, - host); /* perhaps the hostname is actually an ip address */ + strncpy(influx_ip, host, + 16); /* perhaps the hostname is actually an ip address */ + influx_ip[16] = '\0'; } } } @@ -338,7 +340,7 @@ void ic_push() } for(i = 0; i < 1024; i++) /* empty the buffer */ result[i] = 0; - if((ret = read(sockfd, result, sizeof(result))) > 0) { + if((ret = read(sockfd, result, sizeof(result) - 1)) > 0) { result[ret] = 0; DEBUG fprintf( stderr, "received bytes=%d data=<%s>\n", ret, result);

4 months, 2 weeks

git:master:181c6026: core: rthreads - init return variable

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 181c602616e13323786335e8e28f182871aa5e9b URL: https://github.com/kamailio/kamailio/commit/181c602616e13323786335e8e28f182… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-03-01T09:37:59+01:00 core: rthreads - init return variable --- Modified: src/core/rthreads.h --- Diff: https://github.com/kamailio/kamailio/commit/181c602616e13323786335e8e28f182… Patch: https://github.com/kamailio/kamailio/commit/181c602616e13323786335e8e28f182… --- diff --git a/src/core/rthreads.h b/src/core/rthreads.h index 0f4f0cf8b8a..1a04f9982c7 100644 --- a/src/core/rthreads.h +++ b/src/core/rthreads.h @@ -38,7 +38,7 @@ static void *run_threadP(_thread_proto fn, void *arg) { #ifdef USE_TLS pthread_t tid; - void *ret; + void *ret = NULL; if(likely(ksr_tls_threads_mode == 0 || (ksr_tls_threads_mode == 1 && process_no > 0))) { @@ -75,7 +75,7 @@ static void *run_threadPI(_thread_protoPI fn, void *arg1, int arg2) { #ifdef USE_TLS pthread_t tid; - void *ret; + void *ret = NULL; if(likely(ksr_tls_threads_mode == 0 || (ksr_tls_threads_mode == 1 && process_no > 0))) { @@ -150,7 +150,7 @@ static int run_thread4PP(_thread_proto4PP fn, void *arg1, void *arg2) { #ifdef USE_TLS pthread_t tid; - int ret; + int ret = 0; if(likely(ksr_tls_threads_mode == 0 || (ksr_tls_threads_mode == 1 && process_no > 0))) { @@ -238,7 +238,7 @@ static int run_thread4P5I2P2(_thread_proto4P5I2P2 fn, void *arg1, void *arg2, { #ifdef USE_TLS pthread_t tid; - int ret; + int ret = 0; if(likely(ksr_tls_threads_mode == 0 || (ksr_tls_threads_mode == 1 && process_no > 0))) { @@ -276,7 +276,7 @@ static int run_thread4L(_thread_proto4L fn, long arg1) { #ifdef USE_TLS pthread_t tid; - int ret; + int ret = 0; if(likely(ksr_tls_threads_mode == 0 || (ksr_tls_threads_mode == 1 && process_no > 0))) {

4 months, 2 weeks

git:5.8:0ddbd961: tls: make explicit ENGINE deprecation in OpenSSL 3

by S-P Chan

Module: kamailio Branch: 5.8 Commit: 0ddbd961e91abf3e967285d80cc1c6f8d2aa5cf9 URL: https://github.com/kamailio/kamailio/commit/0ddbd961e91abf3e967285d80cc1c6f… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-03-01T08:08:09+08:00 tls: make explicit ENGINE deprecation in OpenSSL 3 Cherry-pick from 0c68a5511e --- Modified: src/modules/tls/tls_domain.c Modified: src/modules/tls/tls_mod.c Modified: src/modules/tls/tls_server.c --- Diff: https://github.com/kamailio/kamailio/commit/0ddbd961e91abf3e967285d80cc1c6f… Patch: https://github.com/kamailio/kamailio/commit/0ddbd961e91abf3e967285d80cc1c6f…

4 months, 2 weeks

git:master:0c68a551: tls: make explicit ENGINE deprecation in OpenSSL 3

by S-P Chan

Module: kamailio Branch: master Commit: 0c68a5511e83e4367954172e3050fe997e47e9c8 URL: https://github.com/kamailio/kamailio/commit/0c68a5511e83e4367954172e3050fe9… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-03-01T08:06:13+08:00 tls: make explicit ENGINE deprecation in OpenSSL 3 --- Modified: src/modules/tls/tls_domain.c Modified: src/modules/tls/tls_mod.c Modified: src/modules/tls/tls_server.c --- Diff: https://github.com/kamailio/kamailio/commit/0c68a5511e83e4367954172e3050fe9… Patch: https://github.com/kamailio/kamailio/commit/0c68a5511e83e4367954172e3050fe9…

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