sr-dev February 2025

sr-dev@lists.kamailio.org

9 participants
163 discussions

git:master:69086114: utils/kamcmd: cmake fix on custom command
by Victor Seva 11 Feb '25

11 Feb '25

Module: kamailio Branch: master Commit: 69086114ad6eee016d1b7634f37c5ef31d4543e0 URL: https://github.com/kamailio/kamailio/commit/69086114ad6eee016d1b7634f37c5ef… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2025-02-11T01:35:56+01:00 utils/kamcmd: cmake fix on custom command --- Modified: utils/kamcmd/CMakeLists.txt --- Diff: https://github.com/kamailio/kamailio/commit/69086114ad6eee016d1b7634f37c5ef… Patch: https://github.com/kamailio/kamailio/commit/69086114ad6eee016d1b7634f37c5ef… --- diff --git a/utils/kamcmd/CMakeLists.txt b/utils/kamcmd/CMakeLists.txt index 0d1ec28b4c5..f984415ed44 100644 --- a/utils/kamcmd/CMakeLists.txt +++ b/utils/kamcmd/CMakeLists.txt @@ -46,7 +46,7 @@ add_custom_command( -e "s#/usr/sbin/#${CMAKE_INSTALL_FULL_SBINDIR}/#g" -e "s#/usr/lib/${CFG_NAME}/modules/#${CMAKE_INSTALL_FULL_LIBDIR}/${MAIN_NAME}/#g" -e "s#/usr/share/doc/${CFG_NAME}/#${CMAKE_INSTALL_FULL_DOCDIR}/#g" -e - "s#$(SRC_NAME)#$(MAIN_NAME)#g" < ${CMAKE_CURRENT_SOURCE_DIR}/kamcmd.8 > + "s#${SRC_NAME}#${MAIN_NAME}#g" < ${CMAKE_CURRENT_SOURCE_DIR}/kamcmd.8 > ${CMAKE_CURRENT_BINARY_DIR}/kamcmd.8 DEPENDS ${CMAKE_CURRENT_SOURCE_DIR}/kamcmd.8)

1 0

