- sr-dev - kamailio.org

git:master:d26fb80c: modules: readme files regenerated - rtpengine ... [skip ci]
by Kamailio Dev 20 Jun '25

20 Jun '25

Module: kamailio Branch: master Commit: d26fb80c0e022287769c52aef91b269c2d59e787 URL: https://github.com/kamailio/kamailio/commit/d26fb80c0e022287769c52aef91b269… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-06-20T09:31:09+02:00 modules: readme files regenerated - rtpengine ... [skip ci] --- Modified: src/modules/rtpengine/README --- Diff: https://github.com/kamailio/kamailio/commit/d26fb80c0e022287769c52aef91b269… Patch: https://github.com/kamailio/kamailio/commit/d26fb80c0e022287769c52aef91b269…

1 0

git:master:0f688caa: rtpengine: list undocumented params
by Daniel-Constantin Mierla 20 Jun '25

20 Jun '25

Module: kamailio Branch: master Commit: 0f688caa76cbbaf7fc3ca4551a879fc77e961f9f URL: https://github.com/kamailio/kamailio/commit/0f688caa76cbbaf7fc3ca4551a879fc… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-20T09:29:12+02:00 rtpengine: list undocumented params --- Modified: src/modules/rtpengine/doc/rtpengine_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/0f688caa76cbbaf7fc3ca4551a879fc… Patch: https://github.com/kamailio/kamailio/commit/0f688caa76cbbaf7fc3ca4551a879fc… --- diff --git a/src/modules/rtpengine/doc/rtpengine_admin.xml b/src/modules/rtpengine/doc/rtpengine_admin.xml index 5db8b68fd19..500cd0fb59d 100644 --- a/src/modules/rtpengine/doc/rtpengine_admin.xml +++ b/src/modules/rtpengine/doc/rtpengine_admin.xml @@ -2449,6 +2449,48 @@ modparam("rtpengine", "enable_dmq", 1) </example> </section> +<section id="rtpengine.p.dtmf_event_source_label"> + <title><varname>dtmf_event_source_label</varname> (str)</title> + <para> + </para> +</section> + +<section id="rtpengine.p.dtmf_event_tags"> + <title><varname>dtmf_event_tags</varname> (str)</title> + <para> + </para> +</section> + +<section id="rtpengine.p.dtmf_event_type"> + <title><varname>dtmf_event_type</varname> (str)</title> + <para> + </para> +</section> + +<section id="rtpengine.p.dtmf_event_source_ip"> + <title><varname>dtmf_event_source_ip</varname> (str)</title> + <para> + </para> +</section> + +<section id="rtpengine.p.dtmf_event_duration"> + <title><varname>dtmf_event_duration</varname> (str)</title> + <para> + </para> +</section> + +<section id="rtpengine.p.dtmf_event_volume"> + <title><varname>dtmf_event_volume</varname> (str)</title> + <para> + </para> +</section> + +<section id="rtpengine.p.event_callback"> + <title><varname>event_callback</varname> (str)</title> + <para> + </para> +</section> + <section> <title>Functions</title> <section id="rtpengine.f.set_rtpengine_set">

1 0

git:master:2786c2fb: kex: formatted exported structures for stats and rpc
by Daniel-Constantin Mierla 20 Jun '25

20 Jun '25

Module: kamailio Branch: master Commit: 2786c2fb3def1e4bc7d7e1f6ec001e4eb59a9ea9 URL: https://github.com/kamailio/kamailio/commit/2786c2fb3def1e4bc7d7e1f6ec001e4… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-20T08:58:53+02:00 kex: formatted exported structures for stats and rpc --- Modified: src/modules/kex/core_stats.c --- Diff: https://github.com/kamailio/kamailio/commit/2786c2fb3def1e4bc7d7e1f6ec001e4… Patch: https://github.com/kamailio/kamailio/commit/2786c2fb3def1e4bc7d7e1f6ec001e4…

1 0

Re: [kamailio/kamailio] aaaa (2698244)
by Daniel-Constantin Mierla 20 Jun '25

20 Jun '25

@sergey-safarov: do not use such commit messages when pushing to the repository, write meaningful details in the commit message. When merging, squash but still be sure such "random characters" or messages like "WIP 1" (seen in another commit by you) are not in the final commit message. The commit messages go in the release files (ie,. ChangeLog) and makes it look really like a not good development process. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/commit/26982449b7dabe0413258a7602d8810… You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/commit/26982449b7dabe0413258a7602d88107c8f7e7d9/160433191(a)github.com>

1 0

git:master:b3a17751: modules: readme files regenerated - dispatcher ... [skip ci]
by Kamailio Dev 20 Jun '25

20 Jun '25

Module: kamailio Branch: master Commit: b3a177515af96fe4cc4c0408d93cfb651c6031bb URL: https://github.com/kamailio/kamailio/commit/b3a177515af96fe4cc4c0408d93cfb6… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-06-20T08:31:10+02:00 modules: readme files regenerated - dispatcher ... [skip ci] --- Modified: src/modules/dispatcher/README --- Diff: https://github.com/kamailio/kamailio/commit/b3a177515af96fe4cc4c0408d93cfb6… Patch: https://github.com/kamailio/kamailio/commit/b3a177515af96fe4cc4c0408d93cfb6…

1 0

git:master:8ab50bf8: dispatcher: docs for ds_mark_addr()
by Daniel-Constantin Mierla 20 Jun '25

20 Jun '25

Module: kamailio Branch: master Commit: 8ab50bf8a7894e68909c5a81505f5b09e4345802 URL: https://github.com/kamailio/kamailio/commit/8ab50bf8a7894e68909c5a81505f5b0… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-20T08:16:03+02:00 dispatcher: docs for ds_mark_addr() --- Modified: src/modules/dispatcher/doc/dispatcher_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/8ab50bf8a7894e68909c5a81505f5b0… Patch: https://github.com/kamailio/kamailio/commit/8ab50bf8a7894e68909c5a81505f5b0… --- diff --git a/src/modules/dispatcher/doc/dispatcher_admin.xml b/src/modules/dispatcher/doc/dispatcher_admin.xml index 5c36b3ff9f9..126ad986724 100644 --- a/src/modules/dispatcher/doc/dispatcher_admin.xml +++ b/src/modules/dispatcher/doc/dispatcher_admin.xml @@ -1786,6 +1786,27 @@ failure_route[tryagain] { ... } ... +</programlisting> + </example> + </section> + <section id="dispatcher.f.ds_mark_addr"> + <title> + <function moreinfo="none">ds_mark_addr(state, setid, uri)</function> + </title> + <para> + Set the state to the target specified by setid and uri. The parameter + state can have same value as for ds_mark_dst(). All parameters can + contain variables. + </para> + <para> + This function can be used from ANY_ROUTE. + </para> + <example> + <title><function>ds_mark_addr</function> usage</title> + <programlisting format="linespecific"> +... + ds_mark_addr("ip", "1", "sip:127.0.0.1:5080"); # set to inactive and probing +... </programlisting> </example> </section>

1 0

git:master:d85e63e7: dispatcher: added ds_mark_addr(state, group, uri)
by Daniel-Constantin Mierla 20 Jun '25

20 Jun '25

Module: kamailio Branch: master Commit: d85e63e7c5fb422151ba0d8a08b47cedc4929d8d URL: https://github.com/kamailio/kamailio/commit/d85e63e7c5fb422151ba0d8a08b47ce… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-20T08:11:46+02:00 dispatcher: added ds_mark_addr(state, group, uri) - set the state of a target by group and address --- Modified: src/modules/dispatcher/dispatch.c Modified: src/modules/dispatcher/dispatch.h Modified: src/modules/dispatcher/dispatcher.c --- Diff: https://github.com/kamailio/kamailio/commit/d85e63e7c5fb422151ba0d8a08b47ce… Patch: https://github.com/kamailio/kamailio/commit/d85e63e7c5fb422151ba0d8a08b47ce…

