sr-dev

sr-dev@lists.kamailio.org

20 participants
45174 discussions

git:master:d9626dc9: pv_headers: rework pvh_remove_header_param
by Victor Seva 28 Jun '22

28 Jun '22

Module: kamailio Branch: master Commit: d9626dc9f341718ad7d78e62042695124ba52c32 URL: https://github.com/kamailio/kamailio/commit/d9626dc9f341718ad7d78e620426951… Author: Victor Seva <vseva(a)sipwise.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2022-06-28T12:56:35+02:00 pv_headers: rework pvh_remove_header_param * fix KEMI interface, this is suppose to be called like pvh_remove_header_param(header_name, string_to_remove) * use pv buffer for temporal value * use xavi interface to set the new value --- Modified: src/modules/pv_headers/pv_headers.c Modified: src/modules/pv_headers/pvh_func.c Modified: src/modules/pv_headers/pvh_func.h --- Diff: https://github.com/kamailio/kamailio/commit/d9626dc9f341718ad7d78e620426951… Patch: https://github.com/kamailio/kamailio/commit/d9626dc9f341718ad7d78e620426951…

1 0

git:master:03a489ad: pkg/kamailio/deb: update lintian overrides [skip ci]
by Victor Seva 28 Jun '22

28 Jun '22

Module: kamailio Branch: master Commit: 03a489addb248bd7e4820dc281ade4f4f3deda3c URL: https://github.com/kamailio/kamailio/commit/03a489addb248bd7e4820dc281ade4f… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2022-06-28T09:37:59+02:00 pkg/kamailio/deb: update lintian overrides [skip ci] --- Modified: pkg/kamailio/deb/bionic/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/bionic/kamailio.lintian-overrides Modified: pkg/kamailio/deb/bookworm/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/bookworm/kamailio.lintian-overrides Modified: pkg/kamailio/deb/bullseye/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/bullseye/kamailio.lintian-overrides Modified: pkg/kamailio/deb/buster/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/buster/kamailio.lintian-overrides Modified: pkg/kamailio/deb/debian/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/debian/kamailio.lintian-overrides Modified: pkg/kamailio/deb/focal/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/focal/kamailio.lintian-overrides Modified: pkg/kamailio/deb/jessie/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/jessie/kamailio.lintian-overrides Modified: pkg/kamailio/deb/precise/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/precise/kamailio.lintian-overrides Modified: pkg/kamailio/deb/sid/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/sid/kamailio.lintian-overrides Modified: pkg/kamailio/deb/stretch/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/stretch/kamailio.lintian-overrides Modified: pkg/kamailio/deb/trusty/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/trusty/kamailio.lintian-overrides Modified: pkg/kamailio/deb/wheezy/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/wheezy/kamailio.lintian-overrides Modified: pkg/kamailio/deb/xenial/kamailio-tls-modules.lintian-overrides Modified: pkg/kamailio/deb/xenial/kamailio.lintian-overrides --- Diff: https://github.com/kamailio/kamailio/commit/03a489addb248bd7e4820dc281ade4f… Patch: https://github.com/kamailio/kamailio/commit/03a489addb248bd7e4820dc281ade4f…

1 0