[kamailio/kamailio] app_python3: evaluate LIBS variable imediately in Makefile (PR #4085)
by sergey-safarov 08 Feb '25

08 Feb '25

#### Pre-Submission Checklist - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist: - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description latest `make` version does not evaluate variables immediately. PR explicitly evaluates the LIBS makefile variable. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4085 -- Commit Summary -- * app_python3: evaluate LIBS variable imediately in Makefile -- File Changes -- M src/modules/app_python3/Makefile (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4085.patch https://github.com/kamailio/kamailio/pull/4085.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4085 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4085(a)github.com>

5 96

git:master:342e1ad3: core: initial support for generic core parameter definition
by Daniel-Constantin Mierla 08 Feb '25

08 Feb '25

Module: kamailio Branch: master Commit: 342e1ad30872e23641134599ad0c519bfc6db7ce URL: https://github.com/kamailio/kamailio/commit/342e1ad30872e23641134599ad0c519… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-02-08T08:42:00+01:00 core: initial support for generic core parameter definition - allow coreparam[name]=value - simplify adding new core parameters without the need to define rules in lex and yacc --- Added: src/core/coreparam.c Added: src/core/coreparam.h Modified: src/core/cfg.lex Modified: src/core/cfg.y --- Diff: https://github.com/kamailio/kamailio/commit/342e1ad30872e23641134599ad0c519… Patch: https://github.com/kamailio/kamailio/commit/342e1ad30872e23641134599ad0c519…

1 0

[kamailio/kamailio] secfilter: search by prefix is dependant on order of added items (PR #4143)
by Simon Darienko 08 Feb '25

08 Feb '25

#### Pre-Submission Checklist - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist: - [x] PR should be backported to stable branches - [ ] Tested changes locally - [x] Related to issue #4141 #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4143 -- Commit Summary -- * secfilter: secf_check_dst function is dependant on order of items added to the list -- File Changes -- M src/modules/secfilter/secfilter.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4143.patch https://github.com/kamailio/kamailio/pull/4143.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4143 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4143(a)github.com>

1 4

[kamailio/kamailio] Secfilter `secf_check_dst` function is dependant on order of items added to the list (Issue #4141)
by Ihor Olkhovskyi 07 Feb '25

07 Feb '25

If destinations is being added in `secfilter` starting with the lowest length, all other subsequent destinations with higher length are ignored. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4141 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4141(a)github.com>

2 2

[kamailio/kamailio] secfilter: fix secf_check_dst bug (PR #4142)
by Jose Luis Verdeguer 07 Feb '25

07 Feb '25

#### Pre-Submission Checklist    - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [x] Related to issue #4141 (replace XXXX with an open issue number) #### Description  Fix problem with dst.len value You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4142 -- Commit Summary -- * secfilter: fix secf_check_dst bug -- File Changes -- M src/modules/secfilter/secfilter.c (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4142.patch https://github.com/kamailio/kamailio/pull/4142.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4142 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4142(a)github.com>

1 1

git:master:578c2abb: secfilter: fix secf_check_dst bug
by Jose Luis Verdeguer 07 Feb '25

07 Feb '25

Module: kamailio Branch: master Commit: 578c2abb36b9101bfc1dddc1505f7239e928b98e URL: https://github.com/kamailio/kamailio/commit/578c2abb36b9101bfc1dddc1505f723… Author: pepelux <pepeluxx(a)gmail.com> Committer: Jose Luis Verdeguer <pepeluxx(a)gmail.com> Date: 2025-02-07T19:23:39+01:00 secfilter: fix secf_check_dst bug --- Modified: src/modules/secfilter/secfilter.c --- Diff: https://github.com/kamailio/kamailio/commit/578c2abb36b9101bfc1dddc1505f723… Patch: https://github.com/kamailio/kamailio/commit/578c2abb36b9101bfc1dddc1505f723… --- diff --git a/src/modules/secfilter/secfilter.c b/src/modules/secfilter/secfilter.c index ff95ec74762..2518a7d7487 100644 --- a/src/modules/secfilter/secfilter.c +++ b/src/modules/secfilter/secfilter.c @@ -341,6 +341,7 @@ static int ki_check_dst(struct sip_msg *msg, str *val) } } else { /* Any match */ + dst.len = val->len; if(dst.len > list->s.len) dst.len = list->s.len; if(cmpi_str(&list->s, &dst) == 0) {

1 0

git:master:68de38d5: dispatcher: switch to ksr_xrand() to be able to use custom rand function
by Daniel-Constantin Mierla 07 Feb '25

07 Feb '25

Module: kamailio Branch: master Commit: 68de38d51fb3773fbc5dbce0068bec1597783386 URL: https://github.com/kamailio/kamailio/commit/68de38d51fb3773fbc5dbce0068bec1… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-02-07T07:52:19+01:00 dispatcher: switch to ksr_xrand() to be able to use custom rand function --- Modified: src/modules/dispatcher/dispatch.c Modified: src/modules/dispatcher/dispatch.h --- Diff: https://github.com/kamailio/kamailio/commit/68de38d51fb3773fbc5dbce0068bec1… Patch: https://github.com/kamailio/kamailio/commit/68de38d51fb3773fbc5dbce0068bec1… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index ca1636edbd2..9968eab5537 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -57,6 +57,7 @@ #include "../../core/script_cb.h" #include "../../core/kemi.h" #include "../../core/fmsg.h" +#include "../../core/rand/ksrxrand.h" #include "ds_ht.h" #include "api.h" @@ -790,7 +791,7 @@ void shuffle_uint100array(unsigned int *arr) if(arr == NULL) return; for(j = 0; j < 100; j++) { - k = j + (kam_rand() % (100 - j)); + k = j + (ksr_xrand() % (100 - j)); t = arr[j]; arr[j] = arr[k]; arr[k] = t; @@ -807,7 +808,7 @@ void shuffle_char100array(char *arr) if(arr == NULL) return; for(j = 0; j < 100; j++) { - k = j + (kam_rand() % (100 - j)); + k = j + (ksr_xrand() % (100 - j)); t = arr[j]; arr[j] = arr[k]; arr[k] = t; @@ -2614,7 +2615,7 @@ int ds_manage_routes(sip_msg_t *msg, ds_select_state_t *rstate) } break; case DS_ALG_RANDOM: /* 6 - random selection */ - hash = kam_rand(); + hash = ksr_xrand(); break; case DS_ALG_HASHPV: /* 7 - hash on PV value */ if(ds_hash_pvar(msg, &hash) != 0) { diff --git a/src/modules/dispatcher/dispatch.h b/src/modules/dispatcher/dispatch.h index b0d37bce288..c72b53bc7e2 100644 --- a/src/modules/dispatcher/dispatch.h +++ b/src/modules/dispatcher/dispatch.h @@ -35,7 +35,6 @@ #include "../../core/pvar.h" #include "../../core/xavp.h" #include "../../core/parser/msg_parser.h" -#include "../../core/rand/kam_rand.h" #include "../../modules/tm/tm_load.h"

1 0

git:master:c33f1f49: core: ksrxrand - framework to set custom rand/srand functions
by Daniel-Constantin Mierla 07 Feb '25

07 Feb '25

Module: kamailio Branch: master Commit: c33f1f494f1e7ff9deb887f58644fb21934f9b0e URL: https://github.com/kamailio/kamailio/commit/c33f1f494f1e7ff9deb887f58644fb2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-02-07T07:44:45+01:00 core: ksrxrand - framework to set custom rand/srand functions --- Added: src/core/rand/ksrxrand.c Added: src/core/rand/ksrxrand.h --- Diff: https://github.com/kamailio/kamailio/commit/c33f1f494f1e7ff9deb887f58644fb2… Patch: https://github.com/kamailio/kamailio/commit/c33f1f494f1e7ff9deb887f58644fb2… --- diff --git a/src/core/rand/ksrxrand.c b/src/core/rand/ksrxrand.c new file mode 100644 index 00000000000..e0938cf47d5 --- /dev/null +++ b/src/core/rand/ksrxrand.c @@ -0,0 +1,79 @@ +/* + * Copyright (C) 2025 Daniel-Constantin Mierla (asipto.com) + * + * This file is part of Kamailio, a free SIP server. + * + * Permission to use, copy, modify, and distribute this software for any + * purpose with or without fee is hereby granted, provided that the above + * copyright notice and this permission notice appear in all copies. + * + * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES + * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF + * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR + * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES + * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN + * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF + * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. + */ + +#include <string.h> + +#include "../dprint.h" + +#include "fastrand.h" +#include "ksrxrand.h" + +/** + * wrapper function for rand()/random() + */ +int ksr_wrand(void) +{ +#if RAND_MAX < INT_MAX + return (int)random(); +#else + return rand(); +#endif +} + +/** + * wrapper function for srand()/srandom() + */ +void ksr_wsrand(unsigned int seed) +{ +#if RAND_MAX < INT_MAX + return srandom(seed); +#else + return srand(seed); +#endif +} + +/** + * wrapper function for fastrand() + */ +int ksr_wfastrand(void) +{ + return (int)(fastrand() % ((unsigned)KSR_XRAND_MAX + 1)); +} + +/** + * global with internal RAND API + */ +ksr_xrand_t _ksr_xrand_api = {.xrand = ksr_wrand, .xsrand = ksr_wsrand}; + +/** + * + */ +int ksr_xrand_set(char *name) +{ + if(strcmp(name, "fast") == 0) { + _ksr_xrand_api.xrand = ksr_wfastrand; + _ksr_xrand_api.xsrand = fastrand_seed; + return 0; + } + if((strcmp(name, "rand") == 0) || (strcmp(name, "rand") == 0)) { + /* default - nothing to do */ + return 0; + } + LM_WARN("unknown rand engine: %s\n", name); + return -1; +} diff --git a/src/core/rand/ksrxrand.h b/src/core/rand/ksrxrand.h new file mode 100644 index 00000000000..51f026e9bc6 --- /dev/null +++ b/src/core/rand/ksrxrand.h @@ -0,0 +1,47 @@ +/* + * Copyright (C) 2025 Daniel-Constantin Mierla (asipto.com) + * + * This file is part of Kamailio, a free SIP server. + * + * Permission to use, copy, modify, and distribute this software for any + * purpose with or without fee is hereby granted, provided that the above + * copyright notice and this permission notice appear in all copies. + * + * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES + * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF + * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR + * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES + * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN + * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF + * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. + */ + +#ifndef __KSRXRAND_H__ +#define __KSRXRAND_H__ + +#include <limits.h> +#include <stdlib.h> + +#if RAND_MAX < INT_MAX +#define KSR_XRAND_MAX ((int)(0x7FFFFFFF)) /* (1<<31) - 1 */ +#else +#define KSR_XRAND_MAX RAND_MAX +#endif + +typedef int (*ksr_xrand_f)(void); +typedef void (*ksr_xsrand_f)(unsigned int seed); + +typedef struct ksr_xrand +{ + ksr_xrand_f xrand; + ksr_xsrand_f xsrand; +} ksr_xrand_t; + +extern ksr_xrand_t _ksr_xrand_api; + +#define ksr_xrand() _ksr_xrand_api.xrand() +#define ksr_xsrand(s) _ksr_xrand_api.xsrand(s) + +int ksr_xrand_set(char *name); + +#endif

1 0

[kamailio/kamailio] Problems with ims_ipsec_pcscf and closed tcp connections (Issue #4138)
by Atanas Bakalov 07 Feb '25

07 Feb '25

### Description Let's assume we have a MO call from UE to P-CSCF that's over ipsec. Let's assume the picked ports are like UE(6101) -> P-CSCF(6100) We have also set tcp_connection_lifetime to be 20 sec. 20 sec after INVITE connection is RST from Kamailio as expected. Then ( > 20sec later) an INVITE response comes from B side, and must be delivered to the UE. Kamailio opens a new connection and tries to establish a connection from 5060 (standard port)->6101 instead of 6100(P-CSCF) -> 6101(UE). IPSEC associations are not engaged and UE does not handle the request properly. ### Troubleshooting #### Reproduction #### Debugging Data #### Log Messages #### SIP Traffic ### Possible Solutions The IPSEC module should re-open the tcp connection if it sees it's been dropped as core module has no awareness of IPSEC. * **Operating System**: All -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4138 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4138(a)github.com>

2 1

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev February 2025