1 0

git:master:62b18cdf: core: mod fixup - helper function for s-i-s parameters
by Daniel-Constantin Mierla 20 Jun '25

20 Jun '25

Module: kamailio Branch: master Commit: 62b18cdfb62721304e5f82fd71063303cff06641 URL: https://github.com/kamailio/kamailio/commit/62b18cdfb62721304e5f82fd7106330… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-20T08:11:46+02:00 core: mod fixup - helper function for s-i-s parameters --- Modified: src/core/mod_fix.c Modified: src/core/mod_fix.h --- Diff: https://github.com/kamailio/kamailio/commit/62b18cdfb62721304e5f82fd7106330… Patch: https://github.com/kamailio/kamailio/commit/62b18cdfb62721304e5f82fd7106330… --- diff --git a/src/core/mod_fix.c b/src/core/mod_fix.c index 767c6a45a82..900c982e3c1 100644 --- a/src/core/mod_fix.c +++ b/src/core/mod_fix.c @@ -930,6 +930,40 @@ int fixup_free_ssi(void **param, int param_no) } } +/** + * + */ +int fixup_sis(void **param, int param_no) +{ + switch(param_no) { + case 1: + return fixup_spve_null(param, 1); + case 2: + return fixup_igp_null(param, 1); + case 3: + return fixup_spve_null(param, 1); + default: + return E_UNSPEC; + } +} + +/** + * + */ +int fixup_free_sis(void **param, int param_no) +{ + switch(param_no) { + case 1: + return fixup_free_spve_null(param, 1); + case 2: + return fixup_free_igp_null(param, 1); + case 3: + return fixup_free_spve_null(param, 1); + default: + return E_UNSPEC; + } +} + /** * */ diff --git a/src/core/mod_fix.h b/src/core/mod_fix.h index 6ec0280ac70..f0e6f0e47f4 100644 --- a/src/core/mod_fix.h +++ b/src/core/mod_fix.h @@ -178,6 +178,9 @@ int fixup_free_ssi(void **param, int param_no); int fixup_sii(void **param, int param_no); int fixup_free_sii(void **param, int param_no); +int fixup_sis(void **param, int param_no); +int fixup_free_sis(void **param, int param_no); + int fixup_sssi(void **param, int param_no); int fixup_free_sssi(void **param, int param_no);

1 0

