- sr-dev - kamailio.org

[kamailio/kamailio] Invalid $nghttp2(...) (Issue #4299)
by QWERTY 26 Jun '25

26 Jun '25

2954456878 created an issue (kamailio/kamailio#4299) version: kamailio 6.0.1 (x86_64/linux) loadmodule "nghttp2.so" modparam("nghttp2", "listen_addr", "0.0.0.0") modparam("nghttp2", "listen_port", "8363") event_route[nghttp2:request] { xinfo("request: $nghttp2(method) - url: $nghttp2(url) - data: [$nghttp2(data)]\n"); nghttp2_reply("200", "<html><body>OK</body></html>"); } Startup error： Jun 25 10:42:50 debiancrs /usr/local/kamailio6-devel/sbin/kamailio[41907]: ERROR: nghttp2 [nghttp2_mod.c:246]: pv_parse_nghttp2_name(): invalid variable name [url] Jun 25 10:42:50 debiancrs /usr/local/kamailio6-devel/sbin/kamailio[41907]: ERROR: <core> [core/pvapi.c:954]: pv_parse_spec2(): pvar "nghttp2" has an invalid name param [url] Jun 25 10:42:50 debiancrs /usr/local/kamailio6-devel/sbin/kamailio[41907]: ERROR: <core> [core/pvapi.c:1092]: pv_parse_spec2(): wrong char [)/41] in [$nghttp2(url)] at [12 (5)] Jun 25 10:42:50 debiancrs /usr/local/kamailio6-devel/sbin/kamailio[41907]: ERROR: xlog [xlog.c:550]: xdbg_fixup_helper(): wrong format[request: $nghttp2(method) - url: $nghttp2(url) - data: [$nghttp2(data)]#012] -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4299 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4299(a)github.com>

2 2

git:master:64bc28f1: modules: readme files regenerated - nghttp2 ... [skip ci]
by Kamailio Dev 26 Jun '25

26 Jun '25

Module: kamailio Branch: master Commit: 64bc28f1279809829b714d23314b3e62030a8ccb URL: https://github.com/kamailio/kamailio/commit/64bc28f1279809829b714d23314b3e6… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2025-06-26T09:31:10+02:00 modules: readme files regenerated - nghttp2 ... [skip ci] --- Modified: src/modules/nghttp2/README --- Diff: https://github.com/kamailio/kamailio/commit/64bc28f1279809829b714d23314b3e6… Patch: https://github.com/kamailio/kamailio/commit/64bc28f1279809829b714d23314b3e6… --- diff --git a/src/modules/nghttp2/README b/src/modules/nghttp2/README index f880b2e5a13..fc79a0fbae3 100644 --- a/src/modules/nghttp2/README +++ b/src/modules/nghttp2/README @@ -184,15 +184,15 @@ event_route[nghttp2:request] { The event route is executed when a new HTTP request is received. Inside it, the $nghttp2(...) group of variables is available, giving - access to several attributes of the HTTP request, such as method, URL, - data (body) or headers. + access to several attributes of the HTTP request, such as method, URL + path, data (body) or headers. ... ... loadmodule "nghttp2.so ... event_route[nghttp2:request] { - xinfo("request: $nghttp2(method) - url: $nghttp2(url) - data: [$nghttp2(data -)]\n"); + xinfo("request: $nghttp2(method) - url: $nghttp2(path) - data: [$nghttp2(dat +a)]\n"); nghttp2_reply("200", "OK", "text/html", "<html><body>OK</body></html>"); }

1 0

git:master:b513666f: nghttp2: docs updated to use the proper $nghttp2(path) variable
by Daniel-Constantin Mierla 26 Jun '25

26 Jun '25

Module: kamailio Branch: master Commit: b513666f76d2220d8914fadcc4968845efbf170f URL: https://github.com/kamailio/kamailio/commit/b513666f76d2220d8914fadcc496884… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-26T09:05:10+02:00 nghttp2: docs updated to use the proper $nghttp2(path) variable --- Modified: src/modules/nghttp2/doc/nghttp2_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/b513666f76d2220d8914fadcc496884… Patch: https://github.com/kamailio/kamailio/commit/b513666f76d2220d8914fadcc496884… --- diff --git a/src/modules/nghttp2/doc/nghttp2_admin.xml b/src/modules/nghttp2/doc/nghttp2_admin.xml index 54cde446de6..a09bb5d0b33 100644 --- a/src/modules/nghttp2/doc/nghttp2_admin.xml +++ b/src/modules/nghttp2/doc/nghttp2_admin.xml @@ -182,7 +182,7 @@ event_route[nghttp2:request] { <para> Inside it, the $nghttp2(...) group of variables is available, giving access to several attributes of the HTTP request, such as method, - URL, data (body) or headers. + URL path, data (body) or headers. </para> <programlisting format="linespecific"> ... @@ -190,7 +190,7 @@ event_route[nghttp2:request] { loadmodule "nghttp2.so ... event_route[nghttp2:request] { - xinfo("request: $nghttp2(method) - url: $nghttp2(url) - data: [$nghttp2(data)]\n"); + xinfo("request: $nghttp2(method) - url: $nghttp2(path) - data: [$nghttp2(data)]\n"); nghttp2_reply("200", "OK", "text/html", "<html><body>OK</body></html>"); }

1 0

[kamailio/kamailio] Topos not working Properly for ACK (Issue #4208)
by Pandi1981 26 Jun '25

26 Jun '25

Pandi1981 created an issue (kamailio/kamailio#4208) We are facing an issue while using TOPOS in Kamailio P-CSCF (running under Docker). Kamailio version: 6.0.0 operating system: "10 (buster)" **Call Flow & Issue Description:** Note: Public IP and Private IP mapping using the below configuration. ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ listen=udp:LOCAL_IP:5060 name "internal" listen=udp:LOCAL_IP:6080 advertise PUBLIC_IP:6080 name "external" ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ **Call Flow (INVITE):** A-Party --> Kamailio P-CSCF (Public IP) --> Kamailio P-CSCF (Private IP) --> Kamailio S-CSCF (Private IP) B-Party <-- Kamailio P-CSCF (Public IP) <-- Kamailio P-CSCF (Private IP) <-- Kamailio S-CSCF (Private IP) Everything works correctly up to the point where the 200 OK is received by the A-Party. However, when the A-Party sends the ACK, it is received by the Kamailio P-CSCF but is not forwarded to the B-Party. Upon investigation, we found that when Kamailio receives the ACK from the A-Party, the rr (Record-Route) module fails to recognize its own Route header and thus falls back to strict routing instead of performing loose routing. This results in an incorrect rewrite of the Request-URI of the ACK, causing it to fail to reach the B-Party. Essentially, the Route header is not popped as expected and the ACK routing breaks. TOPOS module is configured as follows: loadmodule "topos_redis.so" loadmodule "ndb_redis.so" loadmodule "topos.so" modparam("ndb_redis", "server", "name=tps;addr=127.0.0.1;port=6379;db=0") modparam("topos", "storage", "redis") modparam("topos_redis", "serverid", "tps") modparam("topos", "rr_update", 1) modparam("topos", "contact_mode", 1) modparam("topos", "xavu_field_a_contact", "a_contact") modparam("topos", "xavu_field_b_contact", "b_contact") modparam("topos", "xavu_cfg", "tps") Please guide us how to achieve and what we are doing wrong. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4208 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4208(a)github.com>

3 6

[kamailio/kamailio] http_client leaks TLS shared memory (Issue #4231)
by Alexander Bakker 25 Jun '25

25 Jun '25

alexbakker created an issue (kamailio/kamailio#4231) ### Description Performing an HTTPS request using the ``http_client`` module results in a shared memory leak, seemingly related to TLS. ### Troubleshooting #### Reproduction Our use case for ``http_client`` is to update an htable with the latest banned IP's from APIBAN as described here: https://github.com/apiban/apiban?tab=readme-ov-file#integration-into-kamail…. Normally, we refresh the list of banned IP's once every 15 minutes, so the shared memory usage grows fairly slowly. The issue can be reproduced using the Kamailio configuration file below. To demonstrate the issue and make the shared memory usage grow faster, we set up a timer that calls ``http_client_query`` in a tight loop. To prevent unnecessary load on the APIBAN infrastructure, we request an example file from my personal server instead. ``` #!KAMAILIO debug=2 log_stderror=yes fork=yes enable_tls=1 loadmodule "tls.so" loadmodule "cfg_rpc.so" loadmodule "pv.so" loadmodule "xlog.so" loadmodule "ctl.so" loadmodule "rtimer.so" loadmodule "http_client.so" modparam("tls", "certificate", "") modparam("tls", "private_key", "") modparam("rtimer", "timer", "name=apiban;interval=100u;mode=1;") modparam("rtimer", "exec", "timer=apiban;route=APIBAN;") route[APIBAN] { xinfo("running apiban refresh\n"); http_client_query("https://alexbakker.me/u/7bvjn9jfas.txt", "$var(banned)"); } ``` Start Kamailio: ``` kamailio -f kamailio.cfg -DD ``` And then watch the shared memory usage grow rapidly: ``` watch -n1 -- kamcmd core.shmmem ``` #### Log Messages Eventually, Kamailio will run out of shared memory and print the following messages to the log: ``` 21(184184) INFO: <script>: running apiban refresh 21(184184) ERROR: http_client [functions.c:471]: curL_request_url(): failed to perform curl (56) (url: https://alexbakker.me/u/7bvjn9jfas.txt) 21(184184) INFO: <script>: running apiban refresh 21(184184) ERROR: http_client [functions.c:471]: curL_request_url(): failed to perform curl (56) (url: https://alexbakker.me/u/7bvjn9jfas.txt) 21(184184) INFO: <script>: running apiban refresh 21(184184) WARNING: http_client [functions.c:453]: curL_request_url(): TLS error in curl connection (url: https://alexbakker.me/u/7bvjn9jfas.txt) 21(184184) INFO: <script>: running apiban refresh 21(184184) WARNING: http_client [functions.c:453]: curL_request_url(): TLS error in curl connection (url: https://alexbakker.me/u/7bvjn9jfas.txt) 21(184184) INFO: <script>: running apiban refresh 21(184184) WARNING: http_client [functions.c:453]: curL_request_url(): TLS error in curl connection (url: https://alexbakker.me/u/7bvjn9jfas.txt) 21(184184) INFO: <script>: running apiban refresh 21(184184) WARNING: http_client [functions.c:453]: curL_request_url(): TLS error in curl connection (url: https://alexbakker.me/u/7bvjn9jfas.txt) 21(184184) INFO: <script>: running apiban refresh 21(184184) ERROR: <core> [core/mem/q_malloc.c:758]: qm_realloc(): qm_realloc(0x7585c73c4000, 1024) called from tls: tls_init.c: ser_realloc(372), module: tls; qm_malloc() failed! 21(184184) INFO: <script>: running apiban refresh 21(184184) INFO: <script>: running apiban refresh 21(184184) ERROR: <core> [core/mem/q_malloc.c:758]: qm_realloc(): qm_realloc(0x7585c73c4000, 1024) called from tls: tls_init.c: ser_realloc(372), module: tls; qm_malloc() failed! 21(184184) WARNING: http_client [functions.c:453]: curL_request_url(): TLS error in curl connection (url: https://alexbakker.me/u/7bvjn9jfas.txt) 21(184184) INFO: <script>: running apiban refresh 21(184184) INFO: <script>: running apiban refresh 21(184184) WARNING: http_client [functions.c:463]: curL_request_url(): TLS CA certificate read error (url: https://alexbakker.me/u/7bvjn9jfas.txt) ``` ``mem_dump_shm`` reports a large list of TLS-related allocations. Just sharing the last few lines here, as they're seemingly all from the same location. ``` 20(204914) ALERT: qm_status: qm_status(): 28500. N address=0x705ab3178710 frag=0x705ab31786d0 size=112 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28501. N address=0x705ab31787f0 frag=0x705ab31787b0 size=48 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28502. N address=0x705ab3178890 frag=0x705ab3178850 size=32 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28503. N address=0x705ab3178920 frag=0x705ab31788e0 size=32 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28504. N address=0x705ab31789b0 frag=0x705ab3178970 size=32 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28505. N address=0x705ab3178a40 frag=0x705ab3178a00 size=48 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28506. N address=0x705ab3178ae0 frag=0x705ab3178aa0 size=48 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28507. N address=0x705ab3178b80 frag=0x705ab3178b40 size=144 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28509. N address=0x705ab3178d00 frag=0x705ab3178cc0 size=32 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28510. N address=0x705ab3178d90 frag=0x705ab3178d50 size=32 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_realloc(372) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28512. N address=0x705ab3178ea0 frag=0x705ab3178e60 size=16 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28513. N address=0x705ab3178f20 frag=0x705ab3178ee0 size=32 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28515. N address=0x705ab317a3c0 frag=0x705ab317a380 size=112 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28516. N address=0x705ab317a4a0 frag=0x705ab317a460 size=512 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28517. N address=0x705ab317a710 frag=0x705ab317a6d0 size=1504 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed 20(204914) ALERT: qm_status: qm_status(): 28518. N address=0x705ab317ad60 frag=0x705ab317ad20 size=1040 used=1 20(204914) ALERT: qm_status: qm_status(): alloc'd from tls: tls_init.c: ser_malloc(364) 20(204914) ALERT: qm_status: qm_status(): start check=f0f0f0f0, end check= c0c0c0c0, abcdefed ``` ### Possible Solutions I'm not aware of a workaround for this issue. For our specific use case, I'll move the APIBAN polling out of Kamailio to a separate process for now. ### Additional Information I've verified that this issue is reproducible on ``5.5.7``, ``5.6.6``, ``5.7.6``, ``5.8.6`` and ``6.0.1``. * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 6.0.1 (x86_64/linux) fce50d 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_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: fce50d compiled on 06:42:29 May 6 2025 with gcc 12.2.0 ``` * **Operating System**: ``` $ lsb_release -a No LSB modules are available. Distributor ID: Debian Description: Debian GNU/Linux 12 (bookworm) Release: 12 Codename: bookworm $ uname -a Linux ip-10-146-128-121 6.1.0-32-cloud-arm64 #1 SMP Debian 6.1.129-1 (2025-03-06) aarch64 GNU/Linux ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4231 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4231(a)github.com>

4 11

git:master:092ad634: core: print ip and port in log message
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: master Commit: 092ad634a2741ad21fd29586a30bd2bcb9ac653b URL: https://github.com/kamailio/kamailio/commit/092ad634a2741ad21fd29586a30bd2b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T20:34:23+02:00 core: print ip and port in log message --- Modified: src/core/msg_translator.c --- Diff: https://github.com/kamailio/kamailio/commit/092ad634a2741ad21fd29586a30bd2b… Patch: https://github.com/kamailio/kamailio/commit/092ad634a2741ad21fd29586a30bd2b… --- diff --git a/src/core/msg_translator.c b/src/core/msg_translator.c index 3a46f7106f5..e27b70deec1 100644 --- a/src/core/msg_translator.c +++ b/src/core/msg_translator.c @@ -2918,7 +2918,7 @@ char *via_builder(unsigned int *len, sip_msg_t *msg, #endif /* USE_COMP */ int port; int proto; - struct ip_addr ip; + struct ip_addr ip = {0}; union sockaddr_union *from = NULL; union sockaddr_union local_addr; struct tcp_connection *con = NULL; @@ -3064,8 +3064,8 @@ char *via_builder(unsigned int *len, sip_msg_t *msg, if(con == NULL) { LM_WARN("TCP/TLS connection (id: %d) for WebSocket could not be " - "found\n", - send_info->id); + "found - likely it is gone (dst: [%s]:%d)\n", + send_info->id, (port) ? ip_addr2a(&ip) : "", port); pkg_free(line_buf); return 0; }

1 0

[kamailio/kamailio] Release 6.0.2 - 6.0.2
by github-actions[bot] 25 Jun '25

25 Jun '25

-- View it on GitHub: https://github.com/kamailio/kamailio/releases/tag/6.0.2 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/releases/227792045(a)github.com>

1 0

[kamailio/kamailio] "dns_hash_put(): unlinked item" downgrade of the WARN messages to INFO (Issue #4287)
by hubgurujr 25 Jun '25

25 Jun '25

hubgurujr created an issue (kamailio/kamailio#4287) Seeing a lot of warning messages in logs, running version: kamailio 5.8.4 (x86_64/linux) 1e6738, but I think this is false positive and should be informational, not a warning. Jun 10 14:14:17 siprouter1-pve186 /usr/local/sbin/kamailio[8111]:WARNING: <core> [core/dns_cache.c:170]: dns_hash_put(): unlinked item 0x7fa0f9b9fcc0 Jun 10 14:14:19 siprouter1-pve186 /usr/local/sbin/kamailio[8122]:WARNING: <core> [core/dns_cache.c:185]: dns_hash_put_shm_unsafe():unlinked item 0x7fa0f9b9fcc0 This is occurring when a DNS URL changes IP but I see no processing issues with SIP Messages. [emal-thread.txt](https://github.com/user-attachments/files/20776557/emal-th… -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/4287 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/4287(a)github.com>

2 8

git:master:3e3cfc6b: github: alpine docker image, removed cleanup
by Sergey Safarov 25 Jun '25

25 Jun '25

Module: kamailio Branch: master Commit: 3e3cfc6b113b3e9a7f1a3ac91897687fbb71f29a URL: https://github.com/kamailio/kamailio/commit/3e3cfc6b113b3e9a7f1a3ac91897687… Author: Sergey Safarov <s.safarov(a)anycast-lb.net> Committer: Sergey Safarov <s.safarov(a)anycast-lb.net> Date: 2025-06-25T18:46:07+03:00 github: alpine docker image, removed cleanup --- Modified: .github/workflows/alpine.yml --- Diff: https://github.com/kamailio/kamailio/commit/3e3cfc6b113b3e9a7f1a3ac91897687… Patch: https://github.com/kamailio/kamailio/commit/3e3cfc6b113b3e9a7f1a3ac91897687… --- diff --git a/.github/workflows/alpine.yml b/.github/workflows/alpine.yml index e5f7885e898..82fc4082199 100644 --- a/.github/workflows/alpine.yml +++ b/.github/workflows/alpine.yml @@ -426,18 +426,3 @@ jobs: context: "context" build-args: | IMAGE=${{ needs.check.outputs.alpine_tag }} - - cleanup: - name: Cleanup untaged images - runs-on: ubuntu-latest - continue-on-error: true - permissions: - packages: write - needs: - - images - steps: - - name: Cleanup untaged - uses: dataaxiom/ghcr-cleanup-action@v1 - with: - owner: ${{ github.repository_owner }} - package: ${{ env.IMAGE_NAME }}

1 0

git:master:db653681: dispatcher: clone dispatcher uri for event route variable
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: master Commit: db65368108b0de14e40cbbd7c92561e5c8740dac URL: https://github.com/kamailio/kamailio/commit/db65368108b0de14e40cbbd7c92561e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T17:03:58+02:00 dispatcher: clone dispatcher uri for event route variable --- Modified: src/modules/dispatcher/dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/db65368108b0de14e40cbbd7c92561e… Patch: https://github.com/kamailio/kamailio/commit/db65368108b0de14e40cbbd7c92561e… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index cb7ccf44685..4a5c2d90cf8 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -143,6 +143,7 @@ static int ds_strictest_match = 0; #define _ds_list (ds_lists[*ds_crt_idx]) #define _ds_list_nr (*ds_list_nr) +void ds_rctx_set_uri(ds_rctx_t *rctx, str *uri); static void ds_run_route( struct sip_msg *msg, str *uri, char *route, ds_rctx_t *rctx); @@ -3045,7 +3046,7 @@ int ds_mark_addr(sip_msg_t *msg, int state, int group, str *uri, int mode) rctx.code = 800; } rctx.setid = group; - rctx.uri = *uri; + ds_rctx_set_uri(&rctx, uri); ret = ds_update_state(msg, group, uri, state, mode, &rctx); @@ -3478,6 +3479,28 @@ int ds_update_state(sip_msg_t *msg, int group, str *address, int state, * */ static ds_rctx_t *_ds_rctx = NULL; +static char _ds_rctx_buri[MAX_URI_SIZE]; + +/** + * + */ +void ds_rctx_set_uri(ds_rctx_t *rctx, str *uri) +{ + _ds_rctx_buri[0] = '\0'; + rctx->uri.s = _ds_rctx_buri; + rctx->uri.len = 0; + if(uri == NULL || uri->s == NULL || uri->len < 0) { + return; + } + if(uri->len >= MAX_URI_SIZE - 1) { + LM_ERR("uri too long: %d\n", uri->len); + return; + } + memcpy(_ds_rctx_buri, uri->s, uri->len); + _ds_rctx_buri[uri->len] = '\0'; + rctx->uri.len = uri->len; + return; +} /** * @@ -4073,7 +4096,7 @@ static void ds_options_callback( } } rctx.setid = group; - rctx.uri = uri; + ds_rctx_set_uri(&rctx, &uri); /* Check if in the meantime someone disabled probing of the target * through RPC or reload */ @@ -4237,7 +4260,7 @@ void ds_ping_set(ds_set_t *node) rctx.reason.s = "Sending keepalive failed"; rctx.reason.len = 24; rctx.setid = node->id; - rctx.uri = node->dlist[j].uri; + ds_rctx_set_uri(&rctx, &node->dlist[j].uri); /* check if meantime someone disabled the target via RPC */ if(!(node->dlist[j].flags & DS_DISABLED_DST) && ds_update_state(NULL, node->id, &node->dlist[j].uri,

1 0

git:master:0e914414: dispatcher: $dsv(name) extended to return setid and uri
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: master Commit: 0e914414e48637168ab9ab682a9463412cf8f3fe URL: https://github.com/kamailio/kamailio/commit/0e914414e48637168ab9ab682a94634… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T13:22:17+02:00 dispatcher: $dsv(name) extended to return setid and uri --- 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/0e914414e48637168ab9ab682a94634… Patch: https://github.com/kamailio/kamailio/commit/0e914414e48637168ab9ab682a94634… --- diff --git a/src/modules/dispatcher/dispatch.c b/src/modules/dispatcher/dispatch.c index d484c40233c..cb7ccf44685 100644 --- a/src/modules/dispatcher/dispatch.c +++ b/src/modules/dispatcher/dispatch.c @@ -3044,6 +3044,9 @@ int ds_mark_addr(sip_msg_t *msg, int state, int group, str *uri, int mode) } else { rctx.code = 800; } + rctx.setid = group; + rctx.uri = *uri; + ret = ds_update_state(msg, group, uri, state, mode, &rctx); LM_DBG("state [%d] grp [%d] dst [%.*s]\n", state, group, uri->len, uri->s); @@ -4069,6 +4072,8 @@ static void ds_options_callback( rctx.reason = ps->rpl->first_line.u.reply.reason; } } + rctx.setid = group; + rctx.uri = uri; /* Check if in the meantime someone disabled probing of the target * through RPC or reload */ @@ -4231,6 +4236,8 @@ void ds_ping_set(ds_set_t *node) rctx.code = 500; rctx.reason.s = "Sending keepalive failed"; rctx.reason.len = 24; + rctx.setid = node->id; + rctx.uri = node->dlist[j].uri; /* check if meantime someone disabled the target via RPC */ if(!(node->dlist[j].flags & DS_DISABLED_DST) && ds_update_state(NULL, node->id, &node->dlist[j].uri, diff --git a/src/modules/dispatcher/dispatch.h b/src/modules/dispatcher/dispatch.h index c47ad4fbf5e..ffaab7cb7e8 100644 --- a/src/modules/dispatcher/dispatch.h +++ b/src/modules/dispatcher/dispatch.h @@ -90,6 +90,8 @@ typedef struct ds_rctx int flags; int code; str reason; + str uri; + int setid; } ds_rctx_t; extern str ds_db_url; diff --git a/src/modules/dispatcher/dispatcher.c b/src/modules/dispatcher/dispatcher.c index 00960311ead..35cfd3cc4a5 100644 --- a/src/modules/dispatcher/dispatcher.c +++ b/src/modules/dispatcher/dispatcher.c @@ -1392,6 +1392,13 @@ static int pv_get_dsv(sip_msg_t *msg, pv_param_t *param, pv_value_t *res) return pv_get_null(msg, param, res); case 2: return pv_get_sintval(msg, param, res, rctx->flags); + case 3: + if(rctx->uri.s != NULL && rctx->uri.len > 0) { + return pv_get_strval(msg, param, res, &rctx->uri); + } + return pv_get_null(msg, param, res); + case 4: + return pv_get_sintval(msg, param, res, rctx->setid); default: return pv_get_null(msg, param, res); } @@ -1406,6 +1413,12 @@ static int pv_parse_dsv(pv_spec_p sp, str *in) return -1; switch(in->len) { + case 3: + if(strncmp(in->s, "uri", 3) == 0) + sp->pvp.pvn.u.isname.name.n = 3; + else + goto error; + break; case 4: if(strncmp(in->s, "code", 4) == 0) sp->pvp.pvn.u.isname.name.n = 0; @@ -1415,6 +1428,10 @@ static int pv_parse_dsv(pv_spec_p sp, str *in) case 5: if(strncmp(in->s, "flags", 5) == 0) sp->pvp.pvn.u.isname.name.n = 2; + else if(strncmp(in->s, "setid", 5) == 0) + sp->pvp.pvn.u.isname.name.n = 4; + else if(strncmp(in->s, "group", 5) == 0) + sp->pvp.pvn.u.isname.name.n = 4; else goto error; break;

1 0

Kamailio v6.0.2 Released
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Hello, Kamailio SIP Server v6.0.2 stable release is out. This is a maintenance release of the latest stable branch, 6.0, that includes fixes since the release of v6.0.2. There is no change to database schema or configuration language structure that you have to do on previous installations of v6.0.x. Deployments running previous v6.0.x versions are strongly recommended to be upgraded to v6.0.2. For more details about version 6.0.2 (including links and guidelines to download the tarball or from GIT repository), visit: * https://www.kamailio.org/w/2025/06/kamailio-v6-0-2-released/ RPM, Debian/Ubuntu packages will be available soon as well. Many thanks to all contributing and using Kamailio! Cheers, Daniel -- Daniel-Constantin Mierla (@ asipto.com) twitter.com/miconda -- linkedin.com/in/miconda Kamailio Consultancy, Training and Development Services -- asipto.com

1 0

git:6.0:d4dc5dc1: ChangeLog: content updated for v6.0.2
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: 6.0 Commit: d4dc5dc14c62c7a016a3840d46f00064f926506b URL: https://github.com/kamailio/kamailio/commit/d4dc5dc14c62c7a016a3840d46f0006… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T10:35:30+02:00 ChangeLog: content updated for v6.0.2 --- Modified: ChangeLog --- Diff: https://github.com/kamailio/kamailio/commit/d4dc5dc14c62c7a016a3840d46f0006… Patch: https://github.com/kamailio/kamailio/commit/d4dc5dc14c62c7a016a3840d46f0006…

1 0

SIP retransmits over a TCP connection between two kamailios. TCP questions
by Stefan Mititelu 25 Jun '25

25 Jun '25

Hi, Recently experienced issues with 1 TCP connection between 2 kamailio servers: 1. KAM1 sends 2x forked INVITEs to KAM2 2. KAM2 starts config route processing for INVITE1. But blocks for ~1s due to rtpengine module pinging some inactive IPs 3. KAM1 re-transmits forked INVITE2 Worth mentioning that: 1. KAM2 uses same TCP connection for receiving KDMQs too. During that period, noticed KDMQ default_callback error triggered, due to timeout. So clearly, no KDMQs were processed anymore, during that time. 2. No errors related to TCP connection logged 3. kamailio version 5.8, tcp_reuse_port=yes, and don't set any route_locks_size, used 4 socket workers for that specific TCP connection Looked for quite a while in tcp_main.c and tcp_read.c trying to figure out what is happening with TCP connection(s) in general, and come to the following conclusion: TCP connection structure is held by the TCP socket worker process until the SIP request is completely received in the buffer, parsed *and* processed routing config for it. Afterwards TCP socket worker releases the TCP connection structure by signalling this back to the TCP_MAIN process. Thus other TCP socket worker would be able to handle *next* SIP request, for *the same* TCP connection. ...but while one TCP socket worker executes config route, no other TCP socket workers will be able to handle *next* SIP request, for *the same* TCP connection. My questions are: 1. Is the above conclusion correct? => this explains the above issue, and want to double check I understood the core tcp code correctly 2. Can async socket workers solve this? Thank you, Stefan

3 3

git:6.0:ab318d26: Makefile.defs: version set to 6.0.2
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: 6.0 Commit: ab318d268404799950cb992aa4498756f88bbed3 URL: https://github.com/kamailio/kamailio/commit/ab318d268404799950cb992aa449875… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T08:56:30+02:00 Makefile.defs: version set to 6.0.2 --- Modified: src/Makefile.defs --- Diff: https://github.com/kamailio/kamailio/commit/ab318d268404799950cb992aa449875… Patch: https://github.com/kamailio/kamailio/commit/ab318d268404799950cb992aa449875… --- diff --git a/src/Makefile.defs b/src/Makefile.defs index 7d911276323..855f83cc175 100644 --- a/src/Makefile.defs +++ b/src/Makefile.defs @@ -106,7 +106,7 @@ INSTALL_FLAVOUR=$(FLAVOUR) # version number VERSION = 6 PATCHLEVEL = 0 -SUBLEVEL = 1 +SUBLEVEL = 2 EXTRAVERSION = # memory manager switcher

1 0

git:6.0:16e04c66: CMakeLists.txt: version set to 6.0.2
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: 6.0 Commit: 16e04c66058386012cb975fa8e73db4c82152063 URL: https://github.com/kamailio/kamailio/commit/16e04c66058386012cb975fa8e73db4… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T08:55:15+02:00 CMakeLists.txt: version set to 6.0.2 --- Modified: CMakeLists.txt --- Diff: https://github.com/kamailio/kamailio/commit/16e04c66058386012cb975fa8e73db4… Patch: https://github.com/kamailio/kamailio/commit/16e04c66058386012cb975fa8e73db4… --- diff --git a/CMakeLists.txt b/CMakeLists.txt index 67fdbf351cc..dc4a5287f8e 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -5,7 +5,7 @@ cmake_minimum_required(VERSION 3.10) # Set the project name project( kamailio - VERSION 6.0.0 + VERSION 6.0.2 DESCRIPTION "Kamailio SIP Server" HOMEPAGE_URL "https://www.kamailio.org")

1 0

git:6.0:6c9e5413: pkg/kamailio/deb: set version to 6.0.2
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: 6.0 Commit: 6c9e5413656a7f375f42fdf5fc9888398d1d2524 URL: https://github.com/kamailio/kamailio/commit/6c9e5413656a7f375f42fdf5fc98883… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T08:51:38+02:00 pkg/kamailio/deb: set version to 6.0.2 --- Modified: pkg/kamailio/deb/bionic/changelog Modified: pkg/kamailio/deb/bookworm/changelog Modified: pkg/kamailio/deb/bullseye/changelog Modified: pkg/kamailio/deb/buster/changelog Modified: pkg/kamailio/deb/debian/changelog Modified: pkg/kamailio/deb/focal/changelog Modified: pkg/kamailio/deb/jammy/changelog Modified: pkg/kamailio/deb/jessie/changelog Modified: pkg/kamailio/deb/noble/changelog Modified: pkg/kamailio/deb/precise/changelog Modified: pkg/kamailio/deb/sid/changelog Modified: pkg/kamailio/deb/stretch/changelog Modified: pkg/kamailio/deb/trusty/changelog Modified: pkg/kamailio/deb/wheezy/changelog Modified: pkg/kamailio/deb/xenial/changelog --- Diff: https://github.com/kamailio/kamailio/commit/6c9e5413656a7f375f42fdf5fc98883… Patch: https://github.com/kamailio/kamailio/commit/6c9e5413656a7f375f42fdf5fc98883…

1 0

git:6.0:cadf2e22: pkg/kamailio: version set 6.0.2 for rpms and alpine
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: 6.0 Commit: cadf2e2248177b4e38a9f7611ddab4be455b17e1 URL: https://github.com/kamailio/kamailio/commit/cadf2e2248177b4e38a9f7611ddab4b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T08:48:31+02:00 pkg/kamailio: version set 6.0.2 for rpms and alpine --- Modified: pkg/kamailio/alpine/APKBUILD Modified: pkg/kamailio/obs/kamailio.spec --- Diff: https://github.com/kamailio/kamailio/commit/cadf2e2248177b4e38a9f7611ddab4b… Patch: https://github.com/kamailio/kamailio/commit/cadf2e2248177b4e38a9f7611ddab4b… --- diff --git a/pkg/kamailio/alpine/APKBUILD b/pkg/kamailio/alpine/APKBUILD index 2215305313f..4754aa04fb2 100644 --- a/pkg/kamailio/alpine/APKBUILD +++ b/pkg/kamailio/alpine/APKBUILD @@ -4,7 +4,7 @@ # Maintainer: Nathan Angelacos <nangel(a)alpinelinux.org> pkgname=kamailio -pkgver=6.0.1 +pkgver=6.0.2 pkgrel=0 # If building from a git snapshot, specify the gitcommit diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec index 68b7c46b90f..e8f28a267f8 100644 --- a/pkg/kamailio/obs/kamailio.spec +++ b/pkg/kamailio/obs/kamailio.spec @@ -1,5 +1,5 @@ %define name kamailio -%define ver 6.0.1 +%define ver 6.0.2 %define rel dev1.0%{dist} %if 0%{?fedora}

1 0

git:6.0:c522cff3: tls: fix setting cipher suites for tls verions prior to 1.3
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: 6.0 Commit: c522cff31e01abc822ae488a830f08b292f69829 URL: https://github.com/kamailio/kamailio/commit/c522cff31e01abc822ae488a830f08b… Author: Ovidiu Sas <osas(a)voipembedded.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T08:09:10+02:00 tls: fix setting cipher suites for tls verions prior to 1.3 (cherry picked from commit 84a29ac892d1097a8d08a4853fd2749ee039405c) --- Modified: src/modules/tls/tls_domain.c --- Diff: https://github.com/kamailio/kamailio/commit/c522cff31e01abc822ae488a830f08b… Patch: https://github.com/kamailio/kamailio/commit/c522cff31e01abc822ae488a830f08b… --- diff --git a/src/modules/tls/tls_domain.c b/src/modules/tls/tls_domain.c index c7f35df63ee..afc05e2f1c7 100644 --- a/src/modules/tls/tls_domain.c +++ b/src/modules/tls/tls_domain.c @@ -749,12 +749,12 @@ static int set_cipher_list(tls_domain_t *d) ERR("%s: Failure to set SSL context cipher suites \"%s\"\n", tls_domain_str(d), cipher_list); return -1; - } else { - if(SSL_CTX_set_cipher_list(d->ctx[i], cipher_list) == 0) { - ERR("%s: Failure to set SSL context cipher list \"%s\"\n", - tls_domain_str(d), cipher_list); - return -1; - } + } + } else { + if(SSL_CTX_set_cipher_list(d->ctx[i], cipher_list) == 0) { + ERR("%s: Failure to set SSL context cipher list \"%s\"\n", + tls_domain_str(d), cipher_list); + return -1; } } #endif

1 0

git:6.0:0e820635: tls: use SSL_CTX_set_ciphersuites() when using openSSL 1.1.1
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Module: kamailio Branch: 6.0 Commit: 0e820635bbbf9d887e1d481f2dc0243603eb9624 URL: https://github.com/kamailio/kamailio/commit/0e820635bbbf9d887e1d481f2dc0243… Author: Nicolas Chapleau <nicchap(a)outlook.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-25T08:08:53+02:00 tls: use SSL_CTX_set_ciphersuites() when using openSSL 1.1.1 (cherry picked from commit 6faf0470d0fab1594249e43d91000f324c5e25a7) --- Modified: src/modules/tls/tls_domain.c --- Diff: https://github.com/kamailio/kamailio/commit/0e820635bbbf9d887e1d481f2dc0243… Patch: https://github.com/kamailio/kamailio/commit/0e820635bbbf9d887e1d481f2dc0243… --- diff --git a/src/modules/tls/tls_domain.c b/src/modules/tls/tls_domain.c index 6527b16a9fa..c7f35df63ee 100644 --- a/src/modules/tls/tls_domain.c +++ b/src/modules/tls/tls_domain.c @@ -737,11 +737,27 @@ static int set_cipher_list(tls_domain_t *d) return 0; procs_no = get_max_procs(); for(i = 0; i < procs_no; i++) { +#if OPENSSL_VERSION_NUMBER < 0x030000000L if(SSL_CTX_set_cipher_list(d->ctx[i], cipher_list) == 0) { ERR("%s: Failure to set SSL context cipher list \"%s\"\n", tls_domain_str(d), cipher_list); return -1; } +#else + if(d->method == TLS_USE_TLSv1_3 || d->method == TLS_USE_TLSv1_3_PLUS) { + if(SSL_CTX_set_ciphersuites(d->ctx[i], cipher_list) == 0) { + ERR("%s: Failure to set SSL context cipher suites \"%s\"\n", + tls_domain_str(d), cipher_list); + return -1; + } else { + if(SSL_CTX_set_cipher_list(d->ctx[i], cipher_list) == 0) { + ERR("%s: Failure to set SSL context cipher list \"%s\"\n", + tls_domain_str(d), cipher_list); + return -1; + } + } + } +#endif #if !defined(OPENSSL_NO_ECDH) && OPENSSL_VERSION_NUMBER < 0x10100000L setup_ecdh(d->ctx[i]); #endif

1 0

Releasing v6.0.2
by Daniel-Constantin Mierla 25 Jun '25

25 Jun '25

Hello, I am considering to release Kamailio v6.0.2 (out of branch 6.0) on Wednesday, June 25, 2025. If anyone is aware of issues not yet on the bug tracker, report them there asap in order to have a better chance to be fixed. Cheers, Daniel -- Daniel-Constantin Mierla (@ asipto.com) twitter.com/miconda -- linkedin.com/in/miconda Kamailio Consultancy, Training and Development Services -- asipto.com

2 2

git:master:d79bc9b9: core: dns - macro wrappers to print more details on dns hash put
by Daniel-Constantin Mierla 24 Jun '25

24 Jun '25

Module: kamailio Branch: master Commit: d79bc9b9b85f247d8b4bca37d2f84d4dcb412730 URL: https://github.com/kamailio/kamailio/commit/d79bc9b9b85f247d8b4bca37d2f84d4… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-24T20:40:34+02:00 core: dns - macro wrappers to print more details on dns hash put --- Modified: src/core/dns_cache.c Modified: src/core/dns_cache.h --- Diff: https://github.com/kamailio/kamailio/commit/d79bc9b9b85f247d8b4bca37d2f84d4… Patch: https://github.com/kamailio/kamailio/commit/d79bc9b9b85f247d8b4bca37d2f84d4… --- diff --git a/src/core/dns_cache.c b/src/core/dns_cache.c index 0a2553f58f2..c6dc68a7930 100644 --- a/src/core/dns_cache.c +++ b/src/core/dns_cache.c @@ -162,14 +162,15 @@ inline static void dns_destroy_entry_shm_unsafe(struct dns_hash_entry *e) /* dec. the internal refcnt and if 0 deletes the entry */ -void dns_hash_put(struct dns_hash_entry *e) +void dns_hash_put_entry( + struct dns_hash_entry *e, const char *fpath, unsigned int line) { if(e != NULL) { if(atomic_dec_and_test(&e->refcnt)) { /* atomic_sub_long(dns_cache_total_used, e->total_size); */ dns_destroy_entry(e); } else if(e->next == NULL && e->prev == NULL) { - LM_WARN("unlinked item %p\n", e); + LM_WARN("unlinked item %p (%s:%u)\n", e, fpath, line); } } } @@ -177,14 +178,15 @@ void dns_hash_put(struct dns_hash_entry *e) /* same as above but uses dns_destroy_unsafe (assumes shm_lock held -- tm * optimization) */ -void dns_hash_put_shm_unsafe(struct dns_hash_entry *e) +void dns_hash_put_entry_shm_unsafe( + struct dns_hash_entry *e, const char *fpath, unsigned int line) { if(e != NULL) { if(atomic_dec_and_test(&e->refcnt)) { /* atomic_sub_long(dns_cache_total_used, e->total_size); */ dns_destroy_entry_shm_unsafe(e); } else if(e->next == NULL && e->prev == NULL) { - LM_WARN("unlinked item %p\n", e); + LM_WARN("unlinked item %p (%s:%u)\n", e, fpath, line); } } } diff --git a/src/core/dns_cache.h b/src/core/dns_cache.h index d92fd3dc27f..18bb3733730 100644 --- a/src/core/dns_cache.h +++ b/src/core/dns_cache.h @@ -193,24 +193,31 @@ int init_dns_cache_stats(int iproc_num); #endif void destroy_dns_cache(void); +void dns_hash_put_entry( + struct dns_hash_entry *e, const char *fpath, unsigned int line); +void dns_hash_put_entry_shm_unsafe( + struct dns_hash_entry *e, const char *fpath, unsigned int line); -void dns_hash_put(struct dns_hash_entry *e); -void dns_hash_put_shm_unsafe(struct dns_hash_entry *e); +#define dns_hash_put(e) dns_hash_put_entry(e, __FILE__, __LINE__) +#define dns_hash_put_shm_unsafe(e) \ + dns_hash_put_entry_shm_unsafe(e, __FILE__, __LINE__) -inline static void dns_srv_handle_put(struct dns_srv_handle *h) +inline static void dns_srv_handle_put_helper( + struct dns_srv_handle *h, const char *fpath, unsigned int line) { if(h) { if(h->srv) { - dns_hash_put(h->srv); + dns_hash_put_entry(h->srv, fpath, line); h->srv = 0; } if(h->a) { - dns_hash_put(h->a); + dns_hash_put_entry(h->a, fpath, line); h->a = 0; } } } +#define dns_srv_handle_put(h) dns_srv_handle_put_helper(h, __FILE__, __LINE__) /** @brief use it when copying, it manually increases the ref cound */ inline static void dns_srv_handle_ref(struct dns_srv_handle *h)

1 0

git:master:deb8405b: dispatcher: renamed field to reflect better the purpose and print in rpc result
by Daniel-Constantin Mierla 24 Jun '25

24 Jun '25

Module: kamailio Branch: master Commit: deb8405b403a5a8da4f2aac3aa167e2a84d15d91 URL: https://github.com/kamailio/kamailio/commit/deb8405b403a5a8da4f2aac3aa167e2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-24T19:18:52+02:00 dispatcher: renamed field to reflect better the purpose and print in rpc result --- 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/deb8405b403a5a8da4f2aac3aa167e2… Patch: https://github.com/kamailio/kamailio/commit/deb8405b403a5a8da4f2aac3aa167e2…

1 0

git:6.0:3c19d508: secsipid: set global url pointer to null after free
by Daniel-Constantin Mierla 24 Jun '25

24 Jun '25

Module: kamailio Branch: 6.0 Commit: 3c19d508ea84d8541b9eca0fc06c0c62309b34cc URL: https://github.com/kamailio/kamailio/commit/3c19d508ea84d8541b9eca0fc06c0c6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-24T18:54:26+02:00 secsipid: set global url pointer to null after free - GH #4284 (cherry picked from commit 15a976808f5e785a6d234734b0a7de0d735d4a5a) --- Modified: src/modules/secsipid/secsipid_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/3c19d508ea84d8541b9eca0fc06c0c6… Patch: https://github.com/kamailio/kamailio/commit/3c19d508ea84d8541b9eca0fc06c0c6… --- diff --git a/src/modules/secsipid/secsipid_mod.c b/src/modules/secsipid/secsipid_mod.c index ebe2e2e50ef..eecf18c77bd 100644 --- a/src/modules/secsipid/secsipid_mod.c +++ b/src/modules/secsipid/secsipid_mod.c @@ -916,6 +916,7 @@ static sr_kemi_xval_t *ki_secsipid_get_url(sip_msg_t *msg, str *surl) if(_secsipid_get_url_val.s != NULL) { free(_secsipid_get_url_val.s); + _secsipid_get_url_val.s = NULL; _secsipid_get_url_val.len = 0; } @@ -959,6 +960,7 @@ static int w_secsipid_get_url(sip_msg_t *msg, char *purl, char *povar) } if(_secsipid_get_url_val.s != NULL) { free(_secsipid_get_url_val.s); + _secsipid_get_url_val.s = NULL; _secsipid_get_url_val.len = 0; }

1 0

git:6.0:6de03f80: websocket: set the field for event route execution on timer removing
by Daniel-Constantin Mierla 24 Jun '25

24 Jun '25

Module: kamailio Branch: 6.0 Commit: 6de03f80bd266ab8b2ddf018b68f3501bd0094b0 URL: https://github.com/kamailio/kamailio/commit/6de03f80bd266ab8b2ddf018b68f350… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2025-06-24T18:54:06+02:00 websocket: set the field for event route execution on timer removing - GH #3950 (cherry picked from commit e77ba1cdf80b384bc55cc457d280b5fc0aef80db) --- Modified: src/modules/websocket/ws_conn.c --- Diff: https://github.com/kamailio/kamailio/commit/6de03f80bd266ab8b2ddf018b68f350… Patch: https://github.com/kamailio/kamailio/commit/6de03f80bd266ab8b2ddf018b68f350… --- diff --git a/src/modules/websocket/ws_conn.c b/src/modules/websocket/ws_conn.c index 18e0e4667bd..577abbe582a 100644 --- a/src/modules/websocket/ws_conn.c +++ b/src/modules/websocket/ws_conn.c @@ -708,6 +708,7 @@ void ws_timer(unsigned int ticks, void *param) LM_DBG("ws structure without active tcp connection\n"); wsc->state = WS_S_REMOVING; wsc->rmticks = get_ticks(); + wsc->run_event = 1; } else { tcpconn_put(con); }

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev