sr-dev May 2022

sr-dev@lists.kamailio.org

12 participants
285 discussions

[kamailio/kamailio] crash related to functions in cfg file parser for invalid cfg (#2239)

by Henning Westerholt

Against current git master. ``` henning@static:~/repositories/kamailio$ cat etc/crash-4.cfg #!KAMAILIO loadmodule "sl.so" request_route { switch (1) { case -1: send_reply("404", "Not Found"); case -2: send_reply("405",("Method Not Allowed"); exit; } ``` > $ ./src/kamailio -f etc/crash-4.cfg -L src/modules/ -Y /tmp -E -d > 0(2278) CRITICAL: <core> [core/cfg.y:3549]: yyerror_at(): parse error in config file /home/henning/repositories/kamailio/etc/crash-4.cfg, line 10, column 43: syntax error > 0(2278) CRITICAL: <core> [core/cfg.y:3549]: yyerror_at(): parse error in config file /home/henning/repositories/kamailio/etc/crash-4.cfg, line 10, column 43: '('')' expected (function call) > Segmentation fault (core dumped) > henning@static:~/repositories/kamailio$ gdb src/kamailio core > Reading symbols from src/kamailio...done. > [New LWP 2278] > Core was generated by `./src/kamailio -f etc/crash-4.cfg -L src/modules/ -Y /tmp -E -d'. > Program terminated with signal SIGSEGV, Segmentation fault. > #0 0x00005580cdbd131e in append_action (a=0x7fad84db9300, b=0x7fad84db97f8) at core/route_struct.c:147 > 147 for(t=a; t->next; t=t->next); > (gdb) bt > #0 0x00005580cdbd131e in append_action (a=0x7fad84db9300, b=0x7fad84db97f8) at core/route_struct.c:147 > #1 0x00005580cdf9208b in yyparse () at core/cfg.y:2428 > #2 0x00005580cdb96c11 in main (argc=9, argv=0x7ffc2d9f6a68) at main.c:2309 > (gdb) f 0 > #0 0x00005580cdbd131e in append_action (a=0x7fad84db9300, b=0x7fad84db97f8) at core/route_struct.c:147 > 147 for(t=a; t->next; t=t->next); > (gdb) p a > $1 = (struct action *) 0x7fad84db9300 > (gdb) p b > $2 = (struct action *) 0x7fad84db97f8 > (gdb) p t > $3 = (struct action *) 0x80 > (gdb) p a->next > $4 = (struct action *) 0x80 > (gdb) p *t > Cannot access memory at address 0x80 -- 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/2239

2 years, 6 months

[kamailio/kamailio] KEMI Fatal Python error: failed releasing import lock after fork (Issue #3125)

by Nigel Daniels

### Description using kamailio 5.6.0 python: 3.8.10 OS : ubuntu 20.0.4 When trying to use kemi with the default configuration I get on startup. ` Fatal Python error: failed releasing import lock after fork ` #### Reproduction Using The configuration described here : https://kamailio.org/docs/tutorials/devel/kamailio-kemi-framework/kemi/#bas… #### Debugging Data ``` 0(1) INFO: <core> [core/kemi.c:106]: sr_kemi_core_info(): ===== kamailio.child_init(1) kamailio | 0(1) DEBUG: <core> [core/sr_module.c:923]: init_child(): initializing PROC_POSTCHILDINIT with rank -126 kamailio | 0(1) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 0 rank -126: app_python3 [stand-alone receiver @ 10.10.0.4:5060] kamailio | Fatal Python error: failed releasing import lock after fork kamailio | Python runtime state: initialized kamailio | kamailio | Current thread 0x00007f3faa857180 (most recent call first): kamailio | <no Python frame> kamailio | 7(14) DEBUG: <core> [core/pt.c:308]: fork_process(): seeding PRNG with 1232772928 kamailio | 7(14) DEBUG: <core> [core/rand/cryptorand.c:36]: cryptorand_seed(): seeding cryptorand generator with 1232772928 kamailio | 7(14) DEBUG: <core> [core/rand/fortuna/random.c:175]: sr_add_entropy(): additional 4 bytes entropy added to cryptographic PRNG kamailio | 7(14) DEBUG: <core> [core/pt.c:313]: fork_process(): test random numbers 987270226 561754816 1532392054 1162699744 kamailio | 7(14) DEBUG: <core> [core/sr_module.c:923]: init_child(): initializing PROC_RPC with rank -2 kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: corex [ctl handler] kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: usrloc [ctl handler] kamailio | 7(14) DEBUG: <core> [core/utils/sruid.c:121]: sruid_init(): root for sruid is [ulcx-62931156-e-] (0 / 16) kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: registrar [ctl handler] kamailio | 7(14) DEBUG: <core> [core/utils/sruid.c:121]: sruid_init(): root for sruid is [uloc-62931156-e-] (0 / 16) kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: jsonrpcs [ctl handler] kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: kex [ctl handler] kamailio | 7(14) DEBUG: kex [kex_mod.c:155]: child_init(): rank is (-2) kamailio | 7(14) DEBUG: <core> [core/utils/sruid.c:121]: sruid_init(): root for sruid is [srid-62931156-e-] (0 / 16) kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: tm [ctl handler] kamailio | 7(14) DEBUG: tm [callid.c:136]: child_init_callid(): callid: '2ef035de7699493f-14(a)10.10.0.4' kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: tmx [ctl handler] kamailio | 7(14) DEBUG: tmx [tmx_mod.c:262]: child_init(): rank is (-2) kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: sl [ctl handler] kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: ctl [ctl handler] kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: debugger [ctl handler] kamailio | 7(14) DEBUG: debugger [debugger_mod.c:224]: child_init(): rank is (-2) kamailio | 7(14) DEBUG: <core> [core/sr_module.c:875]: init_mod_child(): idx 7 rank -2: app_python3 [ctl handler] kamailio | 7(14) DEBUG: app_python3 [apy_kemi.c:234]: sr_apy_kemi_exec_func_ex(): execution of method: .info kamailio | 7(14) DEBUG: app_python3 [apy_kemi.c:266]: sr_apy_kemi_exec_func_ex(): param[1] for: info is str: ===== kamailio.child_init(-2) kamailio | kamailio | 7(14) INFO: <core> [core/kemi.c:106]: sr_kemi_core_info(): ===== kamailio.child_init(-2) kamailio | 7(14) DEBUG: ctl [ctl.c:329]: mod_child(): ctl: mod_child(0), fork_process=0, csl=0x56186255e180 kamailio | 7(14) DEBUG: ctl [ctl.c:336]: mod_child(): ctl: 0 io_listen_loop(1, 0x56186255e180) kamailio | 7(14) INFO: ctl [io_listener.c:213]: io_listen_loop(): io_listen_loop: using epoll_lt io watch method (config) kamailio | 7(14) DEBUG: ctl [io_listener.c:245]: io_listen_loop(): io_listen_loop: adding socket 8, type 2, transport 3 (/var/run/kamailio//kamailio_ctl) kamailio | 7(14) DEBUG: ctl [../../core/io_wait.h:374]: io_watch_add(): processing io_watch_add(0x7f3fa8aa2380, 8, 2, 0x56186255e180) - fd_no=0 kamailio exited with code 139 ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3125 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3125(a)github.com>

2 years, 6 months

[kamailio/kamailio] auth: Add pv_auth_check overload with second passwd parameter (PR #3126)

by Jon Bergli Heier

#### 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 change modifies the `pv_auth_check` function to take an optional second password argument `passwd2`. The motivation for this change is to allow authenticating against two passwords, which is useful if you want to eg. rotate passwords periodically and still allow using the previous password during a transition period. You can technically do this with two calls to `pv_auth_check`, but with `nonce_count` enabled the first call will invalidate nc which will always cause the second call to fail. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3126 -- Commit Summary -- * auth: Add pv_auth_check overload with second passwd parameter -- File Changes -- M src/modules/auth/auth_mod.c (143) M src/modules/auth/doc/auth_functions.xml (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3126.patch https://github.com/kamailio/kamailio/pull/3126.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3126 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3126(a)github.com>

2 years, 6 months

[kamailio/kamailio] tm: try next ip on receiving code other than 503 if dns failover (PR #3129)

by Mvondo Eric

#### Pre-Submission Checklist    - [ ] 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:  - [x] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description  By default when doing dns failover the tm module will try next ips only when receiving 503 code or no response. the new failover_reply_codes provides a way to try next ip when receiving specific codes or class code. for example try next ip even if the previous one returned 408 You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3129 -- Commit Summary -- * Enable tm dns failover to try next ip on receiving code other than 503 -- File Changes -- M src/modules/tm/doc/params.xml (25) M src/modules/tm/t_funcs.c (11) M src/modules/tm/t_reply.c (12) M src/modules/tm/t_reply.h (4) M src/modules/tm/tm.c (139) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3129.patch https://github.com/kamailio/kamailio/pull/3129.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3129 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3129(a)github.com>

2 years, 6 months

[kamailio/kamailio] ims_qos: fix compilation warning (PR #3127)

by Atanas Bakalov

You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3127 -- Commit Summary -- * ims_qos: fix compilation warning -- File Changes -- M src/modules/ims_qos/rx_aar.c (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3127.patch https://github.com/kamailio/kamailio/pull/3127.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3127 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3127(a)github.com>

2 years, 6 months

[kamailio/kamailio] DNS Failover try next destinations on receiving code other than 503 (Issue #3122)

by patrickarton

### Description All ips destination of DNS Failover are tried only if either no response or 503 response. I woud like to try others ips when receiving code other than 503 response (example when receiving 408 from a previous one).  ### Expected behavior #### Actual observed behavior #### Debugging Data ``` None ``` #### Log Messages  ``` None ``` #### SIP Traffic  ``` obvious ``` ### Possible Solutions A possible solution will be to modify the reply_received function of tm's t_reply.c file. ``` int reply_received( struct sip_msg *p_msg ) { } ```  ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` (paste your output here) ``` * **Operating System**:  ``` (paste your output here) ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3122 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3122(a)github.com>

2 years, 6 months

[kamailio/kamailio] 'kamcmd ul.dump' error on 5.6 (Issue #3123)

by MayamaTakeshi

### Description If there are no usrloc registrations, all is good: ``` [root@lab002194-flip-server install]$ kamcmd ul.dump { Domains: { Domain: { Domain: devflip_location Size: 1024 AoRs: { } Stats: { Records: 0 Max-Slots: 0 } } } } ```` But if there are registered SIP endpoints, we get: ``` [root@lab002194-flip-server install]$ kamcmd ul.dump { Domains: { Domain: { Domain: devflip_location Size: 1024 AoRs: { Info: { AoR: oweruaodsiflwosdfjoe(a)lsjfoasjdfosaodfsdfafolekodc.com HashID: -2013386516 Contacts: { Contact: { Address: sip:oweruaodsiflwosdfjoe@10.255.255.195:6060 Expires: 96 Q: 0 ERROR:: while parsing the record 14, @140: ff : bad record ``` #### Reproduction Just register some SIP endpoints against kamailio and issue 'kamcmd ul.dump' ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.6.0 (x86_64/linux) 61e86a 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: 61e86a compiled on 06:54:55 May 26 2022 with gcc 10.2.1 ``` * **Operating System**:  ``` [root@lab002194-flip-server kamailio-5.6]$ lsb_release -a No LSB modules are available. Distributor ID: Debian Description: Debian GNU/Linux 11 (bullseye) Release: 11 Codename: bullseye [root@lab002194-flip-server kamailio-5.6]$ uname -a Linux lab002194-flip-server 5.10.0-14-amd64 #1 SMP Debian 5.10.113-1 (2022-04-29) x86_64 GNU/Linux ``` -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/issues/3123 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/issues/3123(a)github.com>

2 years, 6 months

[kamailio/kamailio] Fix Makefile.groups and SLACK for build on FreeBSD (PR #3130)

by Boris Korzun

#### 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) - [ ] 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 - [X] Related to issue #3091 #### Description `module_group_standard` and `slack` are not build on FreeBSD environment. The proposed patch fixes it and makes the module is consistent with the rest of modules. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/3130 -- Commit Summary -- * Makefile.groups: renamed malloc_test to misctest * slack: Makefile - support for FreeBSD build -- File Changes -- M src/Makefile.groups (2) M src/modules/slack/Makefile (16) -- Patch Links -- https://github.com/kamailio/kamailio/pull/3130.patch https://github.com/kamailio/kamailio/pull/3130.diff -- Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/3130 You are receiving this because you are subscribed to this thread. Message ID: <kamailio/kamailio/pull/3130(a)github.com>

2 years, 6 months

git:master:afcab24d: Merge pull request #3130 from drTr0jan/fix-freebsd

by GitHub

Module: kamailio Branch: master Commit: afcab24d999fa999fbc99cda607423cde9fa8714 URL: https://github.com/kamailio/kamailio/commit/afcab24d999fa999fbc99cda607423c… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2022-05-31T18:43:31+02:00 Merge pull request #3130 from drTr0jan/fix-freebsd Fix Makefile.groups and SLACK for build on FreeBSD --- Modified: src/Makefile.groups Modified: src/modules/slack/Makefile --- Diff: https://github.com/kamailio/kamailio/commit/afcab24d999fa999fbc99cda607423c… Patch: https://github.com/kamailio/kamailio/commit/afcab24d999fa999fbc99cda607423c… --- diff --git a/src/Makefile.groups b/src/Makefile.groups index c388489a66..bd111ca7b5 100644 --- a/src/Makefile.groups +++ b/src/Makefile.groups @@ -38,7 +38,7 @@ mod_list_dbuid=db2_ops uid_auth_db uid_avp_db uid_domain uid_gflags \ uid_uri_db # - modules for devel purposes -mod_list_devel=malloc_test print print_lib +mod_list_devel=misctest print print_lib # - modules depending on pcre3 library mod_list_pcre=dialplan lcr regex diff --git a/src/modules/slack/Makefile b/src/modules/slack/Makefile index 9bc748a2e5..a4f0c1ff81 100644 --- a/src/modules/slack/Makefile +++ b/src/modules/slack/Makefile @@ -8,5 +8,21 @@ include ../../Makefile.defs auto_gen= NAME=slack.so +ifeq ($(CROSS_COMPILE),) +CURL_BUILDER=$(shell \ + if pkg-config --exists libcurl; then \ + echo 'pkg-config libcurl'; \ + else \ + which curl-config; \ + fi) +endif + +ifneq ($(CURL_BUILDER),) + DEFS += $(shell $(CURL_BUILDER) --cflags ) + LIBS += $(shell $(CURL_BUILDER) --libs) +else + DEFS+=-I$(LOCALBASE)/include + LIBS+=-L$(LOCALBASE)/lib -lcurl +endif include ../../Makefile.modules

2 years, 6 months

git:master:36baa8cd: slack: Makefile - support for FreeBSD build

by drTr0jan

Module: kamailio Branch: master Commit: 36baa8cda606e107308303c33c5b98cbb513c653 URL: https://github.com/kamailio/kamailio/commit/36baa8cda606e107308303c33c5b98c… Author: drTr0jan <drtr0jan(a)yandex.ru> Committer: drTr0jan <drtr0jan(a)yandex.ru> Date: 2022-05-31T19:08:29+03:00 slack: Makefile - support for FreeBSD build - added LIBS and DEFS variables, making them consistent with the rest of modules. --- Modified: src/modules/slack/Makefile --- Diff: https://github.com/kamailio/kamailio/commit/36baa8cda606e107308303c33c5b98c… Patch: https://github.com/kamailio/kamailio/commit/36baa8cda606e107308303c33c5b98c… --- diff --git a/src/modules/slack/Makefile b/src/modules/slack/Makefile index 9bc748a2e5..a4f0c1ff81 100644 --- a/src/modules/slack/Makefile +++ b/src/modules/slack/Makefile @@ -8,5 +8,21 @@ include ../../Makefile.defs auto_gen= NAME=slack.so +ifeq ($(CROSS_COMPILE),) +CURL_BUILDER=$(shell \ + if pkg-config --exists libcurl; then \ + echo 'pkg-config libcurl'; \ + else \ + which curl-config; \ + fi) +endif + +ifneq ($(CURL_BUILDER),) + DEFS += $(shell $(CURL_BUILDER) --cflags ) + LIBS += $(shell $(CURL_BUILDER) --libs) +else + DEFS+=-I$(LOCALBASE)/include + LIBS+=-L$(LOCALBASE)/lib -lcurl +endif include ../../Makefile.modules

2 years, 6 months

← Newer
1
...
8
9
10
11
12
13
14
...
29
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev May 2022