[kamailio/kamailio] Kamailio blocks in SSL_accept (Issue #4185)
by Mathias Schneuwly 20 Jun '25

20 Jun '25

schneuwlym created an issue (kamailio/kamailio#4185) ### Description This issue is a follow up of the discussion in the [mailing list](https://lists.kamailio.org/mailman3/hyperkitty/list/sr-users@lists.ka… We are using Kamailio 5.8.4 in a yocto scarthgap based image. We noticed on multiple nodes, that suddenly phones lost their registrations and the log showed messages like `send_fd(): sendmsg failed sending 55 on 52: Too many references: cannot splice (109)`. After debugging we guess that the `send_fd` messages are just a symptom, not the cause. Generating a backtrace using `kamctl trap` shows that all TCP receiver are waiting for a `pthread_rwlock_...` lock. Our deployments, where we randomly see the issue, are very small; 10-30 registrations. I've tested kamailio version 5.8.4, 5.8.5, 5.8.6 and 6.0.1 and I was able to reproduce the issue with all versions. I've also tested version 5.5.4, but with an older yocto (dunfell) which used openssl 1.1.1w, **which did not block!** Yocto Scarthgap uses OpenSSL 3.2.3 and we also updated to version 3.4.1, which didn't show any difference. ### Troubleshooting #### Reproduction Meanwhile we found a way to reproduce the issue within seconds. It requires a bit of brute force, but it locks kamailio reliable. We are generating 3 parallel SIP REGISTER streams with sipexer to 3 different accounts. Following an example of one stream: ``` watch -n 0.1 "./sipexer -register -vl 3 -co -com -ex 3600 -fu "2" -au "2" -ap "test" -tk <path to key>.key -tc <path to certificate>.cer -ti -sd -ru sip:<domain> -cu sip:2@<domain> tls:<fqdn>:5061;sleep 0.1;./sipexer -register -vl 3 -co -com -ex 0 -fu "2" -au "2" -ap "test" -tk <path to key>.key -tc <path to cert>.cer -ti -sd -ru sip:<domain> -cu sip:2@<domain> tls:<fqdn>:5061" ``` Our dialplan is quite complex, but I was also able to reproduce the issue with the default [dialplan](https://raw.githubusercontent.com/kamailio/kamailio/refs/heads/5.… with the following changes: ``` 169a170,171 > #!define WITH_TLS > 205a208,219 > alias=udp:<domain>:5060 > alias=udp:127.0.0.1:5060 > alias=udp:10.0.6.1:5060 > alias=tls:<domain> > alias=tls:10.0.6.1:5061 > alias=tls:10.165.42.76:5061 > alias=tls:<domain>:5061 > alias=udp:<domain> > listen=udp:127.0.0.1:5060 > listen=tls:10.0.6.1:5061 > listen=udp:10.0.6.1:5060 > listen=tls:10.165.42.76:5061 498c512 < modparam("tls", "config", "/usr/local/etc/kamailio/tls.cfg") --- > modparam("tls", "config", "/etc/kamailio/kamailio-tls.cfg") ``` ``` # cat /etc/kamailio/kamailio-tls.cfg [server:default] method = TLSv1+ verify_certificate = yes require_certificate = yes private_key = "<path to key>.key" certificate = "<path to cert>.crt" ca_list = "<path to ca>.pem" [client:default] method = TLSv1+ verify_certificate = yes require_certificate = yes private_key = "<path to key>.key" certificate = "<path to cert>.crt" ca_list = "<path to ca>.pem" ``` #### Debugging Data Following the backtrace of one tcp listener process. The full backtrace can be found [here](https://github.com/user-attachments/files/19430313/gdb_kamailio_20250… ``` ---start 4608 ----------------------------------------------------- [Thread debugging using libthread_db enabled] Using host libthread_db library "/usr/lib/libthread_db.so.1". __futex_abstimed_wait_common (futex_word=futex_word@entry=0x7fc855758a0c, expected=expected@entry=3, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=<optimized out>, cancel=cancel@entry=false) at futex-internal.c:103 warning: 103 futex-internal.c: No such file or directory #0 __futex_abstimed_wait_common (futex_word=futex_word@entry=0x7fc855758a0c, expected=expected@entry=3, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=<optimized out>, cancel=cancel@entry=false) at futex-internal.c:103 err = -512 clockbit = 256 op = <optimized out> #1 0x00007fc85a59429c in __GI___futex_abstimed_wait64 (futex_word=futex_word@entry=0x7fc855758a0c, expected=expected@entry=3, clockid=clockid@entry=0, abstime=abstime@entry=0x0, private=<optimized out>) at futex-internal.c:128 No locals. #2 0x00007fc85a59d830 in __pthread_rwlock_wrlock_full64 (abstime=0x0, clockid=0, rwlock=0x7fc855758a00) at /usr/src/debug/glibc/2.39+git/nptl/pthread_rwlock_common.c:730 private = <optimized out> wf = <optimized out> err = <optimized out> prefer_writer = false may_share_futex_used_flag = true wpf = <optimized out> ready = <optimized out> r = <optimized out> may_share_futex_used_flag = <optimized out> r = <optimized out> done = <optimized out> wpf = <optimized out> ready = <optimized out> __value = <optimized out> prefer_writer = <optimized out> private = <optimized out> wf = <optimized out> err = <optimized out> w = <optimized out> w = <optimized out> private = <optimized out> err = <optimized out> w = <optimized out> wf = <optimized out> wf = <optimized out> __value = <optimized out> #3 ___pthread_rwlock_wrlock (rwlock=0x7fc855758a00) at pthread_rwlock_wrlock.c:26 result = <optimized out> #4 0x00007fc859820b49 in CRYPTO_THREAD_write_lock (lock=<optimized out>) at /usr/src/debug/openssl/3.2.3/crypto/threads_pthread.c:108 No locals. #5 0x00007fc8599bd5da in ossl_prov_drbg_generate (drbg=0x7fc855710040, out=0x7fc856471980 "", outlen=48, strength=256, prediction_resistance=0, adin=0x7ffeec8b67e0 "@", adinlen=8) at /usr/src/debug/openssl/3.2.3/providers/implementations/rands/drbg.c:645 fork_id = <optimized out> reseed_required = 0 ret = 0 __func__ = "ossl_prov_drbg_generate" err = <optimized out> now = <optimized out> #6 0x00007fc8599bd9ca in ossl_drbg_get_seed (vdrbg=<optimized out>, pout=0x7ffeec8b6880, entropy=<optimized out>, min_len=<optimized out>, max_len=<optimized out>, prediction_resistance=0, adin=0x7ffeec8b6838 "\220'\037V\310\177", adin_len=8) at /usr/src/debug/openssl/3.2.3/providers/implementations/rands/drbg.c:175 drbg = 0x7fc855710040 bytes_needed = 48 buffer = 0x7fc856471980 "" __func__ = "ossl_drbg_get_seed" #7 0x00007fc8599bc907 in get_entropy (drbg=<optimized out>, drbg@entry=0x7fc8561f2790, pout=pout@entry=0x7ffeec8b6880, entropy=<optimized out>, min_len=48, max_len=48, prediction_resistance=prediction_resistance@entry=0) at /usr/src/debug/openssl/3.2.3/providers/implementations/rands/drbg.c:245 bytes = <optimized out> p_str = 256 __func__ = "get_entropy" #8 0x00007fc8599bd26c in ossl_prov_drbg_reseed_unlocked (drbg=drbg@entry=0x7fc8561f2790, prediction_resistance=prediction_resistance@entry=0, ent=ent@entry=0x0, ent_len=ent_len@entry=0, adin=adin@entry=0x0, adinlen=adinlen@entry=0) at /usr/src/debug/openssl/3.2.3/providers/implementations/rands/drbg.c:573 entropy = 0x0 entropylen = 0 __func__ = "ossl_prov_drbg_reseed_unlocked" #9 0x00007fc8599bd68e in ossl_prov_drbg_generate (adinlen=0, adin=0x0, prediction_resistance=0, strength=<optimized out>, outlen=32, out=0x7fc8564717c8 "", drbg=0x7fc8561f2790) at /usr/src/debug/openssl/3.2.3/providers/implementations/rands/drbg.c:697 fork_id = <optimized out> reseed_required = <optimized out> ret = <optimized out> fork_id = <optimized out> reseed_required = <optimized out> ret = <optimized out> __func__ = "ossl_prov_drbg_generate" err = <optimized out> now = <optimized out> #10 ossl_prov_drbg_generate (drbg=0x7fc8561f2790, out=0x7fc8564717c8 "", outlen=32, strength=<optimized out>, prediction_resistance=0, adin=0x0, adinlen=0) at /usr/src/debug/openssl/3.2.3/providers/implementations/rands/drbg.c:634 fork_id = <optimized out> reseed_required = 0 ret = 0 __func__ = "ossl_prov_drbg_generate" err = <optimized out> now = <optimized out> #11 0x00007fc8597e75c6 in evp_rand_generate_locked (ctx=ctx@entry=0x7fc8561f26f0, out=out@entry=0x7fc8564717c8 "", outlen=outlen@entry=32, strength=strength@entry=0, prediction_resistance=<optimized out>, prediction_resistance@entry=0, addin=0x0, addin_len=0) at /usr/src/debug/openssl/3.2.3/crypto/evp/evp_rand.c:570 chunk = 32 max_request = 65536 params = {{key = 0x7fc8599e9521 "max_request", data_type = 2, data = 0x7ffeec8b6968, data_size = 8, return_size = 8}, {key = 0x0, data_type = 0, data = 0x0, data_size = 0, return_size = 0}} __func__ = "evp_rand_generate_locked" #12 0x00007fc8597e840b in EVP_RAND_generate (ctx=0x7fc8561f26f0, out=out@entry=0x7fc8564717c8 "", outlen=outlen@entry=32, strength=strength@entry=0, prediction_resistance=prediction_resistance@entry=0, addin=addin@entry=0x0, addin_len=0) at /usr/src/debug/openssl/3.2.3/crypto/evp/evp_rand.c:592 res = <optimized out> #13 0x00007fc8598f9282 in RAND_bytes_ex (ctx=warning: could not convert 'ossl_lib_ctx_st' from the host encoding (ANSI_X3.4-1968) to UTF-32. This normally should not happen, please file a bug report. 0x0, buf=0x7fc8564717c8 "", num=32, strength=0) at /usr/src/debug/openssl/3.2.3/crypto/rand/rand_lib.c:380 rand = <optimized out> meth = <optimized out> __func__ = "RAND_bytes_ex" #14 0x00007fc859bdb62c in def_generate_session_id (ssl=ssl@entry=0x7fc856436e50, id=id@entry=0x7fc8564717c8 "", id_len=id_len@entry=0x7ffeec8b6ae4) at /usr/src/debug/openssl/3.2.3/ssl/ssl_sess.c:326 retry = 0 #15 0x00007fc859bdc132 in ssl_generate_session_id (s=s@entry=0x7fc856436e50, ss=ss@entry=0x7fc856471570) at /usr/src/debug/openssl/3.2.3/ssl/ssl_sess.c:409 tmp = 32 cb = 0x7fc859bdb600 <def_generate_session_id> ssl = <optimized out> __func__ = "ssl_generate_session_id" #16 0x00007fc859bdc31e in ssl_get_new_session (s=s@entry=0x7fc856436e50, session=session@entry=1) at /usr/src/debug/openssl/3.2.3/ssl/ssl_sess.c:464 ss = 0x7fc856471570 __func__ = "ssl_get_new_session" #17 0x00007fc859c3f169 in tls_early_post_process_client_hello (s=0x7fc856436e50) at /usr/src/debug/openssl/3.2.3/ssl/statem/statem_srvr.c:1898 i = <optimized out> loop = <optimized out> sctx = 0x7fc855d24280 j = <optimized out> dgrd = DOWNGRADE_NONE protverr = <optimized out> c = <optimized out> clienthello = 0x7fc856470a90 al = 80 id = <optimized out> comp = 0x0 ciphers = 0x7fc856471330 scsvs = 0x7fc8564713c0 ssl = 0x7fc856436e50 j = <optimized out> i = <optimized out> al = <optimized out> protverr = <optimized out> loop = <optimized out> id = <optimized out> comp = <optimized out> c = <optimized out> ciphers = <optimized out> scsvs = <optimized out> clienthello = <optimized out> dgrd = <optimized out> sctx = <optimized out> ssl = <optimized out> __func__ = "tls_early_post_process_client_hello" err = <optimized out> cipher = <optimized out> trc_out = <optimized out> pos = <optimized out> pref_cipher = <optimized out> master_key_length = <optimized out> m = <optimized out> comp_id = <optimized out> k = <optimized out> m = <optimized out> nn = <optimized out> v = <optimized out> done = <optimized out> o = <optimized out> #18 tls_post_process_client_hello (s=0x7fc856436e50, wst=<optimized out>) at /usr/src/debug/openssl/3.2.3/ssl/statem/statem_srvr.c:2295 rv = <optimized out> cipher = <optimized out> ssl = 0x7fc856436e50 err = <optimized out> __func__ = "tls_post_process_client_hello" #19 0x00007fc859c2bc1b in read_state_machine (s=0x7fc856436e50) at /usr/src/debug/openssl/3.2.3/ssl/statem/statem.c:712 mt = 1 pkt = {curr = 0x7fc856466125 "", remaining = 0} max_message_size = 0x7fc859c3e3e0 <ossl_statem_server_max_message_size> ret = <optimized out> process_message = 0x7fc859c42400 <ossl_statem_server_process_message> post_process_message = 0x7fc859c414f0 <ossl_statem_server_post_process_message> st = 0x7fc856436ee0 len = 273 transition = 0x7fc859c3d330 <ossl_statem_server_read_transition> cb = 0x7fc859c91d90 <sr_ssl_ctx_info_callback> ssl = 0x7fc856436e50 st = <optimized out> ret = <optimized out> mt = <optimized out> len = <optimized out> transition = <optimized out> pkt = <optimized out> process_message = <optimized out> post_process_message = <optimized out> max_message_size = <optimized out> cb = <optimized out> ssl = <optimized out> __func__ = "read_state_machine" #20 state_machine (s=0x7fc856436e50, server=1) at /usr/src/debug/openssl/3.2.3/ssl/statem/statem.c:478 buf = 0x0 cb = 0x7fc859c91d90 <sr_ssl_ctx_info_callback> st = 0x7fc856436ee0 ret = -1 ssret = <optimized out> ssl = 0x7fc856436e50 __func__ = "state_machine" #21 0x00007fc859cbf036 in tls_accept (c=c@entry=0x7fc856461be0, error=error@entry=0x7ffeec8b6dd8) at tls_server.c:457 ret = <optimized out> ssl = 0x7fc856436e50 cert = <optimized out> tls_c = 0x7fc8564171b0 tls_log = <optimized out> __func__ = "tls_accept" ``` ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.8.4 (x86_64/linux) f1d83d flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, MEM_JOIN_FREE, 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 ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_SEND_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: f1d83d compiled on 10:25:58 Nov 12 2024 with x86_64-ruag-linux-gcc 13.3.0 ``` * **Operating System**: It is a yocto scarthgap based image (Version 5.0.6 LTS) ``` Linux ttel 6.6.54-yocto-standard #1 SMP PREEMPT_DYNAMIC Tue Oct 8 03:03:40 UTC 2024 x86_64 GNU/Linux ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4185 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4185(a)github.com>

6 11

git:master:1f9b9f13: modules: readme files regenerated - dispatcher ... [skip ci]
by Kamailio Dev 19 Jun '25

19 Jun '25

Module: kamailio Branch: master Commit: 1f9b9f1305da708461f392e979352464b9c4ee73 URL: https://github.com/kamailio/kamailio/commit/1f9b9f1305da708461f392e97935246… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-06-19T20:31:10+02:00 modules: readme files regenerated - dispatcher ... [skip ci] --- Modified: src/modules/dispatcher/README --- Diff: https://github.com/kamailio/kamailio/commit/1f9b9f1305da708461f392e97935246… Patch: https://github.com/kamailio/kamailio/commit/1f9b9f1305da708461f392e97935246… --- diff --git a/src/modules/dispatcher/README b/src/modules/dispatcher/README index 32726636469..eecd8bb5f95 100644 --- a/src/modules/dispatcher/README +++ b/src/modules/dispatcher/README @@ -1235,10 +1235,13 @@ modparam("dispatcher", "ds_interval_mode", 7200) congestion ms. See also the description of the 'rweight' attribute in the 'Special Attributes' section. - + “12” - dispatch to all destination in setid at once (parallel + + “12” - dispatch to all destinations in setid at once (parallel forking). Note that the XAVPs are no longer set with the values of the destination records, no re-routing making sense - in this case. + in this case. A side effect is that marking the one + destination as inactive is not possible if the transaction + fails (which means that all branches failed, or in other + words, all the destinations were not answering the call). + “13” - latency optimized dispatching - The algorithm will load balance using round-robin prioritizing the gateways with the highest priority.

1 0

git:master:91133002: core: pv - added $defs(name) return defined value as string between double quotes
by Daniel-Constantin Mierla 19 Jun '25

19 Jun '25

Module: kamailio Branch: master Commit: 91133002b6a36a3d1a8c983bfdecf3f9ac9e4b85 URL: https://github.com/kamailio/kamailio/commit/91133002b6a36a3d1a8c983bfdecf3f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-19T20:28:39+02:00 core: pv - added $defs(name) return defined value as string between double quotes - if not already enclosed in single or double quotes --- Modified: src/core/pv_core.c --- Diff: https://github.com/kamailio/kamailio/commit/91133002b6a36a3d1a8c983bfdecf3f… Patch: https://github.com/kamailio/kamailio/commit/91133002b6a36a3d1a8c983bfdecf3f… --- diff --git a/src/core/pv_core.c b/src/core/pv_core.c index f577c4ba3a9..214d78891c6 100644 --- a/src/core/pv_core.c +++ b/src/core/pv_core.c @@ -212,6 +212,51 @@ static int pv_get_defv(sip_msg_t *msg, pv_param_t *param, pv_value_t *res) } } +static int pv_parse_defs_name(pv_spec_p sp, str *in) +{ + if(in == NULL || in->s == NULL || sp == NULL) { + LM_ERR("INVALID DEF NAME\n"); + return -1; + } + sp->pvp.pvn.type = PV_NAME_INTSTR; + sp->pvp.pvn.u.isname.type = AVP_NAME_STR; + sp->pvp.pvn.u.isname.name.s = *in; + return 0; +} + +static int pv_get_defs(sip_msg_t *msg, pv_param_t *param, pv_value_t *res) +{ + str *val = NULL; + str ret = STR_NULL; + + val = pp_define_get( + param->pvn.u.isname.name.s.len, param->pvn.u.isname.name.s.s); + + if(!val) { + return pv_get_null(msg, param, res); + } + + if(val->len < 2) { + return pv_get_strval(msg, param, res, val); + } + if((val->s[0] == '"' && val->s[val->len - 1] == '"') + || (val->s[0] == '\'' && val->s[val->len - 1] == '\'')) { + return pv_get_strval(msg, param, res, val); + } + + ret.s = pv_get_buffer(); + ret.len = pv_get_buffer_size(); + if(ret.len + 3 < val->len) { + return pv_get_null(msg, param, res); + } + ret.s[0] = '"'; + memcpy(ret.s + 1, val->s, val->len); + ret.len = val->len + 2; + ret.s[ret.len - 1] = '"'; + ret.s[ret.len] = '\0'; + return pv_get_strval(msg, param, res, &ret); +} + /* clang-format off */ /** * @@ -231,6 +276,8 @@ static pv_export_t core_pvs[] = { 0, 0, 0}, {STR_STATIC_INIT("defv"), PVT_OTHER, pv_get_defv, 0, pv_parse_defv_name, 0, 0, 0}, + {STR_STATIC_INIT("defs"), PVT_OTHER, pv_get_defs, 0, pv_parse_defs_name, + 0, 0, 0}, {{0, 0}, 0, 0, 0, 0, 0, 0, 0} };

1 0

git:master:1b62f165: dispatcher: docs - more details about using algorithm 12
by Daniel-Constantin Mierla 19 Jun '25

19 Jun '25

Module: kamailio Branch: master Commit: 1b62f165d541cf88fe37778f0e23232e45e86b3d URL: https://github.com/kamailio/kamailio/commit/1b62f165d541cf88fe37778f0e23232… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-19T20:19:38+02:00 dispatcher: docs - more details about using algorithm 12 --- Modified: src/modules/dispatcher/doc/dispatcher_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/1b62f165d541cf88fe37778f0e23232… Patch: https://github.com/kamailio/kamailio/commit/1b62f165d541cf88fe37778f0e23232… --- diff --git a/src/modules/dispatcher/doc/dispatcher_admin.xml b/src/modules/dispatcher/doc/dispatcher_admin.xml index 3a6c26b41b7..5c36b3ff9f9 100644 --- a/src/modules/dispatcher/doc/dispatcher_admin.xml +++ b/src/modules/dispatcher/doc/dispatcher_admin.xml @@ -1413,10 +1413,13 @@ modparam("dispatcher", "ds_interval_mode", 7200) </listitem> <listitem> <para> - <quote>12</quote> - dispatch to all destination in setid at + <quote>12</quote> - dispatch to all destinations in setid at once (parallel forking). Note that the XAVPs are no longer set with the values of the destination records, no re-routing - making sense in this case. + making sense in this case. A side effect is that marking the + one destination as inactive is not possible if the transaction fails + (which means that all branches failed, or in other words, all the + destinations were not answering the call). </para> </listitem> <listitem>

1 0

[kamailio/kamailio] ims_diameter_server not freeing shared memory (Issue #4116)
by MGrimby 19 Jun '25

19 Jun '25

### Description I have run in to an issue when sending diameter requests using the ims_diameter_server module. Shared memory keeps increasing and are not being freed until restart of kamailio. This leads to all shared memory is being used and no more are able to be allocated. This only happens when the request is sent synchronously. When sent asynchronously shared memory is being freed as far as I can see. Following functions have been used for sending requests. diameter_request([peer], appid, commandcode, message) diameter_request_async([peer], appid, commandcode, message) ### Troubleshooting Sent a bunch of diameter requests and verified that shared memory for the following functions are not freed. do_receive(600): AAATranslateMessage(500): AAACreateAVP(137): #### Reproduction It's possible to reproduce by sending sync diameter request and verify shared memory. Example of diameter request block ``` route[CHECK_LOCATION] { diameter_request("hss.ims.test.local", "16777217", "306", "[ " + "{ \"avpCode\":260, \"vendorId\":0, \"Flags\":64, \"list\":[ { \"avpCode\":266, \"vendorId\":0, \"Flags\":64, \"int32\":10415 }, { \"avpCode\":258, \"vendorId\":0, \"Flags\":64, \"int32\":16777217 } ] }, " + "{ \"avpCode\":277, \"vendorId\":0, \"Flags\":64, \"int32\":1 }, " + "{ \"avpCode\":283, \"vendorId\":0, \"Flags\":64, \"string\":\"ims.test.local\"}, " + "{ \"avpCode\":706, \"vendorId\":10415, \"Flags\":192, \"int32\":0 }, " + "{ \"avpCode\":700, \"vendorId\":10415, \"Flags\":192, \"list\": [ { \"avpCode\":601, \"vendorId\":10415, \"Flags\":192, \"string\":\"tel:" + $fU + "\" }]}, " + "{ \"avpCode\":707, \"vendorId\":10415, \"Flags\":192, \"int32\":0 }, " + "{ \"avpCode\":703, \"vendorId\":10415, \"Flags\":192, \"int32\":14 } ]"); xlog("L_INFO", "Sent Diameter request"); } ``` Stats cdp:replies_received = 343310 shmem:fragments = 142 shmem:free_size = 1221489992 shmem:max_used_size = 948242168 shmem:real_used_size = 925993656 shmem:total_size = 2147483648 shmem:used_size = 558205656 kamcmd mod.stats cdp shm do_receive(600): 256245808 AAATranslateMessage(500): 64420712 AAACreateAVP(137): 218100320 Total: 538774000 #### Debugging Data  ``` (paste your debugging data here) ``` #### Log Messages  ``` (paste your log messages here) ``` #### SIP Traffic  ``` (paste your sip traffic here) ``` ### Possible Solutions  ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.7.4 (x86_64/linux) dc393e flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, MEM_JOIN_FREE, 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: dc393e compiled on 13:56:27 Feb 27 2024 with gcc 8.5.0 ``` * **Operating System**:  ``` Linux bmatkamas1 4.18.0-553.16.1.el8_10.x86_64 #1 SMP Thu Aug 1 04:16:12 EDT 2024 x86_64 x86_64 x86_64 GNU/Linux Red Hat Enterprise Linux release 8.10 (Ootpa) ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4116 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4116(a)github.com>

4 6

[kamailio/kamailio] rtpengine: make myseqn more random (Issue #4281)
by Christian Berger 18 Jun '25

18 Jun '25

ChristianBergerSipgate created an issue (kamailio/kamailio#4281) ### Description The rtpengine module uses a `cookie` field in the Protocol controlling rtpengines. This field needs to be unique, and is derived from the `server_id` field set in the Kamailio, the `pid` as well as the value of a variable `myseqn`. In many conditions, this is random enough. However particularly when using docker, the `pid` will be identical across servers, reducing the randomness and causing collisions. ### Expected behavior The 34 bytes of the cookie should be as unique as possible. #### Actual observed behavior We get collisions many times per day, as different kamailio instances randomly choose the same value. ### Possible Solutions 1. Set `myseqn` to a random value during module load. 2. Add an additional random value to the `gencookie` function. 3. Use `%x` instead of `%d` to squeeze more entropy into the 33 characters available, and perhaps use fixed length fields. The problem probably also exists in the following other modules: 1. rtpproxy 2. lrkproxy We are not using those modules so we have not yet experienced collisions here. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4281 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4281(a)github.com>

2 5

git:master:4935196e: lrkproxy: improve randomness of cookie by using PNRG and random sequence numbers
by Henning Westerholt 18 Jun '25

18 Jun '25

Module: kamailio Branch: master Commit: 4935196ee8ce1d99e77c614aaa0131a1972580ed URL: https://github.com/kamailio/kamailio/commit/4935196ee8ce1d99e77c614aaa0131a… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2025-06-18T11:10:02Z lrkproxy: improve randomness of cookie by using PNRG and random sequence numbers --- Modified: src/modules/lrkproxy/lrkproxy.c --- Diff: https://github.com/kamailio/kamailio/commit/4935196ee8ce1d99e77c614aaa0131a… Patch: https://github.com/kamailio/kamailio/commit/4935196ee8ce1d99e77c614aaa0131a… --- diff --git a/src/modules/lrkproxy/lrkproxy.c b/src/modules/lrkproxy/lrkproxy.c index bc0a83f4e23..ef6a347105b 100644 --- a/src/modules/lrkproxy/lrkproxy.c +++ b/src/modules/lrkproxy/lrkproxy.c @@ -77,6 +77,7 @@ #include "../../core/dset.h" #include "../../core/route.h" #include "../../core/kemi.h" +#include "../../core/rand/fastrand.h" #include "../../modules/tm/tm_load.h" #include "lrkproxy.h" #include "lrkproxy_hash.h" @@ -158,7 +159,6 @@ static void mod_destroy(void); static int lrkproxy_disable_tout = 60; static int lrkproxy_retr = 5; static int lrkproxy_tout = 1; -static pid_t mypid; static unsigned int myseqn = 0; //static str nolrkproxy_str = str_init("a=nolrkproxy:yes"); //static str extra_id_pv_param = {NULL, 0}; @@ -668,9 +668,10 @@ static int child_init(int rank) return 0; } - /* Iterate known LRK proxies - create sockets */ - mypid = getpid(); + /* random start value for for cookie sequence number */ + myseqn = fastrand(); + /* Iterate known RTP proxies - create sockets */ lrkp_socks = (int *)pkg_malloc(sizeof(int) * lrkp_no); if(lrkp_socks == NULL) { LM_ERR("no more pkg memory\n"); @@ -791,7 +792,7 @@ static char *gencookie(void) { static char cook[34]; - sprintf(cook, "%d_%u ", (int)mypid, myseqn); + sprintf(cook, "%d_%u ", fastrand(), myseqn); myseqn++; return cook; }

1 0

git:master:04686c40: rtpproxy: improve randomness of cookie by using PNRG and random sequence numbers
by Henning Westerholt 18 Jun '25

18 Jun '25

Module: kamailio Branch: master Commit: 04686c40d380e16e55c359cbc6bfc28f644a4c0b URL: https://github.com/kamailio/kamailio/commit/04686c40d380e16e55c359cbc6bfc28… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2025-06-18T11:09:38Z rtpproxy: improve randomness of cookie by using PNRG and random sequence numbers --- Modified: src/modules/rtpproxy/rtpproxy.c --- Diff: https://github.com/kamailio/kamailio/commit/04686c40d380e16e55c359cbc6bfc28… Patch: https://github.com/kamailio/kamailio/commit/04686c40d380e16e55c359cbc6bfc28… --- diff --git a/src/modules/rtpproxy/rtpproxy.c b/src/modules/rtpproxy/rtpproxy.c index 2057cdfba5d..02641ce2ce3 100644 --- a/src/modules/rtpproxy/rtpproxy.c +++ b/src/modules/rtpproxy/rtpproxy.c @@ -76,6 +76,7 @@ #include "../../core/dset.h" #include "../../core/route.h" #include "../../core/kemi.h" +#include "../../core/rand/fastrand.h" #include "../../modules/tm/tm_load.h" #include "rtpproxy.h" #include "rtpproxy_funcs.h" @@ -153,7 +154,6 @@ static int pv_get_rtppstat_f(struct sip_msg *, pv_param_t *, pv_value_t *); static int rtpproxy_disable_tout = 60; static int rtpproxy_retr = 5; static int rtpproxy_tout = 1; -static pid_t mypid; static unsigned int myseqn = 0; static str nortpproxy_str = str_init("a=nortpproxy:yes"); static str extra_id_pv_param = {NULL, 0}; @@ -788,9 +788,10 @@ static int child_init(int rank) return 0; } - /* Iterate known RTP proxies - create sockets */ - mypid = getpid(); + /* random start value for for cookie sequence number */ + myseqn = fastrand(); + /* Iterate known RTP proxies - create sockets */ rtpp_socks = (int *)pkg_malloc(sizeof(int) * rtpp_no); if(rtpp_socks == NULL) { LM_ERR("no more pkg memory\n"); @@ -1206,7 +1207,7 @@ static char *gencookie(void) { static char cook[34]; - sprintf(cook, "%d_%u ", (int)mypid, myseqn); + sprintf(cook, "%u_%u ", fastrand(), myseqn); myseqn++; return cook; }

1 0

git:master:33b80fcc: rtpengine: improve randomness of cookie by using PNRG and random sequence numbers
by Henning Westerholt 18 Jun '25

18 Jun '25

Module: kamailio Branch: master Commit: 33b80fcc4d84f955b29bd65bccee9ef2a62e7000 URL: https://github.com/kamailio/kamailio/commit/33b80fcc4d84f955b29bd65bccee9ef… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2025-06-18T11:08:12Z rtpengine: improve randomness of cookie by using PNRG and random sequence numbers --- Modified: src/modules/rtpengine/rtpengine.c --- Diff: https://github.com/kamailio/kamailio/commit/33b80fcc4d84f955b29bd65bccee9ef… Patch: https://github.com/kamailio/kamailio/commit/33b80fcc4d84f955b29bd65bccee9ef… --- diff --git a/src/modules/rtpengine/rtpengine.c b/src/modules/rtpengine/rtpengine.c index 292843302fe..e7d6cf9d5b9 100644 --- a/src/modules/rtpengine/rtpengine.c +++ b/src/modules/rtpengine/rtpengine.c @@ -83,6 +83,7 @@ #include "../../core/char_msg_val.h" #include "../../core/utils/srjson.h" #include "../../core/cfg/cfg_struct.h" +#include "../../core/rand/fastrand.h" #include "../../modules/tm/tm_load.h" #include "../../modules/crypto/api.h" #include "../../modules/lwsc/api.h" @@ -327,7 +328,7 @@ static void parse_call_stats(bencode_item_t *, struct sip_msg *); static int control_cmd_tos = -1; static int rtpengine_allow_op = 0; static struct rtpp_node **queried_nodes_ptr = NULL; -static pid_t mypid; + static unsigned int myseqn = 0; static str extra_id_pv_param = {NULL, 0}; static char *setid_avp_param = NULL; @@ -3028,6 +3029,8 @@ static int mos_label_stats_parse(struct minmax_mos_label_stats *mmls) static int child_init(int rank) { + pid_t mypid = 0; + if(!rtpp_set_list) return 0; @@ -3038,7 +3041,7 @@ static int child_init(int rank) if(rank == PROC_MAIN) { if(rtpengine_dtmf_event_sock.len > 0) { - LM_DBG("Register RTPENGINE DTMF WORKER %d\n", mypid); + LM_DBG("Register RTPENGINE DTMF WORKER %d\n", getpid()); /* fork worker process */ mypid = fork_process(PROC_RPC, "RTPENGINE DTMF WORKER", 1); if(mypid < 0) { @@ -3060,7 +3063,9 @@ static int child_init(int rank) return 0; } - mypid = getpid(); + /* random start value for for cookie sequence number */ + myseqn = fastrand(); + LM_ERR("myseqn %u\n", myseqn); // vector of pointers to queried nodes queried_nodes_ptr = (struct rtpp_node **)pkg_malloc( @@ -3171,7 +3176,7 @@ static char *gencookie(void) { static char cook[34]; - snprintf(cook, 34, "%d_%d_%u ", server_id, (int)mypid, myseqn); + snprintf(cook, 34, "%d_%u_%u ", server_id, fastrand(), myseqn); myseqn++; return cook; }

1 0

git:master:300da5c5: core: improve random number generator initialization
by Henning Westerholt 18 Jun '25

18 Jun '25

Module: kamailio Branch: master Commit: 300da5c5a166be6724ebe14be29ac97d029c535f URL: https://github.com/kamailio/kamailio/commit/300da5c5a166be6724ebe14be29ac97… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2025-06-18T10:34:52Z core: improve random number generator initialization - move seeding of cryptographic randomness generator for system randomness to the specific generator - improve seeding of children random number generators by using individual initialized secure generators - don't add predictable values like system time or process pid to the seed values of children generators --- Modified: src/core/pt.c Modified: src/core/rand/cryptorand.c Modified: src/core/rand/cryptorand.h Modified: src/core/rand/fortuna/fortuna.c Modified: src/core/rand/fortuna/fortuna.h Modified: src/core/rand/fortuna/random.c Modified: src/core/rand/fortuna/random.h Modified: src/main.c --- Diff: https://github.com/kamailio/kamailio/commit/300da5c5a166be6724ebe14be29ac97… Patch: https://github.com/kamailio/kamailio/commit/300da5c5a166be6724ebe14be29ac97…

1 0

[kamailio/kamailio] Memory usage increases everytime tls.reload is executed (Issue #3823)
by Sebastian Denz 18 Jun '25

18 Jun '25

### Description We are using Kamailio 5.7.4 on Debian 12 (from http://deb.kamailio.org/kamailio57) with rtpengine as an Edgeproxy for our clients. The instance terminates SIP/TLS (with Cliencertificates) and forwards the SIP Traffic to internal systems. After some days we are getting errors like this `tls_complete_init(): tls: ssl bug #1491 workaround: not enough memory for safe operation: shm=7318616 threshold1=8912896` First we thought Kamailio just doesnt have enough memory, so we doubled it.. But after some days the Logmessage (and Userissues) occured again. So we monitored the shmmem statistics and found that used and max_used are constantly growing til it reaches the limit. As i mentioned we are using client-certificates and so we are also using the CRL feature. We do have a systemd-timer which fetches the CRL every hour and runs 'kamcmd tls.reload' when finished. Our tls.cfg looks like this: ``` [server:default] method = TLSv1.2+ private_key = /etc/letsencrypt/live/hostname.de/privkey.pem certificate = /etc/letsencrypt/live/hostname.de/fullchain.pem ca_list = /etc/kamailio/ca_list.pem ca_path = /etc/kamailio/ca_list.pem crl = /etc/kamailio/combined.crl.pem verify_certificate = yes require_certificate = yes [client:default] verify_certificate = yes require_certificate = yes ``` After testing a bit we found that every time tls.reload is executed Kamailio consumes a bit more memory which eventually leads to all the memory being consumed which leads to issues for our users. See following example: ``` [0][root@edgar-dev:~]# while true ; do /usr/sbin/kamcmd tls.reload ; /usr/sbin/kamcmd core.shmmem ; sleep 1 ; done Ok. TLS configuration reloaded. { total: 268435456 free: 223001520 used: 41352552 real_used: 45433936 max_used: 45445968 fragments: 73 } Ok. TLS configuration reloaded. { total: 268435456 free: 222377960 used: 41975592 real_used: 46057496 max_used: 46069232 fragments: 78 } Ok. TLS configuration reloaded. { total: 268435456 free: 221748664 used: 42604992 real_used: 46686792 max_used: 46698080 fragments: 77 } Ok. TLS configuration reloaded. { total: 268435456 free: 221110832 used: 43242408 real_used: 47324624 max_used: 47335608 fragments: 81 } ^C [130][root@edgar-dev:~]# ``` ### Troubleshooting #### Reproduction Everytime tls.reload is called the memory consumptions grows.. #### Debugging Data  ``` If you let me know what would be interesting for tracking this down, i am happy to provide logs/debugging data! ``` #### Log Messages  ``` If you let me know what would be interesting for tracking this down, i am happy to provide logs/debugging data! ``` #### SIP Traffic SIP doesnt seem to be relevant here ### Possible Solutions Calling tls.reload less often or restart kamailio before memory is consumed ;) ### Additional Information ``` version: kamailio 5.7.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, MEM_JOIN_FREE, 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 12.2.0 ``` * **Operating System**: ``` * Debian GNU/Linux 12 (bookworm) * Linux edgar-dev 6.1.0-20-amd64 #1 SMP PREEMPT_DYNAMIC Debian 6.1.85-1 (2024-04-11) x86_64 GNU/Linux ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3823 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3823(a)github.com>

9 33

[kamailio/kamailio] tm: Emit event only when CANCEL request is locally generated (PR #4286)
by Xenofon Karamanos 18 Jun '25

18 Jun '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 - [ ] 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 - [x] Related to issue #3831 and #4249 #### Description  This PR intoduces a flag for the tm cancel logic. The flag denotes that the cancel request is locally generated and an event of `local_request` should be emmited. Updated also the modules and functions that used tm bindings to adopt for this change. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4286 -- Commit Summary -- * tm: Generate local_request for CANCEL only if locally initiated * dialog: Update to use new tm FLAG for CANCEL * rtp_media_server: Update to use new tm FLAG for CANCEL * tmx: Update to use new tm FLAG for CANCEL -- File Changes -- M src/modules/dialog/dlg_req_within.c (2) M src/modules/rtp_media_server/rtp_media_server.c (2) M src/modules/tm/t_cancel.c (69) M src/modules/tm/t_cancel.h (2) M src/modules/tmx/tmx_mod.c (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4286.patch https://github.com/kamailio/kamailio/pull/4286.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4286 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4286(a)github.com>

3 3

[kamailio/kamailio] dlgs: Fix spelling mistake (PR #4288)
by Xenofon Karamanos 18 Jun '25

18 Jun '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  Fix a spelling mistake You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/4288 -- Commit Summary -- * dlgs: Fix spelling mistake -- File Changes -- M src/modules/dlgs/dlgs_records.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/4288.patch https://github.com/kamailio/kamailio/pull/4288.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/4288 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/4288(a)github.com>

3 5

git:5.8:ae4342ad: dlgs: Fix spelling mistake
by Xenofon Karamanos 18 Jun '25

18 Jun '25

Module: kamailio Branch: 5.8 Commit: ae4342ad20a0e98681f83f1874294869d930e99d URL: https://github.com/kamailio/kamailio/commit/ae4342ad20a0e98681f83f187429486… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2025-06-18T08:44:43Z dlgs: Fix spelling mistake (cherry picked from commit 994d4dfa9cd284f63092543400a5ed82b746a758) --- Modified: src/modules/dlgs/dlgs_records.c Modified: src/modules/dlgs/dlgs_records.h --- Diff: https://github.com/kamailio/kamailio/commit/ae4342ad20a0e98681f83f187429486… Patch: https://github.com/kamailio/kamailio/commit/ae4342ad20a0e98681f83f187429486… --- diff --git a/src/modules/dlgs/dlgs_records.c b/src/modules/dlgs/dlgs_records.c index f6cad29856e..38ecb3d5ee3 100644 --- a/src/modules/dlgs/dlgs_records.c +++ b/src/modules/dlgs/dlgs_records.c @@ -692,7 +692,7 @@ void dlgs_update_stats(dlgs_stats_t *stats, int state, int val) stats->c_confirmed += val; return; case DLGS_STATE_TERMINATED: - stats->c_terminted += val; + stats->c_terminated += val; return; case DLGS_STATE_NOTANSWERED: stats->c_notanswered += val; @@ -1046,7 +1046,7 @@ static void dlgs_rpc_stats(rpc_t *rpc, void *ctx) sta.c_progress += _dlgs_htb->slots[i].astats.c_progress; sta.c_answered += _dlgs_htb->slots[i].astats.c_answered; sta.c_confirmed += _dlgs_htb->slots[i].astats.c_confirmed; - sta.c_terminted += _dlgs_htb->slots[i].astats.c_terminted; + sta.c_terminated += _dlgs_htb->slots[i].astats.c_terminated; sta.c_notanswered += _dlgs_htb->slots[i].astats.c_notanswered; } sti = &sta; @@ -1058,7 +1058,7 @@ static void dlgs_rpc_stats(rpc_t *rpc, void *ctx) } if(rpc->struct_add(ti, "uuuuuu", "init", sti->c_init, "progress", sti->c_progress, "answered", sti->c_answered, "confirmed", - sti->c_confirmed, "terminted", sti->c_terminted, + sti->c_confirmed, "terminated", sti->c_terminated, "notanswered", sti->c_notanswered) < 0) { rpc->fault(ctx, 500, "Internal error creating values"); diff --git a/src/modules/dlgs/dlgs_records.h b/src/modules/dlgs/dlgs_records.h index e6a02824a62..0dfeba797c9 100644 --- a/src/modules/dlgs/dlgs_records.h +++ b/src/modules/dlgs/dlgs_records.h @@ -43,7 +43,7 @@ typedef struct _dlgs_stats { unsigned int c_progress; unsigned int c_answered; unsigned int c_confirmed; - unsigned int c_terminted; + unsigned int c_terminated; unsigned int c_notanswered; } dlgs_stats_t;

1 0

git:6.0:c0e60449: dlgs: Fix spelling mistake
by Xenofon Karamanos 18 Jun '25

18 Jun '25

Module: kamailio Branch: 6.0 Commit: c0e60449ae0d35fd9e753d0da92c98c03e2a4494 URL: https://github.com/kamailio/kamailio/commit/c0e60449ae0d35fd9e753d0da92c98c… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2025-06-18T08:41:37Z dlgs: Fix spelling mistake (cherry picked from commit 994d4dfa9cd284f63092543400a5ed82b746a758) --- Modified: src/modules/dlgs/dlgs_records.c Modified: src/modules/dlgs/dlgs_records.h --- Diff: https://github.com/kamailio/kamailio/commit/c0e60449ae0d35fd9e753d0da92c98c… Patch: https://github.com/kamailio/kamailio/commit/c0e60449ae0d35fd9e753d0da92c98c… --- diff --git a/src/modules/dlgs/dlgs_records.c b/src/modules/dlgs/dlgs_records.c index f6cad29856e..38ecb3d5ee3 100644 --- a/src/modules/dlgs/dlgs_records.c +++ b/src/modules/dlgs/dlgs_records.c @@ -692,7 +692,7 @@ void dlgs_update_stats(dlgs_stats_t *stats, int state, int val) stats->c_confirmed += val; return; case DLGS_STATE_TERMINATED: - stats->c_terminted += val; + stats->c_terminated += val; return; case DLGS_STATE_NOTANSWERED: stats->c_notanswered += val; @@ -1046,7 +1046,7 @@ static void dlgs_rpc_stats(rpc_t *rpc, void *ctx) sta.c_progress += _dlgs_htb->slots[i].astats.c_progress; sta.c_answered += _dlgs_htb->slots[i].astats.c_answered; sta.c_confirmed += _dlgs_htb->slots[i].astats.c_confirmed; - sta.c_terminted += _dlgs_htb->slots[i].astats.c_terminted; + sta.c_terminated += _dlgs_htb->slots[i].astats.c_terminated; sta.c_notanswered += _dlgs_htb->slots[i].astats.c_notanswered; } sti = &sta; @@ -1058,7 +1058,7 @@ static void dlgs_rpc_stats(rpc_t *rpc, void *ctx) } if(rpc->struct_add(ti, "uuuuuu", "init", sti->c_init, "progress", sti->c_progress, "answered", sti->c_answered, "confirmed", - sti->c_confirmed, "terminted", sti->c_terminted, + sti->c_confirmed, "terminated", sti->c_terminated, "notanswered", sti->c_notanswered) < 0) { rpc->fault(ctx, 500, "Internal error creating values"); diff --git a/src/modules/dlgs/dlgs_records.h b/src/modules/dlgs/dlgs_records.h index e6a02824a62..0dfeba797c9 100644 --- a/src/modules/dlgs/dlgs_records.h +++ b/src/modules/dlgs/dlgs_records.h @@ -43,7 +43,7 @@ typedef struct _dlgs_stats { unsigned int c_progress; unsigned int c_answered; unsigned int c_confirmed; - unsigned int c_terminted; + unsigned int c_terminated; unsigned int c_notanswered; } dlgs_stats_t;

1 0

git:6.0:0637ca7e: tmx: Update to use new tm FLAG for CANCEL
by Xenofon Karamanos 18 Jun '25

18 Jun '25

Module: kamailio Branch: 6.0 Commit: 0637ca7e0b7795c33ab9f230c0eee65d6ea80e8d URL: https://github.com/kamailio/kamailio/commit/0637ca7e0b7795c33ab9f230c0eee65… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2025-06-18T08:40:39Z tmx: Update to use new tm FLAG for CANCEL (cherry picked from commit caad49279e6db7e85e3f2b84276f9defb0aa4eef) --- Modified: src/modules/tmx/tmx_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/0637ca7e0b7795c33ab9f230c0eee65… Patch: https://github.com/kamailio/kamailio/commit/0637ca7e0b7795c33ab9f230c0eee65… --- diff --git a/src/modules/tmx/tmx_mod.c b/src/modules/tmx/tmx_mod.c index aae6547834d..4e1e7c78289 100644 --- a/src/modules/tmx/tmx_mod.c +++ b/src/modules/tmx/tmx_mod.c @@ -333,7 +333,7 @@ static int t_cancel_branches_helper(sip_msg_t *msg, int n) LM_DBG("canceling %d/%d\n", n, (int)cancel_data.cancel_bitmap); if(cancel_data.cancel_bitmap == 0) return -1; - _tmx_tmb.cancel_uacs(t, &cancel_data, 0); + _tmx_tmb.cancel_uacs(t, &cancel_data, F_CANCEL_LOCAL); return 1; } @@ -408,7 +408,7 @@ static int ki_t_cancel_callid_reason( cancel_data.reason.cause = rcode; cancel_data.cancel_bitmap = 0; _tmx_tmb.prepare_to_cancel(trans, &cancel_data.cancel_bitmap, 0); - _tmx_tmb.cancel_uacs(trans, &cancel_data, 0); + _tmx_tmb.cancel_uacs(trans, &cancel_data, F_CANCEL_LOCAL); //_tmx_tmb.unref_cell(trans); _tmx_tmb.t_sett(bkt, bkb);

1 0

git:6.0:c3160efc: rtp_media_server: Update to use new tm FLAG for CANCEL
by Xenofon Karamanos 18 Jun '25

18 Jun '25

Module: kamailio Branch: 6.0 Commit: c3160efc9b4af01d8e2997fdc6cb168838f76ba9 URL: https://github.com/kamailio/kamailio/commit/c3160efc9b4af01d8e2997fdc6cb168… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Xenofon Karamanos <22965395+xkaraman(a)users.noreply.github.com> Date: 2025-06-18T08:39:41Z rtp_media_server: Update to use new tm FLAG for CANCEL (cherry picked from commit fc5f804ad92593f74110a9392d63384e04fed218) --- Modified: src/modules/rtp_media_server/rtp_media_server.c --- Diff: https://github.com/kamailio/kamailio/commit/c3160efc9b4af01d8e2997fdc6cb168… Patch: https://github.com/kamailio/kamailio/commit/c3160efc9b4af01d8e2997fdc6cb168… --- diff --git a/src/modules/rtp_media_server/rtp_media_server.c b/src/modules/rtp_media_server/rtp_media_server.c index 720c4afd900..21abb7fb836 100644 --- a/src/modules/rtp_media_server/rtp_media_server.c +++ b/src/modules/rtp_media_server/rtp_media_server.c @@ -1024,7 +1024,7 @@ static int rms_sip_cancel(struct sip_msg *msg, str *callid_s, str *cseq_s) cancel_data.reason.cause = rcode; cancel_data.cancel_bitmap = 0; tmb.prepare_to_cancel(trans, &cancel_data.cancel_bitmap, 0); - tmb.cancel_uacs(trans, &cancel_data, 0); + tmb.cancel_uacs(trans, &cancel_data, F_CANCEL_LOCAL); tmb.t_sett(bkt, bkb); if(!tmb.t_reply(msg, 202, "cancelling")) {

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev