sr-dev

sr-dev@lists.kamailio.org

21 participants
45185 discussions

[kamailio/kamailio] Missing documentation of deprecated ring_insert_callid in siputils (Issue #3212)
by Niklas Larsson 12 Aug '22

12 Aug '22

### Description Upgraded to 5.6 from 5.5, looked through docs - and no mention of removal of ring_insert_callid - so upgrade failed. Found https://github.com/kamailio/kamailio/issues/2989 ### Possible Solutions Add documentation that the function is removed, and how to replace it with textops/htable and/or reply routes -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3212 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3212(a)github.com>

3 3

[kamailio/kamailio] Fix ksr_version_control() for FreeBSD build (PR #3184)
by Boris Korzun 12 Aug '22

12 Aug '22

#### 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 There's a problem after upgrade to 5.6 on FreeBSD. Kamailio isn't starting successful. ``` 14:19:20.540514 kamailio 40921 - - DEBUG: <core> [core/cfg.y:1964]: yyparse(): loading module tls.so 14:19:20.540893 kamailio 40921 - - DEBUG: <core> [core/sr_module.c:513]: ksr_locate_module(): found module to load </usr/local/lib/kamailio/modules/tls.so> 14:19:20.540912 kamailio 40921 - - DEBUG: <core> [core/sr_module.c:563]: load_module(): trying to load </usr/local/lib/kamailio/modules/tls.so> 14:19:20.543301 kamailio 40921 - - ERROR: <core> [core/sr_module.c:352]: ksr_version_control(): no version info in module </usr/local/lib/kamailio/modules/tls.so>: Undefined symbol "_nss_cache_cycle_prevention_function" ``` The problem is in `ksr_version_control()` probably. In FreeBSD environment it should to flush `dlerror` before call `dlsym()` - [dlopen(3)](https://www.freebsd.org/cgi/man.cgi?query=dlopen&sektion=3&a…at=html). You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3184 -- Commit Summary -- * core: fix ksr_version_control() for FreeBSD build -- File Changes -- M src/core/sr_module.c (3) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3184.patch https://github.com/kamailio/kamailio/pull/3184.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3184 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3184(a)github.com>

2 2

git:master:a8769af4: core: fix ksr_version_control() for FreeBSD build
by Daniel-Constantin Mierla 12 Aug '22

12 Aug '22

Module: kamailio Branch: master Commit: a8769af4c3a497d0f6a81cff17861e566ffa87bc URL: https://github.com/kamailio/kamailio/commit/a8769af4c3a497d0f6a81cff17861e5… Author: drTr0jan <drtr0jan(a)yandex.ru> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-08-12T13:52:38+02:00 core: fix ksr_version_control() for FreeBSD build --- Modified: src/core/sr_module.c --- Diff: https://github.com/kamailio/kamailio/commit/a8769af4c3a497d0f6a81cff17861e5… Patch: https://github.com/kamailio/kamailio/commit/a8769af4c3a497d0f6a81cff17861e5… --- diff --git a/src/core/sr_module.c b/src/core/sr_module.c index 2d787d7f9f3..013598c2a76 100644 --- a/src/core/sr_module.c +++ b/src/core/sr_module.c @@ -347,6 +347,9 @@ int ksr_version_control(void *handle, char *path) char **m_flags; char* error; +#ifdef __FreeBSD__ + (void) dlerror(); +#endif m_ver=(char **)dlsym(handle, "module_version"); if ((error=(char *)dlerror())!=0) { LM_ERR("no version info in module <%s>: %s\n", path, error);

1 0

Large memory allocation on module startup
by kthulasizwe＠gmail.com 12 Aug '22

12 Aug '22

Hi Guys, I have recently modified a custom module, where I now shm_mallocxz 2MB, as part of module start up and this memory is not being used as yet. Previously, the would be smaller chunks(100 bytes at most) shm_malloced during message processing. I have observed significant runtime performance degradation from just this single change. I would appreciate any suggestions/thoughts you might have. Below are some the relevant details. OS:Alpine Linux RAM: 16GB Configured Shared memory: 4GB shm_force_alloc=1 Typical shared memory usage: 1.5GB System swapping is disabled, also observed during load testing that no mem is getting swapped.

2 6

[kamailio/kamailio] uac_redirect: segfault when calling get_redirects("1") [regression] (Issue #3197)
by Alex Hermann 12 Aug '22

12 Aug '22

Since updating to 5.6, Kamailio segfaults when calling `get_redirects("1")`. ``` #0 0x000055d12c93c527 in get_str_fparam (dst=dst@entry=0x7ffd246edfe0, msg=msg@entry=0x7f83973b6a80, param=0x0) at core/sr_module.c:1646 val = {n = 1, s = {s = 0x1 <error: Cannot access memory at address 0x1>, len = 749136082}, re = 0x1} ret = <optimized out> avp = <optimized out> pv_val = {rs = {s = 0x1 <error: Cannot access memory at address 0x1>, len = 749136082}, ri = 1, flags = 0} __func__ = "get_str_fparam" #1 0x00007f838d2f0a17 in w_get_redirect2 (msg=0x7f83973b6a80, max_c=0x0, reason=<optimized out>) at uac_redirect.c:337 n = <optimized out> max = <optimized out> sreason = {s = 0x1a <error: Cannot access memory at address 0x1a>, len = -828131391} __func__ = "w_get_redirect2" #2 0x000055d12c975624 in do_action (h=h@entry=0x7ffd246ee820, a=a@entry=0x7f83cedd3670, msg=msg@entry=0x7f83973b6a80) at core/action.c:1085 ret = <optimized out> v = <optimized out> ... ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3197 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3197(a)github.com>

4 5

git:master:3b1354e2: uac_redirect: check if reason parameter ifs provided for get_redirects()
by Daniel-Constantin Mierla 12 Aug '22

12 Aug '22

Module: kamailio Branch: master Commit: 3b1354e2fd26c06fa6bc5ddbee1a012507aa2ea5 URL: https://github.com/kamailio/kamailio/commit/3b1354e2fd26c06fa6bc5ddbee1a012… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-08-12T09:16:24+02:00 uac_redirect: check if reason parameter ifs provided for get_redirects() - cope properly with the common function used for the two function variants - GH #3197 --- Modified: src/modules/uac_redirect/uac_redirect.c --- Diff: https://github.com/kamailio/kamailio/commit/3b1354e2fd26c06fa6bc5ddbee1a012… Patch: https://github.com/kamailio/kamailio/commit/3b1354e2fd26c06fa6bc5ddbee1a012… --- diff --git a/src/modules/uac_redirect/uac_redirect.c b/src/modules/uac_redirect/uac_redirect.c index 76b7592b5a..9925621d0e 100644 --- a/src/modules/uac_redirect/uac_redirect.c +++ b/src/modules/uac_redirect/uac_redirect.c @@ -344,17 +344,19 @@ static int w_get_redirect2(struct sip_msg* msg, char *max_c, char *reason) { int n; unsigned short max; - str sreason; + str sreason = {0}; - if(fixup_get_svalue(msg, (gparam_t*)reason, &sreason)<0) { - LM_ERR("failed to get reason parameter\n"); - return -1; + if(reason!=NULL) { + if(fixup_get_svalue(msg, (gparam_t*)reason, &sreason)<0) { + LM_ERR("failed to get reason parameter\n"); + return -1; + } } msg_tracer( msg, 0); /* get the contacts */ max = (unsigned short)(long)max_c; - n = get_redirect(msg , (max>>8)&0xff, max&0xff, &sreason, bflags); + n = get_redirect(msg , (max>>8)&0xff, max&0xff, (reason)?&sreason:NULL, bflags); reset_filters(); /* reset the tracer */ msg_tracer( msg, 1);

1 0

git:master:939ceb33: usrloc: do not send keepalives when destroying modules
by Daniel-Constantin Mierla 11 Aug '22

11 Aug '22

Module: kamailio Branch: master Commit: 939ceb3390b171b773eeb438b8fff72022e4f6f0 URL: https://github.com/kamailio/kamailio/commit/939ceb3390b171b773eeb438b8fff72… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-08-11T21:49:22+02:00 usrloc: do not send keepalives when destroying modules - GH #3207 --- Modified: src/modules/usrloc/ul_keepalive.c --- Diff: https://github.com/kamailio/kamailio/commit/939ceb3390b171b773eeb438b8fff72… Patch: https://github.com/kamailio/kamailio/commit/939ceb3390b171b773eeb438b8fff72… --- diff --git a/src/modules/usrloc/ul_keepalive.c b/src/modules/usrloc/ul_keepalive.c index 5b1572483b..8808725849 100644 --- a/src/modules/usrloc/ul_keepalive.c +++ b/src/modules/usrloc/ul_keepalive.c @@ -33,6 +33,7 @@ #include "../../core/forward.h" #include "../../core/globals.h" #include "../../core/pvar.h" +#include "../../core/sr_module.h" #include "../../core/parser/parse_uri.h" #include "../../core/parser/parse_from.h" #include "../../core/parser/parse_to.h" @@ -110,6 +111,11 @@ int ul_ka_urecord(urecord_t *ur) if (ul_ka_mode == ULKA_NONE) { return 0; } + + if(likely(destroy_modules_phase()!=0)) { + return 0; + } + LM_DBG("keepalive for aor: %.*s\n", ur->aor.len, ur->aor.s); tnow = time(NULL);

1 0

git:master:40d0c0ef: modules: readme files regenerated - uac_redirect ... [skip ci]
by Kamailio Dev 10 Aug '22

10 Aug '22

Module: kamailio Branch: master Commit: 40d0c0ef19439ba03822c1a875ea0461f3f1106b URL: https://github.com/kamailio/kamailio/commit/40d0c0ef19439ba03822c1a875ea046… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2022-08-10T22:46:19+02:00 modules: readme files regenerated - uac_redirect ... [skip ci] --- Modified: src/modules/uac_redirect/README --- Diff: https://github.com/kamailio/kamailio/commit/40d0c0ef19439ba03822c1a875ea046… Patch: https://github.com/kamailio/kamailio/commit/40d0c0ef19439ba03822c1a875ea046… --- diff --git a/src/modules/uac_redirect/README b/src/modules/uac_redirect/README index 812755bc5d..c016467df9 100644 --- a/src/modules/uac_redirect/README +++ b/src/modules/uac_redirect/README @@ -205,10 +205,12 @@ modparam("uac_redirect","accept_filter",".*(a)siphub\.net") 4.4. acc_function (string) Specifies the accounting function to be used. Just by defining this - parameter, the accounting support will not be enabled. Accounting may - only be enabled via two parameters set_accept_filter() scripting + parameter, the accounting is not done automatically, it has to be + requested via parameters of set_accept_filter() or other scripting function. + Set it to empty string in order to disable binding to acc module. + Its values may be: * acc_log_request * acc_db_request @@ -224,10 +226,7 @@ modparam("uac_redirect","acc_function","acc_db_request") 4.5. acc_db_table (string) Specifies the accounting table to be used if DB accounting was chosen - (acc_function was set to “acc_db_request”). Just by defining this - parameter, the accounting support will not be enabled. Accounting may - only be enabled via two parameters set_accept_filter() scripting - function. + (acc_function was set to “acc_db_request” or “acc_request”). The default value is “acc”.

1 0

git:master:7ef109e6: uac_redirect: clarifications for acc_function mod param
by Daniel-Constantin Mierla 10 Aug '22

10 Aug '22

Module: kamailio Branch: master Commit: 7ef109e6cd1ce7ecbfa56c31ff7555275e59095f URL: https://github.com/kamailio/kamailio/commit/7ef109e6cd1ce7ecbfa56c31ff75552… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-08-10T22:39:03+02:00 uac_redirect: clarifications for acc_function mod param --- Modified: src/modules/uac_redirect/doc/uac_redirect_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/7ef109e6cd1ce7ecbfa56c31ff75552… Patch: https://github.com/kamailio/kamailio/commit/7ef109e6cd1ce7ecbfa56c31ff75552… --- diff --git a/src/modules/uac_redirect/doc/uac_redirect_admin.xml b/src/modules/uac_redirect/doc/uac_redirect_admin.xml index 1388fc740a4..71a448a6ae7 100644 --- a/src/modules/uac_redirect/doc/uac_redirect_admin.xml +++ b/src/modules/uac_redirect/doc/uac_redirect_admin.xml @@ -211,11 +211,14 @@ modparam("uac_redirect","accept_filter",".*(a)siphub\.net") <section id="uac_redirect.p.acc_function"> <title><varname>acc_function</varname> (string)</title> <para> - Specifies the accounting function to be used. Just by defining - this parameter, the accounting support will not be enabled. - Accounting may only be enabled via two parameters - <function moreinfo="none">set_accept_filter()</function> - scripting function. + Specifies the accounting function to be used. Just by defining + this parameter, the accounting is not done automatically, it has to + be requested via parameters of + <function moreinfo="none">set_accept_filter()</function> + or other scripting function. + </para> + <para> + Set it to empty string in order to disable binding to acc module. </para> <para> Its values may be: @@ -248,13 +251,9 @@ modparam("uac_redirect","acc_function","acc_db_request") <section id="uac_redirect.p.acc_db_table"> <title><varname>acc_db_table</varname> (string)</title> <para> - Specifies the accounting table to be used if DB accounting was - chosen (<varname>acc_function</varname> was set to - <quote>acc_db_request</quote>). Just by defining - this parameter, the accounting support will not be enabled. - Accounting may only be enabled via two parameters - <function moreinfo="none">set_accept_filter()</function> - scripting function. + Specifies the accounting table to be used if DB accounting was + chosen (<varname>acc_function</varname> was set to + <quote>acc_db_request</quote> or <quote>acc_request</quote>). </para> <para> <emphasis>

1 0

[kamailio/kamailio] Fix isup generic_number and generic_number_nai decoding (PR #3209)
by russagit 10 Aug '22

10 Aug '22

#### 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:  - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description ISUP parameter Generic Number has a different structure (offset+2 is a Number Quilifirer Indicator) than other numbers, so this fix addresses it's decoding bug You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3209 -- Commit Summary -- * Fix isup generic_number and generic_number_nai decoding -- File Changes -- M src/modules/sipt/ss7_parser.c (8) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3209.patch https://github.com/kamailio/kamailio/pull/3209.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3209 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3209(a)github.com>

2 2

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev