- sr-dev - kamailio.org

git:5.7:bbbcb270: tls: remove thread-enablement on EVP_RAND_CTX
by S-P Chan 14 Jan '24

14 Jan '24

Module: kamailio Branch: 5.7 Commit: bbbcb27040c632642c50209455efbb8225888723 URL: https://github.com/kamailio/kamailio/commit/bbbcb27040c632642c50209455efbb8… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-01-14T14:44:47+08:00 tls: remove thread-enablement on EVP_RAND_CTX - with late initialisation it is not necessary to enable thread locking on EVP_RAND_CTX - the function remains but is not used in case requirements change with OpenSSL >= 3.2 (cherry-pick from 8dffc45ee91aeed839efb38d17040359dcac953a) --- Modified: src/modules/tls/tls_init.c --- Diff: https://github.com/kamailio/kamailio/commit/bbbcb27040c632642c50209455efbb8… Patch: https://github.com/kamailio/kamailio/commit/bbbcb27040c632642c50209455efbb8… --- diff --git a/src/modules/tls/tls_init.c b/src/modules/tls/tls_init.c index 65f5ae72c65..629f69ad6b3 100644 --- a/src/modules/tls/tls_init.c +++ b/src/modules/tls/tls_init.c @@ -731,6 +731,12 @@ int tls_pre_init(void) * - executed before any mod_init() */ #if OPENSSL_VERSION_NUMBER >= 0x030000000L +/* + * With late initialisation it is not necessary to + * enable threading on the EVP_RAND_CTX. This function + * left here in case more complex requirements arise in + * OpenSSL >= 3.2. + */ long tls_h_mod_randctx(void *) { do { OSSL_LIB_CTX *osslglobal = NULL; @@ -768,7 +774,7 @@ long tls_h_mod_randctx(void *) { return 0L; } -#endif +#endif /* OPENSSL_VERSION_NUMBER */ int tls_h_mod_pre_init_f(void) { @@ -794,14 +800,21 @@ int tls_h_mod_pre_init_f(void) SSL_load_error_strings(); #endif +#if 0 #if OPENSSL_VERSION_NUMBER >= 0x030000000L + /* + * With deferred initialisation it is not necessary to enable threading + * on the EVP_RAND_CTX. We leave this block here as an example of how + * to do it in case of future requirements. + */ pthread_t tid; long rl; pthread_create(&tid, NULL, (void *(*)(void *))tls_h_mod_randctx, NULL); pthread_join(tid, (void **)&rl); if ((int)rl) return (int)rl; -#endif +#endif /* OPENSSL_VERSION_NUMBER */ +#endif /* 0 */ tls_mod_preinitialized = 1; return 0;

1 0

git:master:8dffc45e: tls: remove thread-enablement on EVP_RAND_CTX
by S-P Chan 14 Jan '24

14 Jan '24

Module: kamailio Branch: master Commit: 8dffc45ee91aeed839efb38d17040359dcac953a URL: https://github.com/kamailio/kamailio/commit/8dffc45ee91aeed839efb38d1704035… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-01-14T14:08:03+08:00 tls: remove thread-enablement on EVP_RAND_CTX - with late initialisation it is not necessary to enable thread locking on EVP_RAND_CTX - the function remains but is not used in case requirements change with OpenSSL >= 3.2 --- Modified: src/modules/tls/tls_init.c --- Diff: https://github.com/kamailio/kamailio/commit/8dffc45ee91aeed839efb38d1704035… Patch: https://github.com/kamailio/kamailio/commit/8dffc45ee91aeed839efb38d1704035… --- diff --git a/src/modules/tls/tls_init.c b/src/modules/tls/tls_init.c index d077b41e08a..bb1d0f19207 100644 --- a/src/modules/tls/tls_init.c +++ b/src/modules/tls/tls_init.c @@ -733,6 +733,12 @@ int tls_pre_init(void) * - executed before any mod_init() */ #if OPENSSL_VERSION_NUMBER >= 0x030000000L +/* + * With late initialisation it is not necessary to + * enable threading on the EVP_RAND_CTX. This function + * left here in case more complex requirements arise in + * OpenSSL >= 3.2. + */ long tls_h_mod_randctx(void *) { do { OSSL_LIB_CTX *osslglobal = NULL; @@ -770,7 +776,7 @@ long tls_h_mod_randctx(void *) { return 0L; } -#endif +#endif /* OPENSSL_VERSION_NUMBER */ int tls_h_mod_pre_init_f(void) { @@ -796,14 +802,21 @@ int tls_h_mod_pre_init_f(void) SSL_load_error_strings(); #endif +#if 0 #if OPENSSL_VERSION_NUMBER >= 0x030000000L + /* + * With deferred initialisation it is not necessary to enable threading + * on the EVP_RAND_CTX. We leave this block here as an example of how + * to do it in case of future requirements. + */ pthread_t tid; long rl; pthread_create(&tid, NULL, (void *(*)(void *))tls_h_mod_randctx, NULL); pthread_join(tid, (void **)&rl); if ((int)rl) return (int)rl; -#endif +#endif /* OPENSSL_VERSION_NUMBER */ +#endif /* 0 */ tls_mod_preinitialized = 1; return 0;

1 0

git:master:42b9fe97: src/Makefile.defs: recognize GCC 14 as recent â not too old compiler
by Victor Seva 13 Jan '24

13 Jan '24

Module: kamailio Branch: master Commit: 42b9fe9781e6805436227e0e7b88db73767a94b1 URL: https://github.com/kamailio/kamailio/commit/42b9fe9781e6805436227e0e7b88db7… Author: �� <git-dpa(a)aegee.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-01-13T18:51:06+01:00 src/Makefile.defs: recognize GCC 14 as recent �� not too old compiler similar to 87ed3d9c3ae3ecfabe6 --- Modified: src/Makefile.defs --- Diff: https://github.com/kamailio/kamailio/commit/42b9fe9781e6805436227e0e7b88db7… Patch: https://github.com/kamailio/kamailio/commit/42b9fe9781e6805436227e0e7b88db7… --- diff --git a/src/Makefile.defs b/src/Makefile.defs index 68225943982..7a4b7418670 100644 --- a/src/Makefile.defs +++ b/src/Makefile.defs @@ -307,8 +307,8 @@ ifneq (,$(findstring gcc, $(CC_LONGVER))) -e 's/8\.[0-9]$$/8.0+/' \ -e 's/9\.[0-9]\..*/9.0+/' \ -e 's/9\.[0-9]$$/9.0+/' \ - -e 's/1[0-3]\.[0-9]\..*/9.0+/' \ - -e 's/1[0-3]\.[0-9]$$/9.0+/') + -e 's/1[0-4]\.[0-9]\..*/9.0+/' \ + -e 's/1[0-4]\.[0-9]$$/9.0+/') ifeq (,$(strip $(filter-out 3.0 3.4 4.x 4.2+ 4.5+ 5.0+ 6.0+ 7.0+ 8.0+ 9.0+,$(CC_SHORTVER)))) # dependencies can be generated on-the-fly while compiling *.c CC_MKDEP_OPTS=-MMD -MP

1 0

[kamailio/kamailio] src/Makefile.defs: recognize GCC 14 as recent ⇔ not too old compiler (PR #3711)
by Дилян Палаузов 13 Jan '24

13 Jan '24

similar to 87ed3d9c3ae3ecfabe6 GCC 14 will be released in April or May 2024. Adding this now will avoid backporting the change to the 5.8 branch. (Not that 87ed3d9c3ae3ecfabe6 was backported to 5.7.) You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3711 -- Commit Summary -- * src/Makefile.defs: recognize GCC 14 as recent ⇔ not too old compiler -- File Changes -- M src/Makefile.defs (4) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3711.patch https://github.com/kamailio/kamailio/pull/3711.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3711 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3711(a)github.com>

2 1

[kamailio/kamailio] Using PATH makes nathelper to send the SIP ping to itself (#2011)
by pwakano 13 Jan '24

13 Jan '24

### Description When using the PATH header option to keep track of original Kamailio node that received the REGISTER, if nathelper is also used with SIP ping enabled, the SIP OPTIONS is sent to itself following the PATH header and not sent directly to the destination. The nathelper/user location modules should have a flag similar to the "path_check_local" of the registrar module, which is meant to avoid looping the message to itself in case the next hop pointed by the PATH header is "myself".  ### Troubleshooting #### Reproduction Configure the registrar module to support PATH and before saving the contact info add the PATH header with the local Kamailio information. Also make sure to have the nathelper and usrloc modules configured to send the keepalive SIP OPTIONS to the registered extensions. By the time the SIP OPTIONS is sent, Kamailio sends to itself, then the message has to be loose_route'd to the final destination. This could be avoided by having the nathelper module to identify the next hop is Kamailio itself skipping this destination as a call to "lookup()" does for the registrar module when the flag path_check_local is true.  #### SIP Traffic  172.128.128.10 is the Kamailio IP and 172.128.128.20 is the extension IP ``` 11:51:07.224713 IP (tos 0x60, ttl 64, id 64497, offset 0, flags [none], proto UDP (17), length 518) 172.128.128.10.5060 > 172.128.128.10.5060: SIP, length: 490 OPTIONS sip:345671002@172.128.128.20:5060;rinstance=d1eb3444a5cec5a1;transport=UDP SIP/2.0 Via: SIP/2.0/UDP 172.128.128.10:5060;branch=z9hG4bK2540713 Route: <sip:172.128.128.10:5060;received=sip:172.128.128.20:5060;lr;cluster_node=192.168.156.10:5060> From: sip:sbc@mydomain.com;tag=uloc-5d2e6499-2fb5-1-0-c15309d2 To: sip:345671002@172.128.128.20:5060;rinstance=d1eb3444a5cec5a1;transport=UDP Call-ID: 40ee1573-7506bba3-5e30c05(a)172.128.128.10 CSeq: 1 OPTIONS Content-Length: 0 11:51:07.226132 IP (tos 0x60, ttl 64, id 44359, offset 0, flags [none], proto UDP (17), length 502) 172.128.128.10.5060 > 172.128.128.20.5060: SIP, length: 474 OPTIONS sip:345671002@172.128.128.20:5060;rinstance=d1eb3444a5cec5a1;transport=UDP SIP/2.0 Via: SIP/2.0/UDP 172.128.128.10;branch=z9hG4bKb958.b807e80b742db62504a45220d8f0e974.0 Via: SIP/2.0/UDP 172.128.128.10:5060;branch=z9hG4bK2540713 From: sip:sbc@mydomain.com;tag=uloc-5d2e6499-2fb5-1-0-c15309d2 To: sip:345671002@172.128.128.20:5060;rinstance=d1eb3444a5cec5a1;transport=UDP Call-ID: 40ee1573-7506bba3-5e30c05(a)172.128.128.10 CSeq: 1 OPTIONS Content-Length: 0 11:51:07.303474 IP (tos 0x0, ttl 128, id 10816, offset 0, flags [none], proto UDP (17), length 786) 172.128.128.20.5060 > 172.128.128.10.5060: SIP, length: 758 SIP/2.0 200 OK Via: SIP/2.0/UDP 172.128.128.10;branch=z9hG4bKb958.b807e80b742db62504a45220d8f0e974.0 Via: SIP/2.0/UDP 172.128.128.10:5060;branch=z9hG4bK2540713 Contact: <sip:172.128.128.20:5060> To: <sip:345671002@172.128.128.20:5060;rinstance=d1eb3444a5cec5a1;transport=UDP>;tag=bb0c1f2d From: sip:sbc@mydomain.com;tag=uloc-5d2e6499-2fb5-1-0-c15309d2 Call-ID: 40ee1573-7506bba3-5e30c05(a)172.128.128.10 CSeq: 1 OPTIONS Accept: application/sdp, application/sdp Accept-Language: en Allow: INVITE, ACK, CANCEL, BYE, NOTIFY, REFER, MESSAGE, OPTIONS, INFO, SUBSCRIBE Supported: replaces, norefersub, extended-refer, timer, outbound, path, X-cisco-serviceuri User-Agent: Z 3.15.40006 rv2.8.20 Allow-Events: presence, kpml, talk Content-Length: 0 11:51:07.306821 IP (tos 0x60, ttl 64, id 64570, offset 0, flags [none], proto UDP (17), length 727) 172.128.128.10.5060 > 172.128.128.10.5060: SIP, length: 699 SIP/2.0 200 OK Via: SIP/2.0/UDP 172.128.128.10:5060;branch=z9hG4bK2540713 Contact: <sip:172.128.128.20:5060;alias=172.128.128.20~5060~1> To: <sip:345671002@172.128.128.20:5060;rinstance=d1eb3444a5cec5a1;transport=UDP>;tag=bb0c1f2d From: sip:sbc@mydomain.com;tag=uloc-5d2e6499-2fb5-1-0-c15309d2 Call-ID: 40ee1573-7506bba3-5e30c05(a)172.128.128.10 CSeq: 1 OPTIONS Accept: application/sdp, application/sdp Accept-Language: en Allow: INVITE, ACK, CANCEL, BYE, NOTIFY, REFER, MESSAGE, OPTIONS, INFO, SUBSCRIBE Supported: replaces, norefersub, extended-refer, timer, outbound, path, X-cisco-serviceuri User-Agent: Z 3.15.40006 rv2.8.20 Allow-Events: presence, kpml, talk Content-Length: 0 ``` ### Possible Solutions  Not found. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` # kamailio -v version: kamailio 5.2.2 (x86_64/linux) 67f967 flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLACKLIST, HAVE_RESOLV_RES 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: 67f967 compiled on 11:40:41 Mar 11 2019 with gcc 4.8.5 ``` * **Operating System**:  ``` # uname -a Linux kamailio-1 3.10.0-514.21.2.el7.x86_64 #1 SMP Tue Jun 20 12:24:47 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux ``` -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/2011

4 7

[kamailio/kamailio] unistr.h: No such file or directory (#2020)
by stefan.wen 12 Jan '24

12 Jan '24

### Description when i make websocket module: ws_frame.c:32:20: fatal error: unistr.h: No such file or directory #include <unistr.h> ^ compilation terminated. make: *** [ws_frame.o] Error 1 OS: centos7 and i try reinstall libunistring,Still failed -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/2020

4 4

git:master:1c13e99e: topoh: prefix global variable
by Daniel-Constantin Mierla 12 Jan '24

12 Jan '24

Module: kamailio Branch: master Commit: 1c13e99efb119f75ddbd6a0aa4798cd27accbff5 URL: https://github.com/kamailio/kamailio/commit/1c13e99efb119f75ddbd6a0aa4798cd… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-12T20:02:14+01:00 topoh: prefix global variable --- Modified: src/modules/topoh/topoh_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/1c13e99efb119f75ddbd6a0aa4798cd… Patch: https://github.com/kamailio/kamailio/commit/1c13e99efb119f75ddbd6a0aa4798cd… --- diff --git a/src/modules/topoh/topoh_mod.c b/src/modules/topoh/topoh_mod.c index 02098b1c1b0..fe63bf88c7f 100644 --- a/src/modules/topoh/topoh_mod.c +++ b/src/modules/topoh/topoh_mod.c @@ -95,7 +95,7 @@ int _th_use_mode = 0; struct str_hash_table *th_socket_hash_table; -sanity_api_t scb; +static sanity_api_t _tph_scb; int th_msg_received(sr_event_param_t *evp); int th_msg_sent(sr_event_param_t *evp); @@ -186,7 +186,7 @@ static int mod_init(void) } if(th_sanity_checks != 0) { - if(sanity_load_api(&scb) < 0) { + if(sanity_load_api(&_tph_scb) < 0) { LM_ERR("cannot bind to sanity module\n"); goto error; } @@ -515,7 +515,7 @@ int th_msg_received(sr_event_param_t *evp) th_cookie_value.len = 2; if(msg.first_line.type == SIP_REQUEST) { if(th_sanity_checks != 0) { - if(scb.check_defaults(&msg) < 1) { + if(_tph_scb.check_defaults(&msg) < 1) { LM_ERR("sanity checks failed\n"); goto done; }

1 0

git:master:cb92b6a2: topos: prefix global variable
by Daniel-Constantin Mierla 12 Jan '24

12 Jan '24

Module: kamailio Branch: master Commit: cb92b6a23d622b06bcb18195b5c0525fcd273e7a URL: https://github.com/kamailio/kamailio/commit/cb92b6a23d622b06bcb18195b5c0525… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-12T19:53:57+01:00 topos: prefix global variable --- Modified: src/modules/topos/topos_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/cb92b6a23d622b06bcb18195b5c0525… Patch: https://github.com/kamailio/kamailio/commit/cb92b6a23d622b06bcb18195b5c0525… --- diff --git a/src/modules/topos/topos_mod.c b/src/modules/topos/topos_mod.c index 7bcb2fea3c9..e37ca847beb 100644 --- a/src/modules/topos/topos_mod.c +++ b/src/modules/topos/topos_mod.c @@ -134,7 +134,7 @@ str _tps_xavu_field_contact_host = STR_NULL; str _tps_context_param = STR_NULL; str _tps_context_value = STR_NULL; -sanity_api_t scb; +static sanity_api_t _tps_scb; int tps_msg_received(sr_event_param_t *evp); int tps_msg_sent(sr_event_param_t *evp); @@ -297,7 +297,7 @@ static int mod_init(void) } if(_tps_sanity_checks != 0) { - if(sanity_load_api(&scb) < 0) { + if(sanity_load_api(&_tps_scb) < 0) { LM_ERR("cannot bind to sanity module\n"); goto error; } @@ -539,7 +539,7 @@ int tps_msg_received(sr_event_param_t *evp) if(msg.first_line.type == SIP_REQUEST) { if(_tps_sanity_checks != 0) { - if(scb.check_defaults(&msg) < 1) { + if(_tps_scb.check_defaults(&msg) < 1) { LM_ERR("sanity checks failed\n"); goto done; }

1 0

git:master:987e21a9: xhttp: prefix for global variable
by Daniel-Constantin Mierla 12 Jan '24

12 Jan '24

Module: kamailio Branch: master Commit: 987e21a914edcebc9eafe21af19c8a476bf73b06 URL: https://github.com/kamailio/kamailio/commit/987e21a914edcebc9eafe21af19c8a4… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-12T19:50:28+01:00 xhttp: prefix for global variable --- Modified: src/modules/xhttp/xhttp_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/987e21a914edcebc9eafe21af19c8a4… Patch: https://github.com/kamailio/kamailio/commit/987e21a914edcebc9eafe21af19c8a4… --- diff --git a/src/modules/xhttp/xhttp_mod.c b/src/modules/xhttp/xhttp_mod.c index 0660d409af6..af27baffa41 100644 --- a/src/modules/xhttp/xhttp_mod.c +++ b/src/modules/xhttp/xhttp_mod.c @@ -65,7 +65,7 @@ static char *xhttp_url_skip = NULL; static regex_t xhttp_url_skip_regexp; /** SL API structure */ -sl_api_t slb; +static sl_api_t _xhttp_slb; static str xhttp_event_callback = STR_NULL; @@ -149,7 +149,7 @@ static int mod_init(void) } /* bind the SL API */ - if(sl_load_api(&slb) != 0) { + if(sl_load_api(&_xhttp_slb) != 0) { LM_ERR("cannot bind to SL API\n"); return -1; } @@ -424,7 +424,7 @@ static int xhttp_send_reply( LM_DBG("response with body: %.*s\n", body->len, body->s); } LM_DBG("sending out response: %d %.*s\n", code, reason->len, reason->s); - if(slb.sreply(msg, code, reason) < 0) { + if(_xhttp_slb.sreply(msg, code, reason) < 0) { LM_ERR("Error while sending reply\n"); return -1; }

1 0

git:master:0479c40f: mohqueue: rename local variables to prevent shadowing the global
by Daniel-Constantin Mierla 12 Jan '24

12 Jan '24

Module: kamailio Branch: master Commit: 0479c40fe6fd6765de92eea8f9e0bf04dea8cbd0 URL: https://github.com/kamailio/kamailio/commit/0479c40fe6fd6765de92eea8f9e0bf0… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-12T14:37:02+01:00 mohqueue: rename local variables to prevent shadowing the global --- Modified: src/modules/mohqueue/mohq_funcs.c --- Diff: https://github.com/kamailio/kamailio/commit/0479c40fe6fd6765de92eea8f9e0bf0… Patch: https://github.com/kamailio/kamailio/commit/0479c40fe6fd6765de92eea8f9e0bf0… --- diff --git a/src/modules/mohqueue/mohq_funcs.c b/src/modules/mohqueue/mohq_funcs.c index 75130381746..599e5eb8a1a 100644 --- a/src/modules/mohqueue/mohq_funcs.c +++ b/src/modules/mohqueue/mohq_funcs.c @@ -1278,17 +1278,17 @@ int form_rtp_SDP(str *pstr, call_lst *pcall, char *pSDP) **********/ char *pfncname = "form_rtp_SDP: "; - rtpmap **pmohfiles = + rtpmap **lpmohfiles = find_MOH(pcall->pmohq->mohq_mohdir, pcall->pmohq->mohq_mohfile); - if(!pmohfiles[0]) { + if(!lpmohfiles[0]) { LM_ERR("%sUnable to find any MOH files for queue (%s)!\n", pfncname, pcall->pmohq->mohq_name); return 0; } int nsize = strlen(pSDP) + 2; int nidx; - for(nidx = 0; pmohfiles[nidx]; nidx++) { - nsize += strlen(pmohfiles[nidx]->pencode) // encode length + for(nidx = 0; lpmohfiles[nidx]; nidx++) { + nsize += strlen(lpmohfiles[nidx]->pencode) // encode length + 19; // space, type number, "a=rtpmap:%d ", EOL } @@ -1707,7 +1707,7 @@ void reinvite_msg(sip_msg_t *pmsg, call_lst *pcall) * o look for hold condition and matching payload type **********/ - rtpmap **pmohfiles = + rtpmap **lpmohfiles = find_MOH(pcall->pmohq->mohq_mohdir, pcall->pmohq->mohq_mohfile); int bhold = 0; int bmatch = 0; @@ -1749,8 +1749,8 @@ void reinvite_msg(sip_msg_t *pmsg, call_lst *pcall) ppayload = ppayload->next) { int ntype = atoi(ppayload->rtp_payload.s); int nidx; - for(nidx = 0; pmohfiles[nidx]; nidx++) { - if(pmohfiles[nidx]->ntype == ntype) { + for(nidx = 0; lpmohfiles[nidx]; nidx++) { + if(lpmohfiles[nidx]->ntype == ntype) { bmatch = 1; break; }

1 0

git:master:0ea60341: avp: rename local variable to prevent shadowing the global structure
by Daniel-Constantin Mierla 12 Jan '24

12 Jan '24

Module: kamailio Branch: master Commit: 0ea603413de9505b5312e5930168102e816dced3 URL: https://github.com/kamailio/kamailio/commit/0ea603413de9505b5312e5930168102… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-12T13:10:33+01:00 avp: rename local variable to prevent shadowing the global structure --- Modified: src/modules/avp/avp.c --- Diff: https://github.com/kamailio/kamailio/commit/0ea603413de9505b5312e5930168102… Patch: https://github.com/kamailio/kamailio/commit/0ea603413de9505b5312e5930168102… --- diff --git a/src/modules/avp/avp.c b/src/modules/avp/avp.c index 38fa08fe1cb..d04fe47f880 100644 --- a/src/modules/avp/avp.c +++ b/src/modules/avp/avp.c @@ -1454,7 +1454,7 @@ static int attr_hdr_body2attrs2( static int attr_hdr_body2attrs_fixup(void **param, int param_no) { - char *c, *params; + char *c, *sparams; hdr_name_t *h = NULL; int n; str *s; @@ -1467,9 +1467,9 @@ static int attr_hdr_body2attrs_fixup(void **param, int param_no) c); return E_CFG; } else { - params = strchr(c, PARAM_DELIM); - if(params) - n = params - c; + sparams = strchr(c, PARAM_DELIM); + if(sparams) + n = sparams - c; else n = strlen(c); if(n == 0) { @@ -1486,10 +1486,10 @@ static int attr_hdr_body2attrs_fixup(void **param, int param_no) h->name.s.s = (char *)h + sizeof(hdr_name_t); memcpy(h->name.s.s, c, n + 1); } - if(params) { + if(sparams) { h->val_types = 0; - while(*params) { - switch(*params) { + while(*sparams) { + switch(*sparams) { case 'i': case 'I': h->val_types = VAL_TYPE_INT; @@ -1504,11 +1504,11 @@ static int attr_hdr_body2attrs_fixup(void **param, int param_no) LOG(L_ERR, "attr_hdr_body2attrs_fixup: bad field param " "modifier near '%s'\n", - params); + sparams); pkg_free(h); return E_CFG; } - params++; + sparams++; } if(!h->val_types) { LOG(L_ERR, "attr_hdr_body2attrs_fixup: no field param modifier "

1 0

[kamailio/kamailio] Rtpengine fixes (PR #3710)
by Xenofon Karamanos 11 Jan '24

11 Jan '24

#### 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) - [x] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  This PR add support for the `codec-accept` and `codec-consume` flags of rtpengine (supported since [mr9.2.1](https://github.com/sipwise/rtpengine/commit/336cb5b280079b0e7b2412…). You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3710 -- Commit Summary -- * rtpengine: Add support for codec-accept and codec-consume * rtpengine/docs: Add codec-consume docs -- File Changes -- M src/modules/rtpengine/doc/rtpengine_admin.xml (5) M src/modules/rtpengine/rtpengine.c (9) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3710.patch https://github.com/kamailio/kamailio/pull/3710.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3710 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3710(a)github.com>

4 7

git:master:1671c964: modules: readme files regenerated - rtpengine ... [skip ci]
by Kamailio Dev 11 Jan '24

11 Jan '24

Module: kamailio Branch: master Commit: 1671c96462f76d902b4c331c2f51e99cd2e33af1 URL: https://github.com/kamailio/kamailio/commit/1671c96462f76d902b4c331c2f51e99… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-01-11T17:17:12+01:00 modules: readme files regenerated - rtpengine ... [skip ci] --- Modified: src/modules/rtpengine/README --- Diff: https://github.com/kamailio/kamailio/commit/1671c96462f76d902b4c331c2f51e99… Patch: https://github.com/kamailio/kamailio/commit/1671c96462f76d902b4c331c2f51e99… --- diff --git a/src/modules/rtpengine/README b/src/modules/rtpengine/README index 96693dcd018..7cfc4772863 100644 --- a/src/modules/rtpengine/README +++ b/src/modules/rtpengine/README @@ -2419,6 +2419,9 @@ rtpengine_offer(); still be accepted towards the original offerer and then used for transcoding. It is a more selective version of what the `always transcode` flag does. + + codec-consume=... - Identical to mask but enables the + transcoding engine even if no other transcoding related + options are given. + T.38=decode - If the offered SDP contains a media section advertising T.38 over UDPTL, translate it to a regular audio media section over RTP. By default, PCMU and PCMA will be used

1 0

git:master:1aec0c13: rtpengine/docs: Add codec-consume docs
by Victor Seva 11 Jan '24

11 Jan '24

Module: kamailio Branch: master Commit: 1aec0c1397e4230e52f2b9427fdb32b10f649ecb URL: https://github.com/kamailio/kamailio/commit/1aec0c1397e4230e52f2b9427fdb32b… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-01-11T17:04:07+01:00 rtpengine/docs: Add codec-consume docs --- Modified: src/modules/rtpengine/doc/rtpengine_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/1aec0c1397e4230e52f2b9427fdb32b… Patch: https://github.com/kamailio/kamailio/commit/1aec0c1397e4230e52f2b9427fdb32b… --- diff --git a/src/modules/rtpengine/doc/rtpengine_admin.xml b/src/modules/rtpengine/doc/rtpengine_admin.xml index 1e394453493..fea67a665a2 100644 --- a/src/modules/rtpengine/doc/rtpengine_admin.xml +++ b/src/modules/rtpengine/doc/rtpengine_admin.xml @@ -2746,6 +2746,11 @@ rtpengine_offer(); version of what the `always transcode` flag does. </para></listitem> + <listitem><para> + <emphasis>codec-consume=...</emphasis> - Identical to mask but enables the transcoding engine even if no other transcoding + related options are given. + </para></listitem> + <listitem><para> <emphasis>T.38=decode</emphasis> - If the offered &sdp; contains a media section advertising T.38 over UDPTL, translate it to a regular audio media section

1 0

git:master:a40d5b7d: rtpengine: Add support for codec-accept and codec-consume
by Victor Seva 11 Jan '24

11 Jan '24

Module: kamailio Branch: master Commit: a40d5b7d65394606f2200a5932284b874e196063 URL: https://github.com/kamailio/kamailio/commit/a40d5b7d65394606f2200a5932284b8… Author: Xenofon Karamanos <xk(a)gilawa.com> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2024-01-11T17:04:07+01:00 rtpengine: Add support for codec-accept and codec-consume --- Modified: src/modules/rtpengine/rtpengine.c --- Diff: https://github.com/kamailio/kamailio/commit/a40d5b7d65394606f2200a5932284b8… Patch: https://github.com/kamailio/kamailio/commit/a40d5b7d65394606f2200a5932284b8… --- diff --git a/src/modules/rtpengine/rtpengine.c b/src/modules/rtpengine/rtpengine.c index 11745d6b863..e3e4c6e1d50 100644 --- a/src/modules/rtpengine/rtpengine.c +++ b/src/modules/rtpengine/rtpengine.c @@ -126,7 +126,8 @@ struct ng_flags_parse { int via, to, packetize, transport, directional; bencode_item_t *dict, *flags, *direction, *replace, *rtcp_mux, *sdes, *t38, *received_from, *codec, *codec_strip, *codec_offer, - *codec_transcode, *codec_mask, *codec_set, *codec_except; + *codec_transcode, *codec_mask, *codec_set, *codec_except, *codec_accept, + *codec_consume; str call_id, from_tag, to_tag; }; @@ -2735,6 +2736,12 @@ static int parse_flags(struct ng_flags_parse *ng_flags, struct sip_msg *msg, if(parse_codec_flag(ng_flags, &key, &val, "codec-except", NULL, "except", &ng_flags->codec_except)) goto next; + if(parse_codec_flag(ng_flags, &key, &val, "codec-accept", NULL, + "accept", &ng_flags->codec_accept)) + goto next; + if(parse_codec_flag(ng_flags, &key, &val, "codec-consume", NULL, + "consume", &ng_flags->codec_consume)) + goto next; /* check for specially handled items */ switch(key.len) {

1 0

[kamailio/kamailio] UAC module ：uac_reg_tm_callback(): got sip response 408 (Issue #3707)
by QWERTY 11 Jan '24

11 Jan '24

Register with remote，if there is no response to request，get in the logs an error message like: ERROR: uac [uac_reg.c:1001]: uac_reg_tm_callback(): got sip response 408 while registering How to get this timeout information in routing ? event_route[ ] ？ -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3707 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3707(a)github.com>

2 2

git:master:b678e66a: p_usrloc: rename short global variable
by Daniel-Constantin Mierla 11 Jan '24

11 Jan '24

Module: kamailio Branch: master Commit: b678e66a974f3f88f5ed05e57e5d898e4715fe0a URL: https://github.com/kamailio/kamailio/commit/b678e66a974f3f88f5ed05e57e5d898… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-11T15:34:33+01:00 p_usrloc: rename short global variable --- Modified: src/modules/p_usrloc/ul_db_handle.c --- Diff: https://github.com/kamailio/kamailio/commit/b678e66a974f3f88f5ed05e57e5d898… Patch: https://github.com/kamailio/kamailio/commit/b678e66a974f3f88f5ed05e57e5d898…

1 0

git:master:302231bf: ratelimit: rename variables to prevent global structure shadowing
by Daniel-Constantin Mierla 11 Jan '24

11 Jan '24

Module: kamailio Branch: master Commit: 302231bf2262ef73a6aad4a2f476df7c9a3c05f0 URL: https://github.com/kamailio/kamailio/commit/302231bf2262ef73a6aad4a2f476df7… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-11T15:23:55+01:00 ratelimit: rename variables to prevent global structure shadowing --- Modified: src/modules/ratelimit/ratelimit.c --- Diff: https://github.com/kamailio/kamailio/commit/302231bf2262ef73a6aad4a2f476df7… Patch: https://github.com/kamailio/kamailio/commit/302231bf2262ef73a6aad4a2f476df7… --- diff --git a/src/modules/ratelimit/ratelimit.c b/src/modules/ratelimit/ratelimit.c index 4e327460044..5cac92464db 100644 --- a/src/modules/ratelimit/ratelimit.c +++ b/src/modules/ratelimit/ratelimit.c @@ -924,7 +924,7 @@ static int init_params(void) * parses a "pipe_no:algorithm:bandwidth" line * \return 0 on success */ -static int parse_pipe_params(char *line, pipe_params_t *params) +static int parse_pipe_params(char *line, pipe_params_t *pparams) { regmatch_t m[4]; str algo_str; @@ -938,12 +938,12 @@ static int parse_pipe_params(char *line, pipe_params_t *params) LM_DBG("pipe: [%.*s|%.*s|%.*s]\n", RXLS(m, line, 1), RXLS(m, line, 2), RXLS(m, line, 3)); - params->no = atoi(RXS(m, line, 1)); - params->limit = atoi(RXS(m, line, 3)); + pparams->no = atoi(RXS(m, line, 1)); + pparams->limit = atoi(RXS(m, line, 3)); algo_str.s = RXS(m, line, 2); algo_str.len = RXL(m, line, 2); - if(str_map_str(algo_names, &algo_str, &params->algo)) + if(str_map_str(algo_names, &algo_str, &pparams->algo)) return -1; return 0; @@ -953,7 +953,7 @@ static int parse_pipe_params(char *line, pipe_params_t *params) * parses a "pipe_no:method" line * \return 0 on success */ -static int parse_queue_params(char *line, rl_queue_params_t *params) +static int parse_queue_params(char *line, rl_queue_params_t *qparams) { regmatch_t m[3]; int len; @@ -966,16 +966,16 @@ static int parse_queue_params(char *line, rl_queue_params_t *params) } LM_DBG("queue: [%.*s|%.*s]\n", RXLS(m, line, 1), RXLS(m, line, 2)); - params->pipe = atoi(RXS(m, line, 1)); + qparams->pipe = atoi(RXS(m, line, 1)); len = RXL(m, line, 2); - params->method.s = (char *)pkg_malloc(len + 1); - if(params->method.s == 0) { + qparams->method.s = (char *)pkg_malloc(len + 1); + if(qparams->method.s == 0) { LM_ERR("no memory left for method in params\n"); return -1; } - params->method.len = len; - memcpy(params->method.s, RXS(m, line, 2), len + 1); + qparams->method.len = len; + memcpy(qparams->method.s, RXS(m, line, 2), len + 1); return 0; } @@ -1017,19 +1017,19 @@ static int check_feedback_setpoints(int modparam) static int add_pipe_params(modparam_t type, void *val) { char *param_line = val; - pipe_params_t params; + pipe_params_t pparams; - if(parse_pipe_params(param_line, &params)) + if(parse_pipe_params(param_line, &pparams)) return -1; - if(params.no < 0 || params.no >= MAX_PIPES) { + if(pparams.no < 0 || pparams.no >= MAX_PIPES) { LM_ERR("pipe number %d not allowed (MAX_PIPES=%d, 0-based)\n", - params.no, MAX_PIPES); + pparams.no, MAX_PIPES); return -1; } - pipes[params.no].algo_mp = params.algo; - pipes[params.no].limit_mp = params.limit; + pipes[pparams.no].algo_mp = pparams.algo; + pipes[pparams.no].limit_mp = pparams.limit; return check_feedback_setpoints(1); } @@ -1037,24 +1037,24 @@ static int add_pipe_params(modparam_t type, void *val) static int add_queue_params(modparam_t type, void *val) { char *param_line = val; - rl_queue_params_t params; + rl_queue_params_t qparams; if(nqueues_mp >= MAX_QUEUES) { LM_ERR("MAX_QUEUES reached (%d)\n", MAX_QUEUES); return -1; } - if(parse_queue_params(param_line, &params)) + if(parse_queue_params(param_line, &qparams)) return -1; - if(params.pipe >= MAX_PIPES) { + if(qparams.pipe >= MAX_PIPES) { LM_ERR("pipe number %d not allowed (MAX_PIPES=%d, 0-based)\n", - params.pipe, MAX_PIPES); + qparams.pipe, MAX_PIPES); return -1; } - queues[nqueues_mp].pipe_mp = params.pipe; - queues[nqueues_mp].method_mp = params.method; + queues[nqueues_mp].pipe_mp = qparams.pipe; + queues[nqueues_mp].method_mp = qparams.method; nqueues_mp++; return 0;

1 0

[kamailio/kamailio] http_async_client: exit mod_init if tm is not loaded (PR #3706)
by Federico Cabiddu 11 Jan '24

11 Jan '24

#### 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  You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3706 -- Commit Summary -- * http_async_client: exit mod_init if tm is not loaded -- File Changes -- M src/modules/http_async_client/http_async_client_mod.c (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3706.patch https://github.com/kamailio/kamailio/pull/3706.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3706 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3706(a)github.com>

3 4

git:master:72595fe4: http_async_client: use error log level (GH #3706)
by Henning Westerholt 11 Jan '24

11 Jan '24

Module: kamailio Branch: master Commit: 72595fe42d28d3e4a12341b405af63d41f827dc4 URL: https://github.com/kamailio/kamailio/commit/72595fe42d28d3e4a12341b405af63d… Author: Henning Westerholt <hw(a)gilawa.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-01-11T10:30:40Z http_async_client: use error log level (GH #3706) --- Modified: src/modules/http_async_client/http_async_client_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/72595fe42d28d3e4a12341b405af63d… Patch: https://github.com/kamailio/kamailio/commit/72595fe42d28d3e4a12341b405af63d… --- diff --git a/src/modules/http_async_client/http_async_client_mod.c b/src/modules/http_async_client/http_async_client_mod.c index e9f6cb0d26b..983a9e387f1 100644 --- a/src/modules/http_async_client/http_async_client_mod.c +++ b/src/modules/http_async_client/http_async_client_mod.c @@ -294,7 +294,7 @@ static int mod_init(void) } if(load_tm_api(&tmb) < 0) { - LM_INFO("cannot load the TM-functions - async relay disabled\n"); + LM_ERR("cannot load the TM-functions - async relay disabled\n"); return -1; }

1 0

git:master:b5003620: core: keep listen socket even if advertise address does not resolve
by Daniel-Constantin Mierla 11 Jan '24

11 Jan '24

Module: kamailio Branch: master Commit: b50036200c5bc09fd2d412a18585b42d63763b27 URL: https://github.com/kamailio/kamailio/commit/b50036200c5bc09fd2d412a18585b42… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-11T08:21:00+01:00 core: keep listen socket even if advertise address does not resolve - the core advertise_address parameter works in the same fashion - previously it was starting by skipping the listen, which resulted in unexpected runtime list of sockets --- Modified: src/core/socket_info.c --- Diff: https://github.com/kamailio/kamailio/commit/b50036200c5bc09fd2d412a18585b42… Patch: https://github.com/kamailio/kamailio/commit/b50036200c5bc09fd2d412a18585b42… --- diff --git a/src/core/socket_info.c b/src/core/socket_info.c index 3f67f16981b..e048e0e2c4b 100644 --- a/src/core/socket_info.c +++ b/src/core/socket_info.c @@ -361,11 +361,13 @@ static inline struct socket_info *new_sock_info(char *name, he = resolvehost(si->useinfo.name.s); if(he == 0) { - LM_ERR("unable to resolve advertised name %s\n", + LM_WARN("unable to resolve advertised name %s\n", si->useinfo.name.s); - goto error; + si->useinfo.address.len = 0; + si->useinfo.address.af = 0; + } else { + hostent2ip_addr(&si->useinfo.address, he, 0); } - hostent2ip_addr(&si->useinfo.address, he, 0); } return si; error:

1 0

[kamailio/kamailio] rtp_media_server module hard codes contact header to "rms" (Issue #3380)
by Daniel Donoghue 11 Jan '24

11 Jan '24

### Description when using rtp_media_server module in my kamailio configuration, the `rms_answer` function generates a "200 OK" with a hard-coded Contact header user. I believe it would make more sense for this to use the To user (`$tU`) as that should be who is answering. #### Reproduction ``` route { if (t_precheck_trans()) { t_check_trans(); exit; } t_check_trans(); if (is_method("INVITE") && !has_totag()) { if (!rms_answer("rms:start_long")) { t_reply("503", "server error"); } exit; } if (rms_dialog_check()) { rms_sip_request(); } } ``` #### Debugging Data n/a #### Log Messages n/a #### SIP Traffic ``` INVITE sip:+461812345678@123.123.example.com SIP/2.0. Via: SIP/2.0/UDP 172.21.0.3:6060;branch=z9hG4bK6810d54f. Max-Forwards: 70. From: <sip:+461812345677@172.21.0.3:6060>;tag=as21f01093. To: <sip:+461812345678@123.123.example.com>. Contact: <sip:+461812345677@172.21.0.3:6060>. Call-ID: 4f56bf7336a114d75402e30f47013748@172.21.0.3:6060. CSeq: 102 INVITE. Allow: INVITE, ACK, CANCEL, OPTIONS, BYE, REFER, SUBSCRIBE, NOTIFY, INFO, PUBLISH, MESSAGE. Supported: replaces, timer. Content-Type: application/sdp. Content-Length: 248. .. v=0. o=root 1511071437 1511071437 IN IP4 172.21.0.3. s=Asterisk PBX 18.10.0. c=IN IP4 172.21.0.3. t=0 0. m=audio 26516 RTP/AVP 8 101. a=rtpmap:8 PCMA/8000. a=rtpmap:101 telephone-event/8000. a=fmtp:101 0-16. a=ptime:20. a=maxptime:150. a=sendrecv. SIP/2.0 200 OK. Via: SIP/2.0/UDP 172.21.0.2;branch=z9hG4bK09e2.aa4ab6455ca95a0ee95268497e877cc4.2;rport=5060. Via: SIP/2.0/UDP 172.21.0.3:6060;rport=6060;branch=z9hG4bK6810d54f. From: <sip:+461812345677@172.21.0.3:6060>;tag=as21f01093. To: <sip:+461812345678@123.123.example.com>;tag=f46321019d9527ace4b30f86d231e4a2-cafc0000. Call-ID: 4f56bf7336a114d75402e30f47013748@172.21.0.3:6060. CSeq: 102 INVITE. Contact: <sip:rms@172.21.0.4:5060>. Content-Type: application/sdp. Server: kamailio (5.6.3 (aarch64/linux)). Content-Length: 99. .. v=0. o=- 1028316687 1 IN IP4 172.21.0.4. s=-. c=IN IP4 172.21.0.4. t=0 0. m=audio 51900 RTP/AVP 8. ``` ### Possible Solutions n/a ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.6.3 (aarch64/linux) flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT-NOSMP, 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 bookworm ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3380 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3380(a)github.com>

4 6

git:5.7:c5ed0e65: tls: historical code comment on repeating SSL_CTX per worker
by S-P Chan 11 Jan '24

11 Jan '24

Module: kamailio Branch: 5.7 Commit: c5ed0e65b2530443b1367986dac2b3da111c0701 URL: https://github.com/kamailio/kamailio/commit/c5ed0e65b2530443b1367986dac2b3d… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-01-11T08:04:58+08:00 tls: historical code comment on repeating SSL_CTX per worker (cherry-pick from 29007ada5bc9e07ede3cdbce285f04d1298c0612) --- Modified: src/modules/tls/tls_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/c5ed0e65b2530443b1367986dac2b3d… Patch: https://github.com/kamailio/kamailio/commit/c5ed0e65b2530443b1367986dac2b3d… --- diff --git a/src/modules/tls/tls_mod.c b/src/modules/tls/tls_mod.c index 7cad1b046e4..beaf1b7b70b 100644 --- a/src/modules/tls/tls_mod.c +++ b/src/modules/tls/tls_mod.c @@ -433,6 +433,16 @@ static int tls_engine_init(); int tls_fix_engine_keys(tls_domains_cfg_t *, tls_domain_t *, tls_domain_t *); #endif +/* + * OpenSSL 1.1.1+: SSL_CTX is repeated in each worker + * + * OpenSSL RSA blinding works in single-process multi-threaded mode + * and depends on pthread_self() to separate threads. In Kamailio multi-process workers + * pthread_self() will not necessarily be unique, this will result in incorrect BN + * operations��hence we create a separate SSL_CTX for each worker + * + * EC operations do not use pthread_self(), so could use shared SSL_CTX + */ static int mod_child(int rank) { if(tls_disable || (tls_domains_cfg == 0))

1 0

git:master:29007ada: tls: historical code comment on repeating SSL_CTX per worker
by S-P Chan 11 Jan '24

11 Jan '24

Module: kamailio Branch: master Commit: 29007ada5bc9e07ede3cdbce285f04d1298c0612 URL: https://github.com/kamailio/kamailio/commit/29007ada5bc9e07ede3cdbce285f04d… Author: S-P Chan <shihping.chan(a)gmail.com> Committer: S-P Chan <shihping.chan(a)gmail.com> Date: 2024-01-11T08:03:07+08:00 tls: historical code comment on repeating SSL_CTX per worker --- Modified: src/modules/tls/tls_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/29007ada5bc9e07ede3cdbce285f04d… Patch: https://github.com/kamailio/kamailio/commit/29007ada5bc9e07ede3cdbce285f04d… --- diff --git a/src/modules/tls/tls_mod.c b/src/modules/tls/tls_mod.c index 7cad1b046e4..beaf1b7b70b 100644 --- a/src/modules/tls/tls_mod.c +++ b/src/modules/tls/tls_mod.c @@ -433,6 +433,16 @@ static int tls_engine_init(); int tls_fix_engine_keys(tls_domains_cfg_t *, tls_domain_t *, tls_domain_t *); #endif +/* + * OpenSSL 1.1.1+: SSL_CTX is repeated in each worker + * + * OpenSSL RSA blinding works in single-process multi-threaded mode + * and depends on pthread_self() to separate threads. In Kamailio multi-process workers + * pthread_self() will not necessarily be unique, this will result in incorrect BN + * operations��hence we create a separate SSL_CTX for each worker + * + * EC operations do not use pthread_self(), so could use shared SSL_CTX + */ static int mod_child(int rank) { if(tls_disable || (tls_domains_cfg == 0))

1 0

git:master:42bae2a6: registrar: cumpute randmized-range expire value as float and then cast
by Daniel-Constantin Mierla 10 Jan '24

10 Jan '24

Module: kamailio Branch: master Commit: 42bae2a634249fe7d8ba174eaffff0ee547f6d2b URL: https://github.com/kamailio/kamailio/commit/42bae2a634249fe7d8ba174eaffff0e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-10T13:45:30+01:00 registrar: cumpute randmized-range expire value as float and then cast --- Modified: src/modules/registrar/sip_msg.c --- Diff: https://github.com/kamailio/kamailio/commit/42bae2a634249fe7d8ba174eaffff0e… Patch: https://github.com/kamailio/kamailio/commit/42bae2a634249fe7d8ba174eaffff0e… --- diff --git a/src/modules/registrar/sip_msg.c b/src/modules/registrar/sip_msg.c index 9ef2c37db47..69e8606fbc1 100644 --- a/src/modules/registrar/sip_msg.c +++ b/src/modules/registrar/sip_msg.c @@ -48,15 +48,17 @@ static struct hdr_field *act_contact; */ static inline int randomize_expires(int expires, int range) { - int range_min; + float range_min; /* if no range is given just return expires */ if(range == 0) return expires; - range_min = expires - (float)range / 100 * expires; + range_min = (float)expires * (1.0 - ((float)range / 100)); - return range_min + (float)(kam_rand() % 100) / 100 * (expires - range_min); + return (int)(range_min + + ((float)(kam_rand() % 100) / 100) + * ((float)expires - range_min)); }

1 0

git:master:403ed489: modules: readme files regenerated - usrloc ... [skip ci]
by Kamailio Dev 10 Jan '24

10 Jan '24

Module: kamailio Branch: master Commit: 403ed48995e5eec5a795edd1748f9fe8a9ee520d URL: https://github.com/kamailio/kamailio/commit/403ed48995e5eec5a795edd1748f9fe… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-01-10T13:32:13+01:00 modules: readme files regenerated - usrloc ... [skip ci] --- Modified: src/modules/usrloc/README --- Diff: https://github.com/kamailio/kamailio/commit/403ed48995e5eec5a795edd1748f9fe… Patch: https://github.com/kamailio/kamailio/commit/403ed48995e5eec5a795edd1748f9fe…

1 0

[kamailio/kamailio] htable dmq can not get the expected result (Issue #3674)
by sznoname 10 Jan '24

10 Jan '24

### Description I have Kam207(192.168.100.207) and Kam208(192.168.100.208) here is Kam207 route: ```cfg debug=2 log_stderror=yes # mpath="/usr/local/lib/kamailio/modules/" loadmodule "tm.so" loadmodule "sl.so" loadmodule "pv.so" loadmodule "siputils.so" loadmodule "xlog.so" loadmodule "ctl.so" loadmodule "textops.so" loadmodule "dmq.so" loadmodule "htable.so" modparam("tm", "failure_reply_mode", 3) modparam("tm", "fr_timer", 30000) modparam("tm", "fr_inv_timer", 120000) modparam("dmq", "server_address", "sip:192.168.100.207:5060") modparam("dmq", "notification_address", "sip:192.168.100.208:5060") modparam("dmq", "multi_notify", 1) modparam("dmq", "num_workers", 4) modparam("dmq", "ping_interval", 15) modparam("htable", "htable", "bancc=>size=10;autoexpire=3600;dmqreplicate=1;") modparam("htable", "enable_dmq", 1) modparam("htable", "dmq_init_sync", 1) request_route { # xinfo("*** $sel(msg)\n"); if(is_method("KDMQ")) { dmq_handle_message(); exit; } sl_send_reply("404", "Not here"); exit; } ``` and here is Kam208 route: ```cfg debug=2 log_stderror=yes # mpath="/usr/local/lib/kamailio/modules/" loadmodule "tm.so" loadmodule "sl.so" loadmodule "pv.so" loadmodule "siputils.so" loadmodule "xlog.so" loadmodule "ctl.so" loadmodule "textops.so" loadmodule "dmq.so" loadmodule "htable.so" modparam("tm", "failure_reply_mode", 3) modparam("tm", "fr_timer", 30000) modparam("tm", "fr_inv_timer", 120000) modparam("dmq", "server_address", "sip:192.168.100.208:5060") modparam("dmq", "notification_address", "sip:192.168.100.207:5060") modparam("dmq", "multi_notify", 1) modparam("dmq", "num_workers", 4) modparam("dmq", "ping_interval", 15) modparam("htable", "htable", "bancc=>size=10;autoexpire=3600;dmqreplicate=1;") modparam("htable", "enable_dmq", 1) modparam("htable", "dmq_init_sync", 1) request_route { # xinfo("*** $sel(msg)\n"); if(is_method("KDMQ")) { dmq_handle_message(); exit; } sl_send_reply("404", "Not here"); exit; } ``` at Kam207 run `kamcmd htable.seti bancc TW 1` at Kam208 run `kamcmd htable.dump bancc`, and got: ``` { entry: 779 size: 1 slot: { { name: TW value: -2147483648 type: int } } } ``` ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` kamailio 5.7.3 (x86_64/linux) ``` * **Operating System**:  ``` Debian11 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3674 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3674(a)github.com>

2 4

git:master:bbb81e5f: usrloc: docs for ka_randomize parameter
by Daniel-Constantin Mierla 10 Jan '24

10 Jan '24

Module: kamailio Branch: master Commit: bbb81e5f6ef5744cc9b74302e8ecee6687199872 URL: https://github.com/kamailio/kamailio/commit/bbb81e5f6ef5744cc9b74302e8ecee6… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-10T13:20:34+01:00 usrloc: docs for ka_randomize parameter --- Modified: src/modules/usrloc/doc/usrloc_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/bbb81e5f6ef5744cc9b74302e8ecee6… Patch: https://github.com/kamailio/kamailio/commit/bbb81e5f6ef5744cc9b74302e8ecee6… --- diff --git a/src/modules/usrloc/doc/usrloc_admin.xml b/src/modules/usrloc/doc/usrloc_admin.xml index 77b18989b4b..d7d78ae7b00 100644 --- a/src/modules/usrloc/doc/usrloc_admin.xml +++ b/src/modules/usrloc/doc/usrloc_admin.xml @@ -1481,6 +1481,31 @@ modparam("usrloc", "ka_interval", 30) </example> </section> + <section id="usrloc.p.ka_randomize"> + <title><varname>ka_randomize</varname> (int)</title> + <para> + The parameter sets the the upper limit to the range of random seconds + to be added to ka_interval before checking if a new keepalive request + has to be sent. It should help to distribute better the sending of + keepalive requests. The keepalive for a location record is going to be + sent in a random fashion between (ka_interval + 0) and + (ka_interval + ka_randomize). + </para> + <para> + <emphasis> + Default value is <quote>20</quote>. + </emphasis> + </para> + <example> + <title>Set <varname>ka_randomize</varname> parameter</title> + <programlisting format="linespecific"> +... +modparam("usrloc", "ka_interval", 30) +... +</programlisting> + </example> + </section> + <section id="usrloc.p.ka_loglevel"> <title><varname>ka_loglevel</varname> (int)</title> <para>

1 0

git:master:5bf1a7ab: usrloc: option to randomize when keepalive is sent
by Daniel-Constantin Mierla 10 Jan '24

10 Jan '24

Module: kamailio Branch: master Commit: 5bf1a7ab30f14d989ab185585427c868429224fa URL: https://github.com/kamailio/kamailio/commit/5bf1a7ab30f14d989ab185585427c86… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-10T13:20:34+01:00 usrloc: option to randomize when keepalive is sent - default adds a range from 0 to 20 seconds to ka_interval --- Modified: src/modules/usrloc/ul_keepalive.c Modified: src/modules/usrloc/usrloc_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/5bf1a7ab30f14d989ab185585427c86… Patch: https://github.com/kamailio/kamailio/commit/5bf1a7ab30f14d989ab185585427c86… --- diff --git a/src/modules/usrloc/ul_keepalive.c b/src/modules/usrloc/ul_keepalive.c index 435f84d7c71..eba6bec5c0d 100644 --- a/src/modules/usrloc/ul_keepalive.c +++ b/src/modules/usrloc/ul_keepalive.c @@ -44,6 +44,7 @@ extern int ul_keepalive_timeout; extern int ul_ka_interval; +extern int ul_ka_randomize; static int ul_ka_send(str *kamsg, dest_info_t *kadst); @@ -110,6 +111,7 @@ int ul_ka_urecord(urecord_t *ur) int i; struct timeval tv; time_t tnow = 0; + int ka_limit = 0; if(ul_ka_mode == ULKA_NONE) { return 0; @@ -160,10 +162,12 @@ int ul_ka_urecord(urecord_t *ur) } } } - if(ul_ka_interval > 0 && uc->last_keepalive > 0 - && (uc->last_keepalive + ul_ka_interval) < tnow) { - /* not yet the time for keepalive */ - continue; + if(ul_ka_interval > 0 && uc->last_keepalive > 0) { + ka_limit = ul_ka_interval - (fastrand() % ul_ka_randomize); + if((uc->last_keepalive + ka_limit) < tnow) { + /* not yet the time for keepalive */ + continue; + } } if(uc->received.len > 0) { sdst = uc->received; diff --git a/src/modules/usrloc/usrloc_mod.c b/src/modules/usrloc/usrloc_mod.c index f493915a564..b14fe7273e1 100644 --- a/src/modules/usrloc/usrloc_mod.c +++ b/src/modules/usrloc/usrloc_mod.c @@ -128,6 +128,7 @@ str ul_ka_method = str_init("OPTIONS"); int ul_ka_mode = 0; int ul_ka_filter = 0; int ul_ka_interval = 40; +int ul_ka_randomize = 20; int ul_ka_loglevel = 255; str ul_ka_logmsg = str_init(" to-uri: [$tu] remote-addr: [$sas]"); pv_elem_t *ul_ka_logfmt = NULL; @@ -302,6 +303,7 @@ static param_export_t params[] = { {"ka_method", PARAM_STR, &ul_ka_method}, {"ka_filter", PARAM_INT, &ul_ka_filter}, {"ka_interval", PARAM_INT, &ul_ka_interval}, + {"ka_randomize", PARAM_INT, &ul_ka_randomize}, {"ka_timeout", PARAM_INT, &ul_keepalive_timeout}, {"ka_loglevel", PARAM_INT, &ul_ka_loglevel}, {"ka_logmsg", PARAM_STR, &ul_ka_logmsg},

1 0

git:master:03f939e3: modules: readme files regenerated - usrloc ... [skip ci]
by Kamailio Dev 10 Jan '24

10 Jan '24

Module: kamailio Branch: master Commit: 03f939e3d1a0dbf56791943f90942984984f2d72 URL: https://github.com/kamailio/kamailio/commit/03f939e3d1a0dbf56791943f9094298… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-01-10T12:47:06+01:00 modules: readme files regenerated - usrloc ... [skip ci] --- Modified: src/modules/usrloc/README --- Diff: https://github.com/kamailio/kamailio/commit/03f939e3d1a0dbf56791943f9094298… Patch: https://github.com/kamailio/kamailio/commit/03f939e3d1a0dbf56791943f9094298…

1 0

git:master:36608af8: usrloc: docs for ka_interval parameter
by Daniel-Constantin Mierla 10 Jan '24

10 Jan '24

Module: kamailio Branch: master Commit: 36608af81b3ed64b3800be1625647ed0a9e5031d URL: https://github.com/kamailio/kamailio/commit/36608af81b3ed64b3800be1625647ed… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-10T12:42:43+01:00 usrloc: docs for ka_interval parameter --- Modified: src/modules/usrloc/doc/usrloc_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/36608af81b3ed64b3800be1625647ed… Patch: https://github.com/kamailio/kamailio/commit/36608af81b3ed64b3800be1625647ed… --- diff --git a/src/modules/usrloc/doc/usrloc_admin.xml b/src/modules/usrloc/doc/usrloc_admin.xml index 5c0796935c7..77b18989b4b 100644 --- a/src/modules/usrloc/doc/usrloc_admin.xml +++ b/src/modules/usrloc/doc/usrloc_admin.xml @@ -1460,6 +1460,27 @@ modparam("usrloc", "ka_timeout", 120) </example> </section> + <section id="usrloc.p.ka_interval"> + <title><varname>ka_interval</varname> (int)</title> + <para> + The parameter sets the interval in seconds after which a new keepalive + request should be sent for a location record. + </para> + <para> + <emphasis> + Default value is <quote>40</quote>. + </emphasis> + </para> + <example> + <title>Set <varname>ka_interval</varname> parameter</title> + <programlisting format="linespecific"> +... +modparam("usrloc", "ka_interval", 30) +... +</programlisting> + </example> + </section> + <section id="usrloc.p.ka_loglevel"> <title><varname>ka_loglevel</varname> (int)</title> <para>

1 0

git:master:53ace443: usrloc: added keepalive interval to set the step for sending
by Daniel-Constantin Mierla 10 Jan '24

10 Jan '24

Module: kamailio Branch: master Commit: 53ace443020075f7ecd12c72f154193c6cfa6af3 URL: https://github.com/kamailio/kamailio/commit/53ace443020075f7ecd12c72f154193… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-10T12:42:43+01:00 usrloc: added keepalive interval to set the step for sending - the keepalive is not going to be sent before the interval elapsed - the keepalive is no longer sent on every timer callback - default 40 seconds --- Modified: src/modules/usrloc/ul_keepalive.c Modified: src/modules/usrloc/usrloc_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/53ace443020075f7ecd12c72f154193… Patch: https://github.com/kamailio/kamailio/commit/53ace443020075f7ecd12c72f154193… --- diff --git a/src/modules/usrloc/ul_keepalive.c b/src/modules/usrloc/ul_keepalive.c index 02c21c0837e..435f84d7c71 100644 --- a/src/modules/usrloc/ul_keepalive.c +++ b/src/modules/usrloc/ul_keepalive.c @@ -43,6 +43,7 @@ #include "ul_keepalive.h" extern int ul_keepalive_timeout; +extern int ul_ka_interval; static int ul_ka_send(str *kamsg, dest_info_t *kadst); @@ -159,6 +160,11 @@ int ul_ka_urecord(urecord_t *ur) } } } + if(ul_ka_interval > 0 && uc->last_keepalive > 0 + && (uc->last_keepalive + ul_ka_interval) < tnow) { + /* not yet the time for keepalive */ + continue; + } if(uc->received.len > 0) { sdst = uc->received; } else { diff --git a/src/modules/usrloc/usrloc_mod.c b/src/modules/usrloc/usrloc_mod.c index e6aa591bbce..f493915a564 100644 --- a/src/modules/usrloc/usrloc_mod.c +++ b/src/modules/usrloc/usrloc_mod.c @@ -127,6 +127,7 @@ str ul_ka_domain = str_init("kamailio.org"); str ul_ka_method = str_init("OPTIONS"); int ul_ka_mode = 0; int ul_ka_filter = 0; +int ul_ka_interval = 40; int ul_ka_loglevel = 255; str ul_ka_logmsg = str_init(" to-uri: [$tu] remote-addr: [$sas]"); pv_elem_t *ul_ka_logfmt = NULL; @@ -300,6 +301,7 @@ static param_export_t params[] = { {"ka_domain", PARAM_STR, &ul_ka_domain}, {"ka_method", PARAM_STR, &ul_ka_method}, {"ka_filter", PARAM_INT, &ul_ka_filter}, + {"ka_interval", PARAM_INT, &ul_ka_interval}, {"ka_timeout", PARAM_INT, &ul_keepalive_timeout}, {"ka_loglevel", PARAM_INT, &ul_ka_loglevel}, {"ka_logmsg", PARAM_STR, &ul_ka_logmsg},

1 0

git:master:5a40d1ef: http_async_client: exit mod_init if tm is not loaded
by Henning Westerholt 10 Jan '24

10 Jan '24

Module: kamailio Branch: master Commit: 5a40d1ef1b484c4fc437a3f0183a101241e80313 URL: https://github.com/kamailio/kamailio/commit/5a40d1ef1b484c4fc437a3f0183a101… Author: Federico Cabiddu <federico.cabiddu(a)gmail.com> Committer: Henning Westerholt <hw(a)gilawa.com> Date: 2024-01-10T10:06:05+01:00 http_async_client: exit mod_init if tm is not loaded --- Modified: src/modules/http_async_client/http_async_client_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/5a40d1ef1b484c4fc437a3f0183a101… Patch: https://github.com/kamailio/kamailio/commit/5a40d1ef1b484c4fc437a3f0183a101… --- diff --git a/src/modules/http_async_client/http_async_client_mod.c b/src/modules/http_async_client/http_async_client_mod.c index 3064a5491d6..e9f6cb0d26b 100644 --- a/src/modules/http_async_client/http_async_client_mod.c +++ b/src/modules/http_async_client/http_async_client_mod.c @@ -295,7 +295,7 @@ static int mod_init(void) if(load_tm_api(&tmb) < 0) { LM_INFO("cannot load the TM-functions - async relay disabled\n"); - memset(&tmb, 0, sizeof(tm_api_t)); + return -1; } /* allocate workers array */

1 0

[kamailio/kamailio] List selectors (#2705)
by sergey-safarov 10 Jan '24

10 Jan '24

#### Pre-Submission Checklist - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [ ] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist:  - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description This is the selectors feature for `lost` module. The module still in development but the core functionality is working. What need to do 1. separate feature into dedicated `.c` file; 2. rebase on current `lost` implementation; 3. add documentation; 4. add a unit test. Would be fine if we can merge this into 5.5. This branch based on e7d68556e3560f488c4f421731f78b3f9abe549e commit You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2705 -- Commit Summary -- * http_client: http_client_request (api) content-type header support * lost: new features, attributes and a new function to dereference location * lost: bug-fix due to a code formatting error * lost: DOM level count fix * http_client: duplicated code removed * lost: README update * lost: memory leak fix and code refactoring * lost: implemented sectors -- File Changes -- M src/modules/http_client/curl_api.c (1) M src/modules/http_client/curl_api.h (3) M src/modules/http_client/functions.c (30) M src/modules/http_client/functions.h (14) M src/modules/lost/doc/lost.xml (2) M src/modules/lost/doc/lost_admin.xml (172) M src/modules/lost/functions.c (921) M src/modules/lost/functions.h (3) M src/modules/lost/lost.c (754) A src/modules/lost/naptr.c (255) A src/modules/lost/naptr.h (38) M src/modules/lost/pidf.c (5) A src/modules/lost/response.c (991) A src/modules/lost/response.h (131) M src/modules/lost/utilities.c (532) M src/modules/lost/utilities.h (65) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2705.patch https://github.com/kamailio/kamailio/pull/2705.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2705

5 16

[kamailio/kamailio] kamailio 5.7.1 as a webrtc server experiences random crashes (Issue #3563)
by themsley-voiceflex 09 Jan '24

09 Jan '24

### Description Crashed after about 30 minutes of uptime. I probably have several bug reports to post on similar lines, they may all be related or they may all be completely different but I will raise a separate bug report for each. ### Troubleshooting Tried varying SHM_MEMORY from 256 to 512 and PKG_MEMORY from 8 to 64 in various increments. None seemed to help. #### Reproduction Open kamailio on tcp port 7443 and set up webrtc to point to it for traffic. `` #### Debugging Data coredumpctl debug PID: 1186 (kamailio) UID: 990 (kamailio) GID: 988 (kamailio) Signal: 11 (SEGV) Timestamp: Wed 2023-09-13 22:39:40 UTC (8min ago) Command Line: /usr/sbin/kamailio -DD -P /run/kamailio/kamailio.pid -f /etc/kamailio/kamailio.cfg -m 256 -M 8 --atexit=no Executable: /usr/sbin/kamailio Control Group: /system.slice/kamailio.service Unit: kamailio.service Slice: system.slice Boot ID: 3fc4251c606f4c6183610573f44c46ba Machine ID: 16e913e686424f13b970f9b1044cc6b3 Hostname: kamrtc01.voiceflex.com Storage: /var/lib/systemd/coredump/core.kamailio.990.3fc4251c606f4c6183610573f44c46ba.1186.1694644780000000.zst (present) Size on Disk: 4.6M Message: Process 1186 (kamailio) of user 990 dumped core. Stack trace of thread 1186: #0 0x00007f156ae338aa __strlen_sse2 (libc.so.6 + 0x338aa) #1 0x00007f156ae84278 __vfprintf_internal (libc.so.6 + 0x84278) #2 0x00007f156af4775a __vsyslog_internal (libc.so.6 + 0x14775a) #3 0x00007f156af47c3a syslog (libc.so.6 + 0x147c3a) #4 0x00000000007164b4 qm_debug_check_frag (kamailio + 0x3164b4) #5 0x000000000071a703 qm_free (kamailio + 0x31a703) #6 0x0000000000725209 qm_shm_free (kamailio + 0x325209) #7 0x00007f15690843d6 ser_free (tls.so + 0x313d6) #8 0x00007f156996d737 ERR_pop_to_mark (libcrypto.so.3 + 0x16d737) #9 0x00007f15699b3e42 ossl_namemap_doall_names (libcrypto.so.3 + 0x1b3e42) #10 0x00007f156999f225 EVP_PKEY_set_type_by_keymgmt (libcrypto.so.3 + 0x19f225) #11 0x00007f156999859a evp_keymgmt_util_assign_pkey (libcrypto.so.3 + 0x19859a) #12 0x00007f15699a21c3 EVP_PKEY_generate (libcrypto.so.3 + 0x1a21c3) #13 0x00007f1569d8082d ssl_generate_param_group (libssl.so.3 + 0x2682d) #14 0x00007f1569db52ea tls_parse_ctos_key_share (libssl.so.3 + 0x5b2ea) #15 0x00007f1569da7f79 tls_parse_all_extensions (libssl.so.3 + 0x4df79) #16 0x00007f1569dd1cde ossl_statem_server_post_process_message (libssl.so.3 + 0x77cde) #17 0x00007f1569dbc2cb state_machine.part.0 (libssl.so.3 + 0x622cb) #18 0x00007f15690ac584 tls_accept (tls.so + 0x59584) #19 0x00007f15690b78be tls_h_read_f (tls.so + 0x648be) #20 0x00000000006c79bc tcp_read_headers (kamailio + 0x2c79bc) #21 0x00000000006cfb94 tcp_read_req (kamailio + 0x2cfb94) #22 0x00000000006d4cc3 handle_io (kamailio + 0x2d4cc3) #23 0x00000000006c271d io_wait_loop_epoll (kamailio + 0x2c271d) #24 0x00000000006d7a47 tcp_receive_loop (kamailio + 0x2d7a47) #25 0x00000000006b7689 tcp_init_children (kamailio + 0x2b7689) #26 0x000000000042fae9 main_loop (kamailio + 0x2fae9) #27 0x0000000000439969 main (kamailio + 0x39969) #28 0x00007f156ae3feb0 __libc_start_call_main (libc.so.6 + 0x3feb0) #29 0x00007f156ae3ff60 __libc_start_main@@GLIBC_2.34 (libc.so.6 + 0x3ff60) #30 0x000000000041dc65 _start (kamailio + 0x1dc65) ELF object binary architecture: AMD x86-64 `` `` (gdb) bt full #0 0x00007f156ae338aa in __strlen_sse2 () from /lib64/libc.so.6 No symbol table info available. #1 0x00007f156ae84278 in __vfprintf_internal () from /lib64/libc.so.6 No symbol table info available. #2 0x00007f156af4775a in __vsyslog_internal () from /lib64/libc.so.6 No symbol table info available. #3 0x00007f156af47c3a in syslog () from /lib64/libc.so.6 No symbol table info available. #4 0x00000000007164b4 in qm_debug_check_frag (qm=0x7f1559053000, f=0x7f155a04cea0, file=0x7f15690cafe1 "tls: tls_init.c", line=405, efile=0x8a1659 "core/mem/q_malloc.c", eline=511) at core/mem/q_malloc.c:123 __llevel = -3 p = 0x10 __func__ = "qm_debug_check_frag" #5 0x000000000071a703 in qm_free (qmp=0x7f1559053000, p=0x7f155a04ced8, file=0x7f15690cafe1 "tls: tls_init.c", func=0x7f15690ccae0 <__func__.0> "ser_free", line=405, mname=0x7f15690caf52 "tls") at core/mem/q_malloc.c:511 qm = 0x7f1559053000 f = 0x7f155a04cea0 size = 140730841078464 next = 0x1 prev = 0x7f155908c6c8 __func__ = "qm_free" #6 0x0000000000725209 in qm_shm_free (qmp=0x7f1559053000, p=0x7f155a04ced8, file=0x7f15690cafe1 "tls: tls_init.c", func=0x7f15690ccae0 <__func__.0> "ser_free", line=405, mname=0x7f15690caf52 "tls") at core/mem/q_malloc.c:1350 No locals. #7 0x00007f15690843d6 in ser_free (ptr=0x7f155a04ced8, fname=0x7f1569b2d829 "crypto/err/err_local.h", fline=88) at tls_init.c:405 __func__ = "ser_free" #8 0x00007f156996d737 in err_clear (deall=0, i=<optimized out>, es=0x7f15590f6770) at crypto/err/err_local.h:88 No locals. #9 ERR_pop_to_mark () at crypto/err/err.c:902 es = 0x7f15590f6770 #10 0x00007f15699b3e42 in ossl_namemap_doall_names (namemap=<optimized out>, number=<optimized out>, fn=0x7f156999de80 <find_ameth>, data=0x7ffe73ca9bb0) at crypto/core_namemap.c:168 cbdata = {number = 107, names = 0x7f15597d9ca0, found = 2} num_names = <optimized out> i = <optimized out> #11 0x00007f156999f225 in EVP_PKEY_set_type_by_keymgmt (pkey=0x7f1559b1b918, keymgmt=0x7f15593dd430) at crypto/evp/p_lib.c:1633 str = {0x0, 0x0} __func__ = "EVP_PKEY_set_type_by_keymgmt" #12 0x00007f156999859a in evp_keymgmt_util_assign_pkey (pkey=0x7f1559b1b918, keymgmt=<optimized out>, keydata=0x7f1559cb4148) at crypto/evp/keymgmt_lib.c:64 __func__ = "evp_keymgmt_util_assign_pkey" #13 0x00007f15699a21c3 in evp_keymgmt_util_gen (cb=0x7f15699a04b0 <ossl_callback_to_pkey_gencb>, cbarg=0x7f1559b1cdf0, genctx=<optimized out>, keymgmt=<optimized out>, target=0x7f1559b1b918) at crypto/evp/keymgmt_lib.c:518 keydata = 0x7f1559cb4148 keydata = <optimized out> #14 EVP_PKEY_generate (ctx=ctx@entry=0x7f1559b1cdf0, ppkey=ppkey@entry=0x7ffe73ca9c70) at crypto/evp/pmeth_gn.c:189 ret = <optimized out> allocated_pkey = <optimized out> gentmp = {-1, -1} __func__ = "EVP_PKEY_generate" #15 0x00007f15699a23ce in EVP_PKEY_paramgen (ctx=ctx@entry=0x7f1559b1cdf0, ppkey=ppkey@entry=0x7ffe73ca9c70) at crypto/evp/pmeth_gn.c:265 __func__ = "EVP_PKEY_paramgen" #16 0x00007f1569d8082d in ssl_generate_param_group (s=0x7f1559ec7320, id=<optimized out>) at ssl/s3_lib.c:4751 --Type <RET> for more, q to quit, c to continue without paging-- pctx = 0x7f1559b1cdf0 pkey = 0x7f1559b1b918 ginf = 0x7f1559463030 __func__ = "ssl_generate_param_group" #17 0x00007f1569db52ea in tls_parse_ctos_key_share (s=0x7f1559ec7320, pkt=<optimized out>, context=<optimized out>, x=<optimized out>, chainidx=<optimized out>) at ssl/statem/extensions_srvr.c:664 group_id = <optimized out> key_share_list = {curr = 0x7f1559fef73d "", remaining = 0} encoded_pt = {curr = 0x7f1559fef71d "\\\024\061\225\003+G\vz\224;\a\261\032D\212\066\304\370!\235!\304C\365\035)9", remaining = 32} clntgroups = 0x7f1559b1cd78 srvrgroups = 0x7f155945b478 clnt_num_groups = 5 srvr_num_groups = 10 found = 0 __func__ = "tls_parse_ctos_key_share" #18 0x00007f1569da7f79 in tls_parse_all_extensions (s=0x7f1559ec7320, context=128, exts=0x7f1559bc1068, x=0x0, chainidx=0, fin=1) at ssl/statem/extensions.c:753 i = 19 numexts = 26 thisexd = <optimized out> #19 0x00007f1569dd1cde in tls_early_post_process_client_hello (s=0x7f1559ec7320) at ssl/statem/statem_srvr.c:1862 j = <optimized out> loop = <optimized out> scsvs = 0x7f1559f649f8 al = 80 protverr = <optimized out> id = <optimized out> comp = 0x0 i = <optimized out> c = <optimized out> ciphers = 0x7f1559cb40c0 clienthello = 0x7f1559e950f0 dgrd = DOWNGRADE_NONE 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> __func__ = {<optimized out> <repeats 36 times>} 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> --Type <RET> for more, q to quit, c to continue without paging-- comp_id = <optimized out> k = <optimized out> m = <optimized out> nn = <optimized out> v = <optimized out> done = <optimized out> o = <optimized out> #20 tls_post_process_client_hello (wst=<optimized out>, s=0x7f1559ec7320) at ssl/statem/statem_srvr.c:2189 rv = <optimized out> cipher = <optimized out> err = <optimized out> cipher = <optimized out> err = <optimized out> __func__ = {<optimized out> <repeats 30 times>} rv = <optimized out> rv = <optimized out> ret = <optimized out> #21 ossl_statem_server_post_process_message (s=s@entry=0x7f1559ec7320, wst=<optimized out>) at ssl/statem/statem_srvr.c:1236 st = <optimized out> __func__ = "ossl_statem_server_post_process_message" #22 0x00007f1569dbc2cb in read_state_machine (s=0x7f1559ec7320) at ssl/statem/statem.c:675 mt = <optimized out> pkt = {curr = 0x7f1559fef820 "", remaining = 0} max_message_size = <optimized out> ret = <optimized out> process_message = 0x7f1569dce720 <ossl_statem_server_process_message> post_process_message = <optimized out> st = 0x7f1559ec7368 len = <optimized out> transition = <optimized out> cb = 0x7f1569071e72 <sr_ssl_ctx_info_callback> st = <optimized out> ret = <optimized out> mt = <optimized out> len = <optimized out> transition = <optimized out> pkt = {curr = <optimized out>, remaining = <optimized out>} process_message = <optimized out> post_process_message = <optimized out> max_message_size = <optimized out> cb = <optimized out> __func__ = {<optimized out> <repeats 19 times>} #23 state_machine (s=0x7f1559ec7320, server=<optimized out>) at ssl/statem/statem.c:442 buf = 0x0 cb = <optimized out> st = <optimized out> ret = <optimized out> ssret = <optimized out> __func__ = {<optimized out> <repeats 14 times>} #24 0x00007f15690ac584 in tls_accept (c=0x7f1559af1e30, error=0x7ffe73cca124) at tls_server.c:470 ret = 0 ssl = 0x7f1559ec7320 --Type <RET> for more, q to quit, c to continue without paging-- cert = 0x1 tls_c = 0x7f1559eb7ca8 tls_log = 0 __func__ = "tls_accept" #25 0x00007f15690b78be in tls_h_read_f (c=0x7f1559af1e30, flags=0x7ffe73cca560) at tls_server.c:1175 r = 0x7f1559af1f58 bytes_free = 16383 bytes_read = 517 read_size = 16383 ssl_error = 0 ssl_read = 0 ssl = 0x7f1559ec7320 rd_buf = "\026\003\001\002\000\001\000\001\374\003\003\037~\204HD\300A\177\303\343\071\354Z\327t\203\377\334iC&\212ў\fװG*\222J\223 \027\213^8\025\212\346H\321·\332\063\060\352=\250\255\257\002\231\306{\026Cp\315\344\274\347\332#\000*\372\372\023\001\023\002\023\003\300,\300+̩\300\060\300/̨\300\n\300\t\300\024\300\023\000\235\000\234\000\065\000/\300\b\300\022\000\n\001\000\001\211\212\212\000\000\000\000\000\030\000\026\000\000\023webrtc.nebulaip.com\000\027\000\000\377\001\000\001\000\000\n\000\f\000\n\232\232\000\035\000\027\000\030\000\031\000\v\000\002\001\000\000\020\000\v\000\t\bhttp/1.1\000\005\000"... wr_buf = "@\241\313s\376\177\000\000\025\003\000\000\025\003\000\000\000\000\001\000\301X\252\275\204\313b\322\000\023\302:y}\",\247\205\364R\353ؑ\372#\326\357'\354D\244\366_\336\355D\227U\360\071u\305WI\370BZ\262\303V\272j(;\363\035\363\372\062\025\365\276ݍm\207\344\373\346\355\320ť\026\235lp(v4/\b\373\276z\002\261U\005P#\230v\215\261O)\021\347\370\071\337\316r\336a\332#\357\245N,\353\024\215\267z\t\026\313u\354\023Jf(\370\227z\023 \242@hy+P\026CM\220c\r9f\374\277\213\025\t\322{\355\000\363Փ\256\v\303\375\027\201\303؇\202\316#Q\215Ds\001\021\254zSF\177\325\350\020\252|"... rd = {buf = 0x7ffe73cba120 "\026\003\001\002", pos = 517, used = 517, size = 65536} wr = {buf = 0x7ffe73caa120 "@\241\313s\376\177", pos = 0, used = 0, size = 65536} tls_c = 0x7f1559eb7ca8 enc_rd_buf = 0x0 n = 0 flush_flags = 0 err_src = 0x7f15690d45a0 "TLS read:" ip_buf = "93.95.124.6", '\000' <repeats 52 times> x = 1793780809 tls_dbg = 4095 __func__ = "tls_h_read_f" #26 0x00000000006c79bc in tcp_read_headers (c=0x7f1559af1e30, read_flags=0x7ffe73cca560) at core/tcp_read.c:441 bytes = 65535 remaining = 0 p = 0x2f759ab3eb3 <error: Cannot access memory at address 0x2f759ab3eb3> r = 0x7f1559af1f58 mc = 32533 body_len = 22955 mfline = 0x7f1559ab3c90 "\201\376\002\367\266/\375\257\344j\272\346\345{\270\375\226\\\224ߌX\230\315\304[\236\201\330J\237\332\332N\224ߘL\222\302\226|\264\377\231\035ӟ\273%\253\306\327\025\335\374\377\177ҝ\230\037\322\370\345|\335\333\325\036\314ȄM\312\316\322\027\234\201\337A\213\316\332F\231\224\324]\234\301\325G\300ՏG\272\233\324dΙ\217\033ʟ\273%\260\316\316\002\273\300\304X\234\335\322\\Ǐ\201\037\360\245\342@Ǐ\212\\\224ߌX\230\315\304[\236\357\301J\237\335\302L\323\301\323M\210\303\327F\215\201\325@\220\221\273%\273\335\331BǏ\212\\\224ߌX\230\315\304[\236\357\301J\237\335\302L\323\301\323M\210\303\327F\215\201"... mtransid = {s = 0x7f1559ab38e8 "V", len = 1504393472} __func__ = "tcp_read_headers" #27 0x00000000006cfb94 in tcp_read_req (con=0x7f1559af1e30, bytes_read=0x7ffe73cca564, read_flags=0x7ffe73cca560) at core/tcp_read.c:1469 bytes = -1 total_bytes = 0 resp = 1 size = 0 req = 0x7f1559af1f58 dst = {send_sock = 0x7f1559ab38e8, to = {s = {sa_family = 0, sa_data = "\000\020\002\000\000\000\001\000\314s\026\000\000"}, sin = {sin_family = 0, sin_port = 4096, sin_addr = {s_addr = 2}, sin_zero = "\001\000\314s\026\000\000"}, sin6 = {sin6_family = 0, sin6_port = 4096, sin6_flowinfo = 2, --Type <RET> for more, q to quit, c to continue without paging-- sin6_addr = {__in6_u = {__u6_addr8 = "\001\000\314s\026\000\000\000ಙ\000\000\000\000", __u6_addr16 = {1, 29644, 22, 0, 45792, 153, 0, 0}, __u6_addr32 = { 1942749185, 22, 10072800, 0}}}, sin6_scope_id = 1794407328}, sas = {ss_family = 0, __ss_padding = "\000\020\002\000\000\000\001\000\314s\026\000\000\000ಙ\000\000\000\000\000\240{\364j\025\177\000\000\200:\253Y\025\177\000\000\240\244\314s\250\204\257M\240\264\231\000\000\000\000\000\024\000\000\000\000\000\000\000\001\000\000\000\001\000\000\000\027\000\000\000\376\177\000\000-uQ\000\000\000\000\000X\245\314s\376\177\000\000\b", '\000' <repeats 22 times>, __ss_align = 140730841211888}}, id = 1, send_flags = {f = 0, blst_imask = 0}, proto = -48 '\320', proto_pad0 = -93 '\243', proto_pad1 = 29644} c = 0 '\000' ret = 0 __func__ = "tcp_read_req" #28 0x00000000006d4cc3 in handle_io (fm=0x7f156a799ce8, events=1, idx=-1) at core/tcp_read.c:1780 ret = 8 n = 8 read_flags = RD_CONN_SHORT_READ con = 0x7f1559af1e30 s = 23 resp = 1 t = 1303348392 ee = 0x0 __func__ = "handle_io" #29 0x00000000006c271d in io_wait_loop_epoll (h=0x99b2e0 <io_w>, t=2, repeat=0) at core/io_wait.h:1070 n = 1 r = 0 fm = 0x7f156a799ce8 revents = 1 __func__ = "io_wait_loop_epoll" #30 0x00000000006d7a47 in tcp_receive_loop (unix_sock=47) at core/tcp_read.c:1976 __func__ = "tcp_receive_loop" #31 0x00000000006b7689 in tcp_init_children (woneinit=0x7ffe73cca91c) at core/tcp_main.c:5239 r = 3 i = 3 reader_fd_1 = 47 pid = 0 si_desc = "tcp receiver (generic)\000\000\000\000\000\000\000\000\000\000\023e\204\000\000\000\000\000\033\000\000\000\000\000\000\000\240{\364j\025\177\000\000\000\000\000\020\000\000\000\000\260\250\314s\376\177\000\000\034\003e\000\000\000\000\000\260\250\314s\376\177\000\000\345\030n\000\000\000\000\000`\250\314s\376\177\000\000\252\264~\000\000\000\000\000\023e\204\000\000\000\000\000\310\311\bY \000\000" si = 0x0 __func__ = "tcp_init_children" #32 0x000000000042fae9 in main_loop () at main.c:1851 i = 4 pid = 1165 si = 0x0 si_desc = "udp receiver child=3 sock=172.16.128.55:5090\000\023\243\360\200\251\314s\376\177\000\000\000\000\000\000\000\000\000\000\340\256\314s\376\177\000\000\023e\204\000\000\000\000\000\033\000\000\000\000\000\000\000X_jj\025\177\000\000\000\000\000\020\000\000\000\000:|\364j\025\177\000\000\060\000\000\000\060\000\000\000\230\252\314s\376\177\000" nrprocs = 4 woneinit = 1 __func__ = "main_loop" #33 0x0000000000439969 in main (argc=11, argv=0x7ffe73ccaff8) at main.c:3086 cfg_stream = 0x1246380 c = -1 r = 0 tmp = 0x7ffe73ccbea5 "" --Type <RET> for more, q to quit, c to continue without paging-- tmp_len = 0 port = 0 proto = 0 ahost = 0x0 aport = 0 options = 0x8322d8 ":f:cm:M:dVIhEeb:l:L:n:vKrRDTN:W:w:t:u:g:P:G:SQ:O:a:A:x:X:Y:" ret = -1 seed = 3223125455 rfd = 4 debug_save = 0 debug_flag = 0 dont_fork_cnt = 2 n_lst = 0x200000064 p = 0x0 st = {st_dev = 24, st_ino = 891, st_nlink = 2, st_mode = 16832, st_uid = 990, st_gid = 988, __pad0 = 0, st_rdev = 0, st_size = 40, st_blksize = 4096, st_blocks = 0, st_atim = {tv_sec = 1694643229, tv_nsec = 729000000}, st_mtim = {tv_sec = 1694643229, tv_nsec = 729000000}, st_ctim = {tv_sec = 1694643229, tv_nsec = 729000000}, __glibc_reserved = {0, 0, 0}} tbuf = "\331\f k\025\177\000\000\070\313\034k\025\177\000\000p\254\314s\376\177\000\000\a\000\000\000\000\000\000\000ظ\034k\025\177\000\000\a\000\000\000#\000\000\000\060\032 k\025\177\000\000\245\346\035k\025\177\000\000\000\000\000\000\000\000\000\000\220\371\035k\025\177\000\000\070\000\341j\025\177\000\000\000*\034k\025\177\000\000\000\000\000\000\000\000\000\000@\254\314s\376\177\000\000\360\016 k\025\177\000\000\330\312\034k\025\177\000\000\330\312\034k\025\177\000\000\000\000\000\000\000\000\000\000\330\070\034k\025\177\000\000@\254\314s\376\177\000\000\220% k\025\177\000\000\000\260\034k\025\177\000\000\022\274\034k\025\177\000\000\066\261\035k\025\177\000\000\344#\340j\025\177\000\000\272"... option_index = 12 long_options = {{name = 0x834976 "help", has_arg = 0, flag = 0x0, val = 104}, {name = 0x82f4f4 "version", has_arg = 0, flag = 0x0, val = 118}, { name = 0x83497b "alias", has_arg = 1, flag = 0x0, val = 1024}, {name = 0x834981 "subst", has_arg = 1, flag = 0x0, val = 1025}, {name = 0x834987 "substdef", has_arg = 1, flag = 0x0, val = 1026}, {name = 0x834990 "substdefs", has_arg = 1, flag = 0x0, val = 1027}, {name = 0x83499a "server-id", has_arg = 1, flag = 0x0, val = 1028}, {name = 0x8349a4 "loadmodule", has_arg = 1, flag = 0x0, val = 1029}, {name = 0x8349af "modparam", has_arg = 1, flag = 0x0, val = 1030}, {name = 0x8349b8 "log-engine", has_arg = 1, flag = 0x0, val = 1031}, {name = 0x8349c3 "debug", has_arg = 1, flag = 0x0, val = 1032}, { name = 0x8349c9 "cfg-print", has_arg = 0, flag = 0x0, val = 1033}, {name = 0x8349d3 "atexit", has_arg = 1, flag = 0x0, val = 1034}, { name = 0x8349da "all-errors", has_arg = 0, flag = 0x0, val = 1035}, {name = 0x0, has_arg = 0, flag = 0x0, val = 0}} __func__ = "main" `` `` (gdb) info locals No symbol table info available. `` #### Log Messages ``` Sep 13 22:39:41 kamrtc01 /usr/sbin/kamailio[2110]: INFO: <core> [core/cfg/cfg_ctx.c:599]: cfg_set_now(): tls.low_mem_threshold1 has been changed to 14155776 Sep 13 22:39:41 kamrtc01 /usr/sbin/kamailio[2110]: INFO: <core> [core/cfg/cfg_ctx.c:599]: cfg_set_now(): tls.low_mem_threshold2 has been changed to 7077888 Sep 13 22:39:40 kamrtc01 /usr/sbin/kamailio[1184]: INFO: <script>: START: REGISTER from sip:webrtc@webrtc.nebulaip.com (IP:90.251.178.17:52715) Sep 13 22:39:40 kamrtc01 /usr/sbin/kamailio[1185]: CRITICAL: <core> [core/mem/q_malloc.c:519]: qm_free(): BUG: freeing already freed pointer (0x7f155a04cde0), called from tls: tls_init.c: ser_free(405), first free tls: tls_init.c: ser_free(405) - ignoring Sep 13 22:39:40 kamrtc01 kernel: kamailio[1186]: segfault at 60 ip 00007f156ae338aa sp 00007ffe73ca9118 error 4 in libc.so.6[7f156ae28000+175000] Sep 13 22:39:40 kamrtc01 kernel: Code: f3 0f 1e fa 66 0f ef c0 66 0f ef c9 66 0f ef d2 66 0f ef db 48 89 f8 48 89 f9 48 81 e1 ff 0f 00 00 48 81 f9 cf 0f 00 00 77 66 <f3> 0f 6f 20 66 0f 74 e0 66 0f d7 d4 85 d2 74 04 0f bc c2 c3 48 83 Sep 13 22:39:40 kamrtc01 systemd[1]: Started Process Core Dump (PID 2102/UID 0). Sep 13 22:39:41 kamrtc01 systemd-coredump[2103]: Process 1186 (kamailio) of user 990 dumped core.#012#012Stack trace of thread 1186:#012#0 0x00007f156ae338aa __strlen_sse2 (libc.so.6 + 0x338aa)#012#1 0x00007f156ae84278 __vfprintf_internal (libc.so.6 + 0x84278)#012#2 0x00007f156af4775a __vsyslog_internal (libc.so.6 + 0x14775a)#012#3 0x00007f156af47c3a syslog (libc.so.6 + 0x147c3a)#012#4 0x00000000007164b4 qm_debug_check_frag (kamailio + 0x3164b4)#012#5 0x000000000071a703 qm_free (kamailio + 0x31a703)#012#6 0x0000000000725209 qm_shm_free (kamailio + 0x325209)#012#7 0x00007f15690843d6 ser_free (tls.so + 0x313d6)#012#8 0x00007f156996d737 ERR_pop_to_mark (libcrypto.so.3 + 0x16d737)#012#9 0x00007f15699b3e42 ossl_namemap_doall_names (libcrypto.so.3 + 0x1b3e42)#012#10 0x00007f156999f225 EVP_PKEY_set_type_by_keymgmt (libcrypto.so.3 + 0x19f225)#012#11 0x00007f156999859a evp_keymgmt_util_assign_pkey (libcrypto.so.3 + 0x19859a)#012#12 0x00007f15699a21c3 EVP_PKEY_generate (libcrypto.so.3 + 0x1a21c3)#012#13 0x00007f1569d8082d ssl_generate_param_group (libssl.so.3 + 0x2682d)#012#14 0x00007f1569db52ea tls_parse_ctos_key_share (libssl.so.3 + 0x5b2ea)#012#15 0x00007f1569da7f79 tls_parse_all_extensions (libssl.so.3 + 0x4df79)#012#16 0x00007f1569dd1cde ossl_statem_server_post_process_message (libssl.so.3 + 0x77cde)#012#17 0x00007f1569dbc2cb state_machine.part.0 (libssl.so.3 + 0x622cb)#012#18 0x00007f15690ac584 tls_accept (tls.so + 0x59584)#012#19 0x00007f15690b78be tls_h_read_f (tls.so + 0x648be)#012#20 0x00000000006c79bc tcp_read_headers (kamailio + 0x2c79bc)#012#21 0x00000000006cfb94 tcp_read_req (kamailio + 0x2cfb94)#012#22 0x00000000006d4cc3 handle_io (kamailio + 0x2d4cc3)#012#23 0x00000000006c271d io_wait_loop_epoll (kamailio + 0x2c271d)#012#24 0x00000000006d7a47 tcp_receive_loop (kamailio + 0x2d7a47)#012#25 0x00000000006b7689 tcp_init_children (kamailio + 0x2b7689)#012#26 0x000000000042fae9 main_loop (kamailio + 0x2fae9)#012#27 0x0000000000439969 main (kamailio + 0x39969)#012#28 0x00007f156ae3feb0 __libc_start_call_main (libc.so.6 + 0x3feb0)#012#29 0x00007f156ae3ff60 __libc_start_main@@GLIBC_2.34 (libc.so.6 + 0x3ff60)#012#30 0x000000000041dc65 _start (kamailio + 0x1dc65)#012ELF object binary architecture: AMD x86-64 Sep 13 22:39:41 kamrtc01 systemd[1]: systemd-coredump(a)0-2102-0.service: Deactivated successfully. Sep 13 22:39:41 kamrtc01 /usr/sbin/kamailio[893]: ALERT: <core> [main.c:776]: handle_sigs(): child process 1186 exited by a signal 11 Sep 13 22:39:41 kamrtc01 /usr/sbin/kamailio[1145]: INFO: <core> [main.c:856]: sig_usr(): signal 15 received ``` #### SIP Traffic  ``` (paste your sip traffic here) ``` ### Possible Solutions Nope, sorry, completely out of my depth on this one. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.7.1 (x86_64/linux) 4238e4 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: 4238e4 compiled on 00:00:00 May 23 2022 with gcc 11.3.1 rpm Build Date : Mon 11 Sep 2023 19:02:26 UTC ``` * **Operating System**: ``` Oracle Enterprise Linux 9 (RHEL 9 clone) Linux kamrtc01.voiceflex.com 5.14.0-284.25.1.0.1.el9_2.x86_64 #1 SMP PREEMPT_DYNAMIC Fri Aug 4 09:00:16 PDT 2023 x86_64 x86_64 x86_64 GNU/Linux openssl-3.0.7-17.0.1.el9_2.x86_64 glibc-2.34-60.0.2.el9.x86_64 libgcc-11.3.1-4.3.0.1.el9.x86_64 kamailio-5.7.1-1.el9.x86_64 kamailio-ims-5.7.1-1.el9.x86_64 kamailio-mysql-5.7.1-1.el9.x86_64 kamailio-redis-5.7.1-1.el9.x86_64 kamailio-regex-5.7.1-1.el9.x86_64 kamailio-snmpstats-5.7.1-1.el9.x86_64 kamailio-tls-5.7.1-1.el9.x86_64 kamailio-websocket-5.7.1-1.el9.x86_64 kamailio-debuginfo-5.7.1-1.el9.x86_64 kamailio-ims-debuginfo-5.7.1-1.el9.x86_64 kamailio-mysql-debuginfo-5.7.1-1.el9.x86_64 kamailio-redis-debuginfo-5.7.1-1.el9.x86_64 kamailio-regex-debuginfo-5.7.1-1.el9.x86_64 kamailio-snmpstats-debuginfo-5.7.1-1.el9.x86_64 kamailio-tls-debuginfo-5.7.1-1.el9.x86_64 kamailio-websocket-debuginfo-5.7.1-1.el9.x86_64 ``` Unfortunately Oracle do not seem to have published recent glibc libgcc libstdc++ or sssd-client debuginfo packages. I will attempt to contact them to ask them to add those. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3563 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3563(a)github.com>

4 19

git:master:f485a0c9: pkg: rpm obs spec - removed app_sqlang module from packaging
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: f485a0c9f138ccfbeb548fa22ea9053afb449875 URL: https://github.com/kamailio/kamailio/commit/f485a0c9f138ccfbeb548fa22ea9053… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-09T15:32:14+01:00 pkg: rpm obs spec - removed app_sqlang module from packaging --- Modified: pkg/kamailio/obs/kamailio.spec --- Diff: https://github.com/kamailio/kamailio/commit/f485a0c9f138ccfbeb548fa22ea9053… Patch: https://github.com/kamailio/kamailio/commit/f485a0c9f138ccfbeb548fa22ea9053… --- diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec index 1dc6687e645..ab556f5cd59 100644 --- a/pkg/kamailio/obs/kamailio.spec +++ b/pkg/kamailio/obs/kamailio.spec @@ -1008,9 +1008,6 @@ Group: %{PKGGROUP} Requires: kamailio = %version BuildRequires: gcc-c++ -%description sqlang -app_sqlang module for Kamailio. - %package sqlite Summary: SQLite database connectivity for Kamailio @@ -2318,12 +2315,6 @@ fi %{_libdir}/kamailio/modules/statsd.so -%files sqlang -%defattr(-,root,root) -%doc %{_docdir}/kamailio/modules/README.app_sqlang -%{_libdir}/kamailio/modules/app_sqlang.so - - %files sqlite %defattr(-,root,root) %doc %{_docdir}/kamailio/modules/README.db_sqlite

1 0

[kamailio/kamailio] Error in make all (Issue #3705)
by Ali 09 Jan '24

09 Jan '24

### Description I've cloned Kamailio and tried to build with `make all` commands and the error below appeared: ``` module not found: modules/app_sqlang make[1]: *** [Makefile:508: modules] Error 1 make[1]: Leaving directory '/usr/local/src/kamailio-devel/kamailio/src' make: *** [Makefile:34: all] Error 2 ``` the error wasn't occured before [9bb1976 "Makefile.groups: removed app_sqlang from modules list"](https://github.com/kamailio/kamailio/commit/9bb1976b4b7d52542a4de37144194bfc24649222) and just appered after pulling new commits. ### Troubleshooting I could solve the issue by set `app_sqlang` as skip_modules or exclude_modules, it's also possible solve it by set in ``` make skip_modules="app_sqlang" cfg ``` * **Ubuntu 22.04** -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3705 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3705(a)github.com>

2 2

git:master:577fb110: htable: read rpc number value as long
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 577fb11010ff61af3c4b567581c0bbc5404654c8 URL: https://github.com/kamailio/kamailio/commit/577fb11010ff61af3c4b567581c0bbc… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-09T14:07:18+01:00 htable: read rpc number value as long - remove autoconvert for number, it is only for string values - GH #3674 --- Modified: src/modules/htable/htable.c --- Diff: https://github.com/kamailio/kamailio/commit/577fb11010ff61af3c4b567581c0bbc… Patch: https://github.com/kamailio/kamailio/commit/577fb11010ff61af3c4b567581c0bbc… --- diff --git a/src/modules/htable/htable.c b/src/modules/htable/htable.c index 5c682972f0c..0fcd280de9c 100644 --- a/src/modules/htable/htable.c +++ b/src/modules/htable/htable.c @@ -1673,7 +1673,8 @@ static void htable_rpc_seti(rpc_t *rpc, void *c) int_str keyvalue; ht_t *ht; - if(rpc->scan(c, "SS.d", &htname, &keyname, &keyvalue.n) < 3) { + keyvalue.n = 0; + if(rpc->scan(c, "SSl", &htname, &keyname, &keyvalue.n) < 3) { rpc->fault(c, 500, "Not enough parameters (htable name, key name and value)"); return; @@ -1710,7 +1711,7 @@ static void htable_rpc_setex(rpc_t *rpc, void *c) int exval; ht_t *ht; - if(rpc->scan(c, "SS.d", &htname, &itname, &exval) < 3) { + if(rpc->scan(c, "SSd", &htname, &itname, &exval) < 3) { rpc->fault(c, 500, "Not enough parameters (htable name, item name and expire)"); return; @@ -1761,10 +1762,10 @@ static void htable_rpc_setxs(rpc_t *rpc, void *c) static void htable_rpc_setxi(rpc_t *rpc, void *c) { str htname, keyname; - int ival; + int ival = 0; int exval; - if(rpc->scan(c, "SS.dd", &htname, &keyname, &ival, &exval) < 4) { + if(rpc->scan(c, "SSdd", &htname, &keyname, &ival, &exval) < 4) { rpc->fault(c, 500, "Not enough parameters (htable name, key name, value and " "expire)");

1 0

git:master:3f4844b4: siputils: reformat module exported structures
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 3f4844b4ad1190a38eff8b377c89a9554be80e59 URL: https://github.com/kamailio/kamailio/commit/3f4844b4ad1190a38eff8b377c89a95… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-09T14:07:18+01:00 siputils: reformat module exported structures --- Modified: src/modules/siputils/siputils.c --- Diff: https://github.com/kamailio/kamailio/commit/3f4844b4ad1190a38eff8b377c89a95… Patch: https://github.com/kamailio/kamailio/commit/3f4844b4ad1190a38eff8b377c89a95…

1 0

git:master:0fcc950d: siputils: added module prefix to global variables
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 0fcc950dc54990a51fe74616ec2d2972a4e1c094 URL: https://github.com/kamailio/kamailio/commit/0fcc950dc54990a51fe74616ec2d297… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-09T14:07:18+01:00 siputils: added module prefix to global variables --- Modified: src/modules/siputils/chargingvector.c --- Diff: https://github.com/kamailio/kamailio/commit/0fcc950dc54990a51fe74616ec2d297… Patch: https://github.com/kamailio/kamailio/commit/0fcc950dc54990a51fe74616ec2d297…

1 0

git:master:52e82aa7: gcrypt: new module for crypto functions
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 52e82aa79febcdc7aa422756e250d402b61dfe14 URL: https://github.com/kamailio/kamailio/commit/52e82aa79febcdc7aa422756e250d40… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-08T20:21:02+01:00 gcrypt: new module for crypto functions - uses the GNU libgcrypt library - implemented AES256 ECB encrypt/decrypt functions --- Added: src/modules/gcrypt/Makefile Added: src/modules/gcrypt/README Added: src/modules/gcrypt/doc/Makefile Added: src/modules/gcrypt/doc/gcrypt.xml Added: src/modules/gcrypt/doc/gcrypt_admin.xml Added: src/modules/gcrypt/gcrypt_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/52e82aa79febcdc7aa422756e250d40… Patch: https://github.com/kamailio/kamailio/commit/52e82aa79febcdc7aa422756e250d40…

2 1

[kamailio/kamailio] sip_trace(address, "", "t") uses the duplicate_uri for replies from auth_challenge instead of address argument in sip_trace() (Issue #3700)
by Niklas Larsson 09 Jan '24

09 Jan '24

### Description Using sip_trace() with different address arguments: ``` modparam("siptrace", "duplicate_uri", "sip:X.X.X.X:9060") # siptrace till Homer if(method == "REGISTER"){ sip_trace("sip:X.X.X.X:9060", "", "t"); } # siptrace till Voipmonitor if (is_method("INVITE|BYE|UPDATE|CANCEL")) { sip_trace("sip:Y.Y.Y.Y:9060", "", "t"); } ``` Will send replies from auth_challenge to the address defined in duplicate_uri. So in the case above - no 407 are sent to "sip:Y.Y.Y.Y:9060", instead the are sent to "sip:X.X.X.X:9060" when challenging an Invite. ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.6.5 (x86_64/linux) flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLOCKLIST, HAVE_RESOLV_RES, TLS_PTHREAD_MUTEX_SHARED ADAPTIVE_WAIT_LOOPS 1024, MAX_RECV_BUFFER_SIZE 262144, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. id: unknown compiled with gcc 8.3.0 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3700 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3700(a)github.com>

3 4

[kamailio/kamailio] error on module (Issue #3704)
by faratell 09 Jan '24

09 Jan '24

Hello. I am trying to install Kamailio branch devel for using Volte on open5gs. when compiled have this error: gcrypt_mod.c:36:10: fatal error: gcrypt.h: No such file or directory 36 | #include <gcrypt.h> | ^~~~~~~~~~ compilation terminated. make[2]: *** [../../Makefile.rules:100: gcrypt_mod.o] Error 1 make[2]: Leaving directory '/usr/local/src/kamailio-devel/kamailio/src/modules/gcrypt' make[1]: *** [Makefile:508: modules] Error 1 make[1]: Leaving directory '/usr/local/src/kamailio-devel/kamailio/src' make: *** [Makefile:34: all] Error 2 I have installed the dependencies of crypt : libcrypto and libssl-dev. -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3704 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3704(a)github.com>

2 2

git:master:587c3e33: modules: readme files regenerated - gcrypt ... [skip ci]
by Kamailio Dev 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 587c3e337c3cbabed1373b9308065d59392a370d URL: https://github.com/kamailio/kamailio/commit/587c3e337c3cbabed1373b9308065d5… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-01-09T12:17:06+01:00 modules: readme files regenerated - gcrypt ... [skip ci] --- Modified: src/modules/gcrypt/README --- Diff: https://github.com/kamailio/kamailio/commit/587c3e337c3cbabed1373b9308065d5… Patch: https://github.com/kamailio/kamailio/commit/587c3e337c3cbabed1373b9308065d5… --- diff --git a/src/modules/gcrypt/README b/src/modules/gcrypt/README index 7c4826b849b..3d7f9af4226 100644 --- a/src/modules/gcrypt/README +++ b/src/modules/gcrypt/README @@ -26,7 +26,8 @@ Daniel-Constantin Mierla 3. Parameters 3.1. init_vector (str) - 3.2. register_callid (int) + 3.2. aes_mode (int) + 3.3. register_callid (int) 4. Functions @@ -36,9 +37,10 @@ Daniel-Constantin Mierla List of Examples 1.1. Set init_vector parameter - 1.2. Set register_callid parameter - 1.3. gcrypt_aes_encrypt usage - 1.4. gcrypt_aes_decrypt usage + 1.2. Set aes_mode parameter + 1.3. Set register_callid parameter + 1.4. gcrypt_aes_encrypt usage + 1.5. gcrypt_aes_decrypt usage Chapter 1. Admin Guide @@ -53,7 +55,8 @@ Chapter 1. Admin Guide 3. Parameters 3.1. init_vector (str) - 3.2. register_callid (int) + 3.2. aes_mode (int) + 3.3. register_callid (int) 4. Functions @@ -85,7 +88,8 @@ Chapter 1. Admin Guide 3. Parameters 3.1. init_vector (str) - 3.2. register_callid (int) + 3.2. aes_mode (int) + 3.3. register_callid (int) 3.1. init_vector (str) @@ -99,7 +103,19 @@ Chapter 1. Admin Guide modparam("gcrypt", "init_vector", "abcdefghijklmnop") ... -3.2. register_callid (int) +3.2. aes_mode (int) + + Set it to 1 in order to do AES CBC encryption mode. By default, it does + AES ECB mode. + + Default value is 0. + + Example 1.2. Set aes_mode parameter +... +modparam("gcrypt", "aes_mode", 1) +... + +3.3. register_callid (int) Set it to 1 in order to register a callback to core for generation of callid values for requests generated by Kamailio tm module. @@ -111,7 +127,7 @@ modparam("gcrypt", "init_vector", "abcdefghijklmnop") Default value is 0. - Example 1.2. Set register_callid parameter + Example 1.3. Set register_callid parameter ... modparam("gcrypt", "register_callid", 1) ... @@ -130,7 +146,7 @@ modparam("gcrypt", "register_callid", 1) This function can be used from ANY_ROUTE. - Example 1.3. gcrypt_aes_encrypt usage + Example 1.4. gcrypt_aes_encrypt usage ... gcrypt_aes_encrypt("$rb", "my-secret-key", "$var(encrypted)"); ... @@ -144,7 +160,7 @@ gcrypt_aes_encrypt("$rb", "my-secret-key", "$var(encrypted)"); This function can be used from ANY_ROUTE. - Example 1.4. gcrypt_aes_decrypt usage + Example 1.5. gcrypt_aes_decrypt usage ... gcrypt_aes_decrypt("$var(encrypted)", "my-secret-key", "$var(text)"); ...

1 0

git:master:33a0eeb4: gcrypt: docs for aes_mode parameters
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 33a0eeb40cb6be28c7921ccb61dcede1ebce4bad URL: https://github.com/kamailio/kamailio/commit/33a0eeb40cb6be28c7921ccb61dcede… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-09T12:00:41+01:00 gcrypt: docs for aes_mode parameters --- Modified: src/modules/gcrypt/doc/gcrypt_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/33a0eeb40cb6be28c7921ccb61dcede… Patch: https://github.com/kamailio/kamailio/commit/33a0eeb40cb6be28c7921ccb61dcede… --- diff --git a/src/modules/gcrypt/doc/gcrypt_admin.xml b/src/modules/gcrypt/doc/gcrypt_admin.xml index b88781425cf..533ec5c80f9 100644 --- a/src/modules/gcrypt/doc/gcrypt_admin.xml +++ b/src/modules/gcrypt/doc/gcrypt_admin.xml @@ -75,6 +75,27 @@ modparam("gcrypt", "init_vector", "abcdefghijklmnop") </programlisting> </example> </section> + <section id="gcrypt.p.aes_mode"> + <title><varname>aes_mode</varname> (int)</title> + <para> + Set it to 1 in order to do AES CBC encryption mode. By default, + it does AES ECB mode. + </para> + <para> + <emphasis> + Default value is 0. + </emphasis> + </para> + <example> + <title>Set <varname>aes_mode</varname> parameter</title> + <programlisting format="linespecific"> +... +modparam("gcrypt", "aes_mode", 1) +... +</programlisting> + </example> + </section> + <section id="gcrypt.p.register_callid"> <title><varname>register_callid</varname> (int)</title> <para>

1 0

git:master:2024253f: gcrypt: added aes_mode parameter
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 2024253fac4c4fda48f2b0c5b9a1f9717294aa22 URL: https://github.com/kamailio/kamailio/commit/2024253fac4c4fda48f2b0c5b9a1f97… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-09T11:56:50+01:00 gcrypt: added aes_mode parameter - specify AES encryption mode: - 0: ECB (GCRY_CIPHER_MODE_ECB) - default - 1: CBC (GCRY_CIPHER_MODE_CBC) --- Modified: src/modules/gcrypt/gcrypt_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/2024253fac4c4fda48f2b0c5b9a1f97… Patch: https://github.com/kamailio/kamailio/commit/2024253fac4c4fda48f2b0c5b9a1f97… --- diff --git a/src/modules/gcrypt/gcrypt_mod.c b/src/modules/gcrypt/gcrypt_mod.c index 1554154d9cc..8a5550139c8 100644 --- a/src/modules/gcrypt/gcrypt_mod.c +++ b/src/modules/gcrypt/gcrypt_mod.c @@ -53,6 +53,8 @@ static int fixup_gcrypt_aes_decrypt(void **param, int param_no); /* init vector value */ static str _gcrypt_init_vector = str_init("SIP/2.0 is RFC3261"); static int _gcrypt_register_callid = 0; +static int _gcrypt_aes_mode_param = 0; +static int _gcrypt_aes_mode = GCRY_CIPHER_MODE_ECB; /* clang-format off */ static cmd_export_t cmds[] = { @@ -66,6 +68,7 @@ static cmd_export_t cmds[] = { static param_export_t params[] = { {"init_vector", PARAM_STR, &_gcrypt_init_vector}, {"register_callid", PARAM_INT, &_gcrypt_register_callid}, + {"aes_mode", PARAM_INT, &_gcrypt_aes_mode_param}, {0, 0, 0} }; @@ -100,6 +103,9 @@ static int mod_init(void) } LM_DBG("registered crypto callid callback\n"); } + if(_gcrypt_aes_mode_param == 1) { + _gcrypt_aes_mode = GCRY_CIPHER_MODE_CBC; + } return 0; } @@ -140,7 +146,7 @@ static int ki_gcrypt_aes_encrypt_helper( gcry_ret = gcry_cipher_open(&cipher_hd, // gcry_cipher_hd_t *hd GCRY_CIPHER_AES256, // int algo - GCRY_CIPHER_MODE_ECB, // int mode + _gcrypt_aes_mode, // int mode 0); // unsigned int flags if(gcry_ret) { LM_ERR("gcry cipher open failed: %s/%s\n", gcry_strsource(gcry_ret), @@ -306,7 +312,7 @@ static int ki_gcrypt_aes_decrypt_helper( gcry_ret = gcry_cipher_open(&cipher_hd, // gcry_cipher_hd_t *hd GCRY_CIPHER_AES256, // int algo - GCRY_CIPHER_MODE_ECB, // int mode + _gcrypt_aes_mode, // int mode 0); // unsigned int flags if(gcry_ret) { LM_ERR("gcry cipher open failed: %s/%s\n", gcry_strsource(gcry_ret),

1 0

git:master:977e4f3e: modules: readme files regenerated - gcrypt ... [skip ci]
by Kamailio Dev 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 977e4f3e4d2a002f8aed55035c4ab6c3ee9ea1b8 URL: https://github.com/kamailio/kamailio/commit/977e4f3e4d2a002f8aed55035c4ab6c… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2024-01-09T11:02:08+01:00 modules: readme files regenerated - gcrypt ... [skip ci] --- Modified: src/modules/gcrypt/README --- Diff: https://github.com/kamailio/kamailio/commit/977e4f3e4d2a002f8aed55035c4ab6c… Patch: https://github.com/kamailio/kamailio/commit/977e4f3e4d2a002f8aed55035c4ab6c… --- diff --git a/src/modules/gcrypt/README b/src/modules/gcrypt/README index f836521537c..7c4826b849b 100644 --- a/src/modules/gcrypt/README +++ b/src/modules/gcrypt/README @@ -26,6 +26,7 @@ Daniel-Constantin Mierla 3. Parameters 3.1. init_vector (str) + 3.2. register_callid (int) 4. Functions @@ -35,8 +36,9 @@ Daniel-Constantin Mierla List of Examples 1.1. Set init_vector parameter - 1.2. gcrypt_aes_encrypt usage - 1.3. gcrypt_aes_decrypt usage + 1.2. Set register_callid parameter + 1.3. gcrypt_aes_encrypt usage + 1.4. gcrypt_aes_decrypt usage Chapter 1. Admin Guide @@ -51,6 +53,7 @@ Chapter 1. Admin Guide 3. Parameters 3.1. init_vector (str) + 3.2. register_callid (int) 4. Functions @@ -82,6 +85,7 @@ Chapter 1. Admin Guide 3. Parameters 3.1. init_vector (str) + 3.2. register_callid (int) 3.1. init_vector (str) @@ -92,7 +96,24 @@ Chapter 1. Admin Guide Example 1.1. Set init_vector parameter ... -modparam("crypto", "init_vector", "abcdefghijklmnop") +modparam("gcrypt", "init_vector", "abcdefghijklmnop") +... + +3.2. register_callid (int) + + Set it to 1 in order to register a callback to core for generation of + callid values for requests generated by Kamailio tm module. + + This callid generator uses libcrypt random and hashing functions for + generating RFC 4122 version 4 UUID with high quality entropy. It is + useful when wanting to have new callids that cannot be predicted from + previous values. + + Default value is 0. + + Example 1.2. Set register_callid parameter +... +modparam("gcrypt", "register_callid", 1) ... 4. Functions @@ -109,7 +130,7 @@ modparam("crypto", "init_vector", "abcdefghijklmnop") This function can be used from ANY_ROUTE. - Example 1.2. gcrypt_aes_encrypt usage + Example 1.3. gcrypt_aes_encrypt usage ... gcrypt_aes_encrypt("$rb", "my-secret-key", "$var(encrypted)"); ... @@ -123,7 +144,7 @@ gcrypt_aes_encrypt("$rb", "my-secret-key", "$var(encrypted)"); This function can be used from ANY_ROUTE. - Example 1.3. gcrypt_aes_decrypt usage + Example 1.4. gcrypt_aes_decrypt usage ... gcrypt_aes_decrypt("$var(encrypted)", "my-secret-key", "$var(text)"); ...

1 0

git:master:4d969662: gcrypt: docs for register_callid parameter
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: 4d96966272761816d16b40dc68adf3b24e79fbaf URL: https://github.com/kamailio/kamailio/commit/4d96966272761816d16b40dc68adf3b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-09T10:52:24+01:00 gcrypt: docs for register_callid parameter --- Modified: src/modules/gcrypt/doc/gcrypt_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/4d96966272761816d16b40dc68adf3b… Patch: https://github.com/kamailio/kamailio/commit/4d96966272761816d16b40dc68adf3b… --- diff --git a/src/modules/gcrypt/doc/gcrypt_admin.xml b/src/modules/gcrypt/doc/gcrypt_admin.xml index 523364318d5..b88781425cf 100644 --- a/src/modules/gcrypt/doc/gcrypt_admin.xml +++ b/src/modules/gcrypt/doc/gcrypt_admin.xml @@ -70,7 +70,33 @@ <title>Set <varname>init_vector</varname> parameter</title> <programlisting format="linespecific"> ... -modparam("crypto", "init_vector", "abcdefghijklmnop") +modparam("gcrypt", "init_vector", "abcdefghijklmnop") +... +</programlisting> + </example> + </section> + <section id="gcrypt.p.register_callid"> + <title><varname>register_callid</varname> (int)</title> + <para> + Set it to 1 in order to register a callback to core for generation + of callid values for requests generated by &kamailio; tm module. + </para> + <para> + This callid generator uses libcrypt random and hashing functions + for generating RFC 4122 version 4 UUID with high quality entropy. + It is useful when wanting to have new callids that cannot be + predicted from previous values. + </para> + <para> + <emphasis> + Default value is 0. + </emphasis> + </para> + <example> + <title>Set <varname>register_callid</varname> parameter</title> + <programlisting format="linespecific"> +... +modparam("gcrypt", "register_callid", 1) ... </programlisting> </example>

1 0

git:master:fae3632a: gcrypt: added functions to generate random call-id in uuid format
by Daniel-Constantin Mierla 09 Jan '24

09 Jan '24

Module: kamailio Branch: master Commit: fae3632a224cde753c2a6e3d1d2a72b6e9aa9ceb URL: https://github.com/kamailio/kamailio/commit/fae3632a224cde753c2a6e3d1d2a72b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2024-01-09T10:49:48+01:00 gcrypt: added functions to generate random call-id in uuid format - alternative to the option offered by crypto module --- Added: src/modules/gcrypt/gcrypt_uuid.c Added: src/modules/gcrypt/gcrypt_uuid.h Modified: src/modules/gcrypt/gcrypt_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/fae3632a224cde753c2a6e3d1d2a72b… Patch: https://github.com/kamailio/kamailio/commit/fae3632a224cde753c2a6e3d1d2a72b…

1 0

[kamailio/kamailio] error on make cdp module (Issue #3701)
by faratell 09 Jan '24

09 Jan '24

Hello. I have tried to install VoLTE Setup with Kamailio IMS on Ubuntu 22.04. but I face two errors: 1) IPsec-tools packages not supported for Ubuntu 22.04. 2) when I tried to compile Kamailio the cdp modules were not compiled and showed this error: make[2]: Entering directory '/usr/local/src/kamailio/src/modules/cdp' Makefile.defs defs skipped gcc -shared -Wl,-O2 -Wl,-E -pthread -rdynamic -ldl -Wl,-Bsymbolic-functions acceptor.o acctstatemachine.o api_process.o authstatemachine.o cdp_functions.o cdp_load.o cdp_mod.o cdp_rpc.o cdp_stats.o common.o config.o configparser.o diameter_avp.o diameter_comm.o diameter_msg.o diameter_peer.o globals.o peer.o peermanager.o peerstatemachine.o receiver.o routing.o session.o tcp_accept.o timer.o transaction.o worker.o -lxml2 -lrt -lpthread -o cdp.so /usr/bin/ld: cdp_mod.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: multiple definition of `dp_first_pid'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: first defined here /usr/bin/ld: cdp_mod.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: multiple definition of `pid_list'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: first defined here /usr/bin/ld: cdp_mod.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: multiple definition of `pid_list_lock'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: first defined here /usr/bin/ld: diameter_peer.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: multiple definition of `dp_first_pid'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: first defined here /usr/bin/ld: diameter_peer.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: multiple definition of `pid_list'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: first defined here /usr/bin/ld: diameter_peer.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: multiple definition of `pid_list_lock'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: first defined here /usr/bin/ld: receiver.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: multiple definition of `dp_first_pid'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: first defined here /usr/bin/ld: receiver.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: multiple definition of `pid_list'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: first defined here /usr/bin/ld: receiver.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: multiple definition of `pid_list_lock'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: first defined here /usr/bin/ld: timer.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: multiple definition of `dp_first_pid'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: first defined here /usr/bin/ld: timer.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: multiple definition of `pid_list'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: first defined here /usr/bin/ld: timer.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: multiple definition of `pid_list_lock'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: first defined here /usr/bin/ld: worker.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: multiple definition of `dp_first_pid'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:68: first defined here /usr/bin/ld: worker.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: multiple definition of `pid_list'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:70: first defined here /usr/bin/ld: worker.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: multiple definition of `pid_list_lock'; acceptor.o:/usr/local/src/kamailio/src/modules/cdp/diameter_peer.h:71: first defined here collect2: error: ld returned 1 exit status make[2]: *** [../../Makefile.rules:191: cdp.so] Error 1 make[2]: Leaving directory '/usr/local/src/kamailio/src/modules/cdp' make[1]: *** [Makefile:511: modules] Error 1 make[1]: Leaving directory '/usr/local/src/kamailio/src' make: *** [Makefile:34: all] Error 2 How can I solve it? -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3701 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3701(a)github.com>

4 9

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev