sr-dev November 2020

sr-dev@lists.kamailio.org

22 participants
311 discussions

[kamailio/kamailio] core on simple config file (#2562)

by sergey-safarov

### Description For test purpose, i created a config file with this content ``` listen=udp:eth0:11000 ``` When I start Kamailio using the command I got core ``` [centos@ip-100-86-32-7 ~]$ kamailio -f kamailio.cfg Segmentation fault (core dumped) [centos@ip-100-86-32-7 ~]$ ``` #### Debugging Data ``` (gdb) bt full #0 0x0000000000000000 in ?? () No symbol table info available. #1 0x00000000005979fb in ksr_shutdown_phase_init () at core/sr_module.c:118 __func__ = "ksr_shutdown_phase_init" #2 0x0000000000431181 in main (argc=3, argv=0x7fffffffe388) at main.c:2545 cfg_stream = 0xba02b0 c = -1 r = -7712 tmp = 0x9c0000009c0 <error: Cannot access memory at address 0x9c0000009c0> tmp_len = 32767 port = -134224128 proto = 0 ahost = 0x0 aport = 0 options = 0x7e3488 ":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 = 3173775974 rfd = 4 debug_save = 0 debug_flag = 0 dont_fork_cnt = 0 n_lst = 0x7ffff7ffe700 p = 0x7ffff70f9205 <handle_intel.constprop+197> "H\205\300uf\213t$\004M\215\216\244" st = {st_dev = 0, st_ino = 0, st_nlink = 0, st_mode = 0, st_uid = 0, st_gid = 0, __pad0 = 0, st_rdev = 0, st_size = 0, st_blksize = 0, st_blocks = 0, st_atim = { tv_sec = 0, tv_nsec = 0}, st_mtim = {tv_sec = 0, tv_nsec = 0}, st_ctim = {tv_sec = 0, tv_nsec = 0}, __glibc_reserved = {0, 0, 0}} tbuf = "|\343\006\367\377\177\000\000ء\377\367\377\177\000\000\000\000\000\000 ", '\000' <repeats 19 times>, " O\335\367\377\177\000\000`\340\377\377\377\177\000\000\a\000\000\000\000\000\000\000\330D\335\367\377\177\000\000\a\000\000\000\t\000\000\000\220\331\377\367\377\177\000\000\314\000\336\367\377\177\000\000\000\000\000\000\000\000\000\000\370\016\336\367\377\177\000\000\260\337\377\377\377\177\000\000\240\343\006\367\377\177\000\000\300\263\376\367\377\177\000\000\000\000\000\000\000\000\000\000\060\340\377\377\377\177", '\000' <repeats 26 times>, "HG\335\367\377\177\000\000\270\344\377\367\377\177\000\000\060\340\377\377\377\177\000\000\000"... option_index = 0 long_options = {{name = 0x7e55f6 "help", has_arg = 0, flag = 0x0, val = 104}, {name = 0x7e0bc1 "version", has_arg = 0, flag = 0x0, val = 118}, { name = 0x7e55fb "alias", has_arg = 1, flag = 0x0, val = 1024}, {name = 0x7e5601 "subst", has_arg = 1, flag = 0x0, val = 1025}, {name = 0x7e5607 "substdef", has_arg = 1, flag = 0x0, val = 1026}, {name = 0x7e5610 "substdefs", has_arg = 1, flag = 0x0, val = 1027}, {name = 0x7e561a "server-id", has_arg = 1, flag = 0x0, val = 1028}, {name = 0x7e5624 "loadmodule", has_arg = 1, flag = 0x0, val = 1029}, {name = 0x7e562f "modparam", has_arg = 1, flag = 0x0, val = 1030}, {name = 0x7e5638 "log-engine", has_arg = 1, flag = 0x0, val = 1031}, {name = 0x7e5643 "debug", has_arg = 1, flag = 0x0, val = 1032}, {name = 0x0, has_arg = 0, flag = 0x0, val = 0}} __func__ = "main" (gdb) info locals No symbol table info available. (gdb) list 1890 +((!sctp_disable)?sctp_listeners:0) 1891 #endif 1892 ; 1893 } 1894 1895 int main(int argc, char** argv) 1896 { 1897 1898 FILE* cfg_stream; 1899 int c,r; (gdb) ``` ### Additional Information * **Kamailio Version** - output of `kamailio -v` ``` [centos@ip-100-86-32-7 ~]$ kamailio -v version: kamailio 5.5.0-dev3 (x86_64/linux) e9624b 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: e9624b compiled on 04:01:18 Nov 10 2020 with gcc 8.3.1 ``` * **Operating System**: ``` [centos@ip-100-86-32-7 ~]$ cat /etc/os-release NAME="CentOS Linux" VERSION="8 (Core)" ID="centos" ID_LIKE="rhel fedora" VERSION_ID="8" PLATFORM_ID="platform:el8" PRETTY_NAME="CentOS Linux 8 (Core)" ANSI_COLOR="0;31" CPE_NAME="cpe:/o:centos:centos:8" HOME_URL="https://www.centos.org/" BUG_REPORT_URL="https://bugs.centos.org/" CENTOS_MANTISBT_PROJECT="CentOS-8" CENTOS_MANTISBT_PROJECT_VERSION="8" REDHAT_SUPPORT_PRODUCT="centos" REDHAT_SUPPORT_PRODUCT_VERSION="8" ``` -- 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/2562

3 years, 5 months

git:master:d4316319: tm: removed function name from log messages

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: d4316319225ba98e940b3631d00531eed6e31f9a URL: https://github.com/kamailio/kamailio/commit/d4316319225ba98e940b3631d00531e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-11-30T15:11:27+01:00 tm: removed function name from log messages - it is already in the prefix --- Modified: src/modules/tm/tm.c --- Diff: https://github.com/kamailio/kamailio/commit/d4316319225ba98e940b3631d00531e… Patch: https://github.com/kamailio/kamailio/commit/d4316319225ba98e940b3631d00531e… --- diff --git a/src/modules/tm/tm.c b/src/modules/tm/tm.c index 1beeee1053..06e315fe14 100644 --- a/src/modules/tm/tm.c +++ b/src/modules/tm/tm.c @@ -2913,7 +2913,7 @@ static int ki_t_relay_to_proto(sip_msg_t *msg, str *sproto) } else if (strncasecmp(sproto->s, "TLS", 3)) { proto = PROTO_TLS; } else { - LM_ERR("t_relay_to_proto failed, bad protocol specified <%s>\n", sproto->s); + LM_ERR("bad protocol specified <%s>\n", sproto->s); return E_UNSPEC; } } @@ -2937,7 +2937,7 @@ static int ki_t_relay_to_proto_addr(sip_msg_t *msg, str *sproto, str *host, unsi } else if (strncasecmp(sproto->s, "TLS", 3)) { proto = PROTO_TLS; } else { - LM_ERR("t_relay_to_proto2 failed, bad protocol specified <%s>\n", sproto->s); + LM_ERR("bad protocol specified <%s>\n", sproto->s); return E_UNSPEC; } }

3 years, 5 months

Re: [sr-dev] [kamailio/kamailio] tm: renamed kemi ki_t_relay_to_proto2() to ki_t_relay_to_proto_addr() (bd504b2)

by aalba6675

Thanks @miconda ! There is one more string (in error message) that probably should be changed as well. ``` LM_ERR("t_relay_to_proto2 failed, bad protocol specified <%s>\n", sproto->s); ``` -- 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/commit/bd504b2352d322e94072ff7915cc4ee…

3 years, 5 months

Re: [sr-dev] [kamailio/kamailio] Audio issue - STUN 401 Unauthorized with RTPEngine (#2567)

by Daniel-Constantin Mierla

This is obviously about how to use kamailio and building its config file kamailio.cfg, not a report related to a bug in the c code of kamailio. Use sr-users(a)lists.kamailio.org for such needs, that is the place for community support. The tracker here is only for bug reports or requests for new features related to the C code of Kamailio application and its related tools. -- 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/2567#issuecomment-735778825

3 years, 5 months

Re: [sr-dev] [kamailio/kamailio] Audio issue - STUN 401 Unauthorized with RTPEngine (#2567)

by Daniel-Constantin Mierla

Closed #2567. -- 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/2567#event-4051878245

3 years, 5 months

git:master:6376e249: dlgs: fix counting dialogs function

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 6376e24960878ca510ab6f1ce0594a351debc409 URL: https://github.com/kamailio/kamailio/commit/6376e24960878ca510ab6f1ce0594a3… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-11-30T14:08:49+01:00 dlgs: fix counting dialogs function - count the dialog in initial state - GH #2568 --- Modified: src/modules/dlgs/dlgs_mod.c Modified: src/modules/dlgs/dlgs_records.c --- Diff: https://github.com/kamailio/kamailio/commit/6376e24960878ca510ab6f1ce0594a3… Patch: https://github.com/kamailio/kamailio/commit/6376e24960878ca510ab6f1ce0594a3… --- diff --git a/src/modules/dlgs/dlgs_mod.c b/src/modules/dlgs/dlgs_mod.c index e26e0d9899..c7f60fcab8 100644 --- a/src/modules/dlgs/dlgs_mod.c +++ b/src/modules/dlgs/dlgs_mod.c @@ -250,13 +250,16 @@ static int w_dlgs_update(sip_msg_t *msg, char *p1, char *p2) */ static int ki_dlgs_count(sip_msg_t *msg, str *vfield, str *vop, str *vdata) { + int ret; + LM_DBG("counting by: [%.*s] [%.*s] [%.*s]\n", vfield->len, vfield->s, vop->len, vop->s, vdata->len, vdata->s); - if(dlgs_count(msg, vfield, vop, vdata) < 0) { - return -1; + ret = dlgs_count(msg, vfield, vop, vdata); + if(ret <= 0) { + return (ret-1); } - return 1; + return ret; } /** diff --git a/src/modules/dlgs/dlgs_records.c b/src/modules/dlgs/dlgs_records.c index 6b4a6b70e8..1f6af22ba3 100644 --- a/src/modules/dlgs/dlgs_records.c +++ b/src/modules/dlgs/dlgs_records.c @@ -340,6 +340,7 @@ int dlgs_add_item(sip_msg_t *msg, str *src, str *dst, str *data) prev->next = nitem; } dsht->slots[idx].esize++; + dlgs_update_stats(&dsht->slots[idx].astats, nitem->state, 1); lock_release(&dsht->slots[idx].lock); return 0; }

3 years, 5 months

git:master:bd504b23: tm: renamed kemi ki_t_relay_to_proto2() to ki_t_relay_to_proto_addr()

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: bd504b2352d322e94072ff7915cc4ee48260cccc URL: https://github.com/kamailio/kamailio/commit/bd504b2352d322e94072ff7915cc4ee… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-11-30T14:08:49+01:00 tm: renamed kemi ki_t_relay_to_proto2() to ki_t_relay_to_proto_addr() --- Modified: src/modules/tm/tm.c --- Diff: https://github.com/kamailio/kamailio/commit/bd504b2352d322e94072ff7915cc4ee… Patch: https://github.com/kamailio/kamailio/commit/bd504b2352d322e94072ff7915cc4ee… --- diff --git a/src/modules/tm/tm.c b/src/modules/tm/tm.c index e46c3fb057..1beeee1053 100644 --- a/src/modules/tm/tm.c +++ b/src/modules/tm/tm.c @@ -2923,7 +2923,7 @@ static int ki_t_relay_to_proto(sip_msg_t *msg, str *sproto) /** * */ -static int ki_t_relay_to_proto2(sip_msg_t *msg, str *sproto, str *host, unsigned int port) +static int ki_t_relay_to_proto_addr(sip_msg_t *msg, str *sproto, str *host, unsigned int port) { int proto = PROTO_NONE; @@ -3284,8 +3284,8 @@ static sr_kemi_t tm_kemi_exports[] = { { SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } }, - { str_init("tm"), str_init("t_relay_to_proto2"), - SR_KEMIP_INT, ki_t_relay_to_proto2, + { str_init("tm"), str_init("t_relay_to_proto_addr"), + SR_KEMIP_INT, ki_t_relay_to_proto_addr, { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_INT, SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE } },

3 years, 5 months

git:master:26b16dff: modules: readme files regenerated - lost ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: 26b16dff41822feb37dd574b7324e817a25e1aff URL: https://github.com/kamailio/kamailio/commit/26b16dff41822feb37dd574b7324e81… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2020-11-30T09:16:16+01:00 modules: readme files regenerated - lost ... [skip ci] --- Modified: src/modules/lost/README --- Diff: https://github.com/kamailio/kamailio/commit/26b16dff41822feb37dd574b7324e81… Patch: https://github.com/kamailio/kamailio/commit/26b16dff41822feb37dd574b7324e81… --- diff --git a/src/modules/lost/README b/src/modules/lost/README index 1cbd906d27..81a4fa4833 100644 --- a/src/modules/lost/README +++ b/src/modules/lost/README @@ -179,7 +179,7 @@ Chapter 1. Admin Guide Example 1.3. Set location_type parameter ... - modparam("http_client", "location_type, "civic geodetic locationURI") + modparam("lost", "location_type, "civic geodetic locationURI") ... 4. Functions @@ -221,7 +221,7 @@ $var(err)"); xlog("L_INFO", "HELD locationRequest: Result code $var(res)\nUrl: $var(url)\n$va r(pidf)"); ... -$var(res) = lost_held_query("heldsrv", "$var(pidf)", "$var(url)"", "$var(err)"); +$var(res) = lost_held_query("heldsrv", "$var(pidf)", "$var(url)", "$var(err)"); xlog("L_INFO", "HELD locationRequest: Result code $var(res)\nUrl: $var(url)\n$va r(pidf)\n"); ...

3 years, 5 months

[kamailio/kamailio] lost: fixed typo (#2566)

by sergey-safarov

#### 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 Fixed type in lost module example. Param `location_type`. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/2566 -- Commit Summary -- * lost: fixed typo -- File Changes -- M src/modules/lost/doc/lost_admin.xml (2) -- Patch Links -- https://github.com/kamailio/kamailio/pull/2566.patch https://github.com/kamailio/kamailio/pull/2566.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/2566

3 years, 5 months

git:master:60cf4502: Merge pull request #2566 from sergey-safarov/master

by GitHub

Module: kamailio Branch: master Commit: 60cf4502bbc55b4b8d246c3967dd8cae87e95c45 URL: https://github.com/kamailio/kamailio/commit/60cf4502bbc55b4b8d246c3967dd8ca… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2020-11-30T09:14:18+01:00 Merge pull request #2566 from sergey-safarov/master lost: fixed typo --- Modified: src/modules/lost/doc/lost_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/60cf4502bbc55b4b8d246c3967dd8ca… Patch: https://github.com/kamailio/kamailio/commit/60cf4502bbc55b4b8d246c3967dd8ca… --- diff --git a/src/modules/lost/doc/lost_admin.xml b/src/modules/lost/doc/lost_admin.xml index c8be8f668b..c5d4f5ad69 100644 --- a/src/modules/lost/doc/lost_admin.xml +++ b/src/modules/lost/doc/lost_admin.xml @@ -168,7 +168,7 @@ <title>Set <varname>location_type</varname> parameter</title> <programlisting format="linespecific"> ... - modparam("http_client", "location_type, "civic geodetic locationURI") + modparam("lost", "location_type, "civic geodetic locationURI") ... </programlisting> </example> @@ -228,7 +228,7 @@ $var(id) = "sip:alice@atlanta"; $var(res) = lost_held_query("heldsrv", "$var(id)" , "$var(pidf)", "$var(url)", "$var(err)"); xlog("L_INFO", "HELD locationRequest: Result code $var(res)\nUrl: $var(url)\n$var(pidf)"); ... -$var(res) = lost_held_query("heldsrv", "$var(pidf)", "$var(url)"", "$var(err)"); +$var(res) = lost_held_query("heldsrv", "$var(pidf)", "$var(url)", "$var(err)"); xlog("L_INFO", "HELD locationRequest: Result code $var(res)\nUrl: $var(url)\n$var(pidf)\n"); ... </programlisting>

3 years, 5 months

← Newer
1
...
7
8
9
10
11
12
13
...
32
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev November 2020