[kamailio/kamailio] second auth via uac_reg_request_to (Issue #3166)
by Alberto bebbo Capponi 27 Jun '22

27 Jun '22

When using uac_reg_request_to and the first trunk fails (as invalid credentials) try a second trunk, but there is something not cleaned or just doesn't get the new trunk auth params. code example ``` failure_route[RETRY_OUTBOUND] { if (t_check_status("401|407")) { if(uac_reg_request_to("$rd", 0)) { t_on_failure("RETRY_OUTBOUND"); xinfo("[RETRY_OUTBOUND] <$ci> | Doing authentication with $dd"); uac_auth(); t_relay(); exit; } } if (next_routing()) { $avp(out_trunk) = $rd; # t_relay(); t_on_failure("RETRY_OUTBOUND"); t_relay(); exit; } } ``` running this code to those targets: - TRUNK 10.3.3.11 (invalid user/pass) - TRUNK 10.3.3.12 (correct user/pass) produce this output in logs: ``` INFO: [RETRY_OUTBOUND] <xxxx1xxx@10.3.3.11:5060> | Doing authentication with 10.3.3.10 INFO: uac [auth.c:450]: uac_auth_mode(): no credential for realm "10.3.3.11" INFO: [RETRY_OUTBOUND] <xxxx1yyyy@10.3.3.12:5060> | Doing authentication with 10.3.3.10 ``` but if I simply remove the first trunk those credential works on the second trunk. is like some stuff (like the $dd) is not cleaned up / refreshed? version: ``` version: kamailio 5.5.4 (x86_64/linux) flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. id: unknown compiled with gcc 8.3.0 ``` running with drouting module to get next trunk -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3166 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3166(a)github.com>

2 2

git:master:c170cc81: tls_wolfssl: make - use kamailio make infra vars
by S-P Chan 27 Jun '22

27 Jun '22

Module: kamailio Branch: master Commit: c170cc811b9a03383c07578f4b4c88e15b96b637 URL: https://github.com/kamailio/kamailio/commit/c170cc811b9a03383c07578f4b4c88e… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2022-06-27T21:50:20+08:00 tls_wolfssl: make - use kamailio make infra vars --- Modified: src/modules/tls_wolfssl/Makefile --- Diff: https://github.com/kamailio/kamailio/commit/c170cc811b9a03383c07578f4b4c88e… Patch: https://github.com/kamailio/kamailio/commit/c170cc811b9a03383c07578f4b4c88e… --- diff --git a/src/modules/tls_wolfssl/Makefile b/src/modules/tls_wolfssl/Makefile index 6dd9e1b770..687fe1a494 100644 --- a/src/modules/tls_wolfssl/Makefile +++ b/src/modules/tls_wolfssl/Makefile @@ -9,17 +9,13 @@ auto_gen= NAME=tls_wolfssl.so LIBS += lib/lib/libwolfssl.a -DEFS += -I./lib/include/ +INCLUDES += -I./lib/include/ include ../../Makefile.modules -SOURCES := $(shell ls *.c) -OBJECTS = $(SOURCES:.c=.o) -MOD_DIR := $(shell pwd) +$(objs): lib/include/wolfssl/options.h -$(OBJECTS): lib/lib/libwolfssl.a - -lib/lib/libwolfssl.a: +lib/include/wolfssl/options.h lib/lib/libwolfssl.a: @cd ./lib/wolfssl; \ if [ ! -f "configure" ]; then \ ./autogen.sh; \ @@ -28,9 +24,8 @@ lib/lib/libwolfssl.a: env -u DEFS -u CFLAGS -u LDFLAGS -u LIBS EXTRA_CFLAGS="-g -fPIC -Wno-error=array-bounds -Wno-error=stringop-overflow" ./configure \ --enable-all --enable-pkcs11 --enable-static --enable-aligndata=no \ --disable-shared --disable-examples \ - --prefix=$(MOD_DIR)/lib \ - --exec-prefix=$(MOD_DIR)/lib; \ - sed -i s/-Werror// Makefile; \ + --prefix=$(CURDIR)/lib \ + --exec-prefix=$(CURDIR)/lib; \ fi; @$(MAKE) -C ./lib/wolfssl install @@ -38,7 +33,7 @@ $(NAME): lib/lib/libwolfssl.a clean-wolfssl: @rm -rf ./lib/{bin,include,share,lib}; \ - cd ./lib/wolfssl; make clean || /bin/true + (cd ./lib/wolfssl; make distclean) || /bin/true clean-module: @rm -f *.o *.so @@ -46,3 +41,6 @@ clean-module: clean-all: clean-wolfssl clean-module .PHONY: clean-wolfssl clean-module clean-all + +# debugging for dependency on lib/wolfssl +$(warning sources = $(sources), auto_gen = $(auto_gen), objs = $(objs))

1 0

git:master:fa6a2d79: ipops: added fixup free functions and use of ANY_ROUTE
by Daniel-Constantin Mierla 27 Jun '22

27 Jun '22

Module: kamailio Branch: master Commit: fa6a2d79aca3bc4733276ce99a53efd8fe63ffd5 URL: https://github.com/kamailio/kamailio/commit/fa6a2d79aca3bc4733276ce99a53efd… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-06-25T14:48:35+02:00 ipops: added fixup free functions and use of ANY_ROUTE --- Modified: src/modules/ipops/ipops_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/fa6a2d79aca3bc4733276ce99a53efd… Patch: https://github.com/kamailio/kamailio/commit/fa6a2d79aca3bc4733276ce99a53efd…

1 0

[kamailio/kamailio] detailed_ipv4_type func pkg memory leak. (Issue #3159)
by 杰哥 25 Jun '22

25 Jun '22

### Description when call detailed_ipv4_type function, pkg memory leak. ```c detailed_ipv4_type($si,"$var(iptype)"); ``` ### Troubleshooting #### Reproduction just call detailed_ipv4_type function. #### Debugging Data ``` ALERT: qm_sums: qm_sums(): count= 48161 size= 1789640 bytes from core: core/rvalue.c: rval_get_str(1293) ``` ### Possible Solutions ip address string not free in rval_get_str() function. [s->s=pkg_malloc(tmp.len+1/* 0 term */);](https://github.com/kamailio/kamailio/blob/1ddc27f199061025a6a43da3e8a1388fcaddfe19/src/core/rvalue.c#L1293) need call pkg_free() free memory. ### Additional Information test 3 version. ``` kamailio 5.5.4 kamailio 5.4.8 kamailio 5.3.9 ``` * **Operating System**: ``` buster 10 ubuntu 20.04 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3159 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3159(a)github.com>

2 7

[kamailio/kamailio] topos breaks early-dialog update even on 5.6.0~rc0 (Issue #3112)
by George Diamantopoulos 24 Jun '22

24 Jun '22

### Description Topos still seems to break early-dialog UPDATEs in my configuration. This is on 5.6.0~rc0+bpo11.20220506004754.1 from 56-nightly which I believe includes c309122 and ceb688b. I haven't tested rc1 yet (there are no debs at this time) but there doesn't seem to be any topos-related commits since rc0 either. ### Troubleshooting #### SIP Traffic Attaching pcaps from a lab environment to demonstrate the problem. One with topos on (failure), one with off (success). There were no other configuration changes between these tests. ### Possible Solutions Only turning topos off seems to resolve this for me. ### Additional Information ``` version: kamailio 5.6.0-rc0 (x86_64/linux) flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. id: unknown compiled with gcc 10.2.1 ``` * **Operating System**: ``` Linux sbcprov0-stage-tis0-cn1 5.10.0-10-amd64 #1 SMP Debian 5.10.84-1 (2021-12-08) x86_64 GNU/Linux ``` [pcaps.tar.gz](https://github.com/kamailio/kamailio/files/8704222/pcaps.tar.… -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3112 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3112(a)github.com>

3 6

git:master:87b698f7: modules: readme files regenerated - textops ... [skip ci]
by Kamailio Dev 24 Jun '22

24 Jun '22

Module: kamailio Branch: master Commit: 87b698f7deca1794019f98167ba49d3e4737ba71 URL: https://github.com/kamailio/kamailio/commit/87b698f7deca1794019f98167ba49d3… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2022-06-24T16:01:17+02:00 modules: readme files regenerated - textops ... [skip ci] --- Modified: src/modules/textops/README Modified: src/modules/tls/README --- Diff: https://github.com/kamailio/kamailio/commit/87b698f7deca1794019f98167ba49d3… Patch: https://github.com/kamailio/kamailio/commit/87b698f7deca1794019f98167ba49d3… --- diff --git a/src/modules/textops/README b/src/modules/textops/README index fabc8e3371..c187c4c8db 100644 --- a/src/modules/textops/README +++ b/src/modules/textops/README @@ -157,8 +157,8 @@ Ovidiu Sas 1.43. has_body usage 1.44. is_audio_on_hold usage 1.45. is_privacy usage - 1.46. in_list() usage - 1.47. in_list() usage + 1.46. in_list usage + 1.47. in_list_prefix usage 1.48. cmp_str usage 1.49. cmp_str usage 1.50. starts_with usage @@ -1239,7 +1239,7 @@ if(is_privacy("id")) Function can be used from all kinds of routes. - Example 1.46. in_list() usage + Example 1.46. in_list usage ... $var(subject) = "fi"; $var(list) = "dk,fi,no,se"; @@ -1257,7 +1257,7 @@ if (in_list("$var(subject)", "$var(list)", ",")) { Function can be used from all kinds of routes. - Example 1.47. in_list() usage + Example 1.47. in_list_prefix usage ... $var(subject) = "final"; $var(list) = "dk,fi,no,se"; diff --git a/src/modules/tls/README b/src/modules/tls/README index 94c8cfc069..c9bed56ec5 100644 --- a/src/modules/tls/README +++ b/src/modules/tls/README @@ -250,7 +250,7 @@ Chapter 1. Admin Guide The most important parameters to set the path to the public certificate and private key files. You can either have them in different file or in the same file in PEM format. The parameters for them are certificate - and private_key. They can be given as modparam or or provided in the + and private_key. They can be given as modparam or provided in the profiles of tls.cfg file. When installing tls module of kamailio, a sample 'tls.cfg' file is

1 0

[kamailio/kamailio] Fix typos (PR #3162)
by Akash Gupta 24 Jun '22

24 Jun '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, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [ ] 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 - [ ] 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 - [ ] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3162 -- Commit Summary -- * textops: Fix typos * tls: Fix typo -- File Changes -- M src/modules/textops/doc/textops_admin.xml (4) M src/modules/tls/doc/tls.xml (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3162.patch https://github.com/kamailio/kamailio/pull/3162.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3162 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3162(a)github.com>

2 2

git:master:c5cd4002: textops: Fix typos
by Daniel-Constantin Mierla 24 Jun '22

24 Jun '22

Module: kamailio Branch: master Commit: c5cd400287e27fce6fcf6272a4efe9a9fa8aa419 URL: https://github.com/kamailio/kamailio/commit/c5cd400287e27fce6fcf6272a4efe9a… Author: Akash Gupta <akash(a)telnyx.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2022-06-24T15:49:50+02:00 textops: Fix typos --- Modified: src/modules/textops/doc/textops_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/c5cd400287e27fce6fcf6272a4efe9a… Patch: https://github.com/kamailio/kamailio/commit/c5cd400287e27fce6fcf6272a4efe9a… --- diff --git a/src/modules/textops/doc/textops_admin.xml b/src/modules/textops/doc/textops_admin.xml index 9805aac6aab..dbca57e5f0a 100644 --- a/src/modules/textops/doc/textops_admin.xml +++ b/src/modules/textops/doc/textops_admin.xml @@ -1711,7 +1711,7 @@ if(is_privacy("id")) Function can be used from all kinds of routes. </para> <example> - <title><function>in_list()</function> usage</title> + <title><function>in_list</function> usage</title> <programlisting format="linespecific"> ... $var(subject) = "fi"; @@ -1735,7 +1735,7 @@ if (in_list("$var(subject)", "$var(list)", ",")) { Function can be used from all kinds of routes. </para> <example> - <title><function>in_list()</function> usage</title> + <title><function>in_list_prefix</function> usage</title> <programlisting format="linespecific"> ... $var(subject) = "final";

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev