sr-dev February 2018

sr-dev@lists.kamailio.org

33 participants
355 discussions

git:5.1:cc638e4c: topos: revert "topos: set uuid fields before checking contact/rr fields"
by Daniel-Constantin Mierla 12 Feb '18

12 Feb '18

Module: kamailio Branch: 5.1 Commit: cc638e4cd6d6f4ff9aca0bf6c9d6c940a3e64e59 URL: https://github.com/kamailio/kamailio/commit/cc638e4cd6d6f4ff9aca0bf6c9d6c94… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-02-12T09:14:26+01:00 topos: revert "topos: set uuid fields before checking contact/rr fields" This reverts commit 97532ca18f595d89edf9b0b6e6688847aa38dd44. (cherry picked from commit be6a419a5b0d6fe3922219a30788ad527a313fd6) --- Modified: src/modules/topos/tps_storage.c --- Diff: https://github.com/kamailio/kamailio/commit/cc638e4cd6d6f4ff9aca0bf6c9d6c94… Patch: https://github.com/kamailio/kamailio/commit/cc638e4cd6d6f4ff9aca0bf6c9d6c94… --- diff --git a/src/modules/topos/tps_storage.c b/src/modules/topos/tps_storage.c index 95f1097bd6..950d1982f4 100644 --- a/src/modules/topos/tps_storage.c +++ b/src/modules/topos/tps_storage.c @@ -214,10 +214,19 @@ int tps_storage_fill_contact(sip_msg_t *msg, tps_data_t *td, str *uuid, int dir) } else { sv = td->as_contact; } + if(sv.len<=0) { + /* no contact - skip */ + return 0; + } + if(td->cp + 8 + (2*uuid->len) + sv.len >= td->cbuf + TPS_DATA_SIZE) { LM_ERR("insufficient data buffer\n"); return -1; } + if (parse_uri(sv.s, sv.len, &puri) < 0) { + LM_ERR("failed to parse the uri\n"); + return -1; + } if(dir==TPS_DIR_DOWNSTREAM) { td->b_uuid.s = td->cp; *td->cp = 'b'; @@ -225,6 +234,8 @@ int tps_storage_fill_contact(sip_msg_t *msg, tps_data_t *td, str *uuid, int dir) memcpy(td->cp, uuid->s, uuid->len); td->cp += uuid->len; td->b_uuid.len = td->cp - td->b_uuid.s; + + td->bs_contact.s = td->cp; } else { td->a_uuid.s = td->cp; *td->cp = 'a'; @@ -232,20 +243,7 @@ int tps_storage_fill_contact(sip_msg_t *msg, tps_data_t *td, str *uuid, int dir) memcpy(td->cp, uuid->s, uuid->len); td->cp += uuid->len; td->a_uuid.len = td->cp - td->a_uuid.s; - } - if(sv.len<=0) { - /* no contact - skip */ - return 0; - } - - if (parse_uri(sv.s, sv.len, &puri) < 0) { - LM_ERR("failed to parse the uri\n"); - return -1; - } - if(dir==TPS_DIR_DOWNSTREAM) { - td->bs_contact.s = td->cp; - } else { td->as_contact.s = td->cp; } *td->cp = '<';

1 0

git:master:be6a419a: topos: revert "topos: set uuid fields before checking contact/rr fields"
by Daniel-Constantin Mierla 12 Feb '18

12 Feb '18

Module: kamailio Branch: master Commit: be6a419a5b0d6fe3922219a30788ad527a313fd6 URL: https://github.com/kamailio/kamailio/commit/be6a419a5b0d6fe3922219a30788ad5… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-02-12T09:12:26+01:00 topos: revert "topos: set uuid fields before checking contact/rr fields" This reverts commit 97532ca18f595d89edf9b0b6e6688847aa38dd44. --- Modified: src/modules/topos/tps_storage.c --- Diff: https://github.com/kamailio/kamailio/commit/be6a419a5b0d6fe3922219a30788ad5… Patch: https://github.com/kamailio/kamailio/commit/be6a419a5b0d6fe3922219a30788ad5… --- diff --git a/src/modules/topos/tps_storage.c b/src/modules/topos/tps_storage.c index 95f1097bd6..950d1982f4 100644 --- a/src/modules/topos/tps_storage.c +++ b/src/modules/topos/tps_storage.c @@ -214,10 +214,19 @@ int tps_storage_fill_contact(sip_msg_t *msg, tps_data_t *td, str *uuid, int dir) } else { sv = td->as_contact; } + if(sv.len<=0) { + /* no contact - skip */ + return 0; + } + if(td->cp + 8 + (2*uuid->len) + sv.len >= td->cbuf + TPS_DATA_SIZE) { LM_ERR("insufficient data buffer\n"); return -1; } + if (parse_uri(sv.s, sv.len, &puri) < 0) { + LM_ERR("failed to parse the uri\n"); + return -1; + } if(dir==TPS_DIR_DOWNSTREAM) { td->b_uuid.s = td->cp; *td->cp = 'b'; @@ -225,6 +234,8 @@ int tps_storage_fill_contact(sip_msg_t *msg, tps_data_t *td, str *uuid, int dir) memcpy(td->cp, uuid->s, uuid->len); td->cp += uuid->len; td->b_uuid.len = td->cp - td->b_uuid.s; + + td->bs_contact.s = td->cp; } else { td->a_uuid.s = td->cp; *td->cp = 'a'; @@ -232,20 +243,7 @@ int tps_storage_fill_contact(sip_msg_t *msg, tps_data_t *td, str *uuid, int dir) memcpy(td->cp, uuid->s, uuid->len); td->cp += uuid->len; td->a_uuid.len = td->cp - td->a_uuid.s; - } - if(sv.len<=0) { - /* no contact - skip */ - return 0; - } - - if (parse_uri(sv.s, sv.len, &puri) < 0) { - LM_ERR("failed to parse the uri\n"); - return -1; - } - if(dir==TPS_DIR_DOWNSTREAM) { - td->bs_contact.s = td->cp; - } else { td->as_contact.s = td->cp; } *td->cp = '<';

1 0

[kamailio/kamailio] rtpengine: rtpengine_offer is stripping all INVITE multipart bodies except application/sdp (#1434)
by foucse 12 Feb '18

12 Feb '18

### Description  When processing a multipart INVITE using the rtpengine module. The rtpengine_offer configuration function (called with default arguments) is stripping all bodies except the application/sdp MIME type body. ### Troubleshooting #### Reproduction  A client is sending an INVITE through a kamailio proxy paired with a rtpengine. ``` Client Proxy Server x INVITE (SDP) x x x x qqqqqqqqqqqqqqqqqqqqqqqqqq> x x x x 100 Trying x x x x <qqqqqqqqqqqqqqqqqqqqqqqqqq x x x x x x INVITE (SDP) x x x x qqqqqqqqqqqqqqqqqqqqqqqqqq> x x x x 100 Trying x x x x <qqqqqqqqqqqqqqqqqqqqqqqqqq x x x x x x x x x x x x x x x x x x x x x x x x x x x x 200 OK (SDP) x x x x <qqqqqqqqqqqqqqqqqqqqqqqqqq x x 200 OK (SDP) x x x x <qqqqqqqqqqqqqqqqqqqqqqqqqq x x x ``` The INVITE contains a application/sdp body and one or several other bodies of different MIME types. When the INVITE is going out of the proxy, all other bodies except the application/sdp MIME are filtered out. While The INVITE "Content-Type" header is still fixed to : `Content-Type: multipart/mixed;boundary="[boundary value]"` RTP Engine receives only the SDP body on its socket and is rewritting well the RTP ports. #### Debugging Data  ``` (paste your debugging data here) ``` #### Log Messages  ``` (paste your log messages here) ``` #### SIP Traffic  ``` (paste your sip traffic here) ``` ### Possible Solutions  This issue has been introduced by the following commit : 36cb753441a78bd13ded119cd468f32623f09fbf. rtpengine.c : ``` diff @@ -3328,6 +3328,7 @@ rtpengine_offer_answer(struct sip_msg *msg, const char *flags, int op, int more) str body, newbody; struct lump *anchor; pv_value_t pv_val; + str cur_body = {0, 0}; dict = rtpp_function_call_ok(&bencbuf, msg, op, flags, &body); if (!dict) @@ -3357,7 +3358,12 @@ rtpengine_offer_answer(struct sip_msg *msg, const char *flags, int op, int more) pkg_free(newbody.s); } else { - anchor = del_lump(msg, body.s - msg->buf, body.len, 0); + /* get the body from the message as body ptr may have changed */ + cur_body.len = 0; + cur_body.s = get_body(msg); + cur_body.len = msg->buf + msg->len - cur_body.s; + + anchor = del_lump(msg, cur_body.s - msg->buf, cur_body.len, 0); if (!anchor) { LM_ERR("del_lump failed\n"); goto error_free; ``` If this commit is removed, the previous behavior is restored. This commit fixes a segfault: "rtpengine: fixed segfault when using read_sdp_pv", so it appears to me that this may be useful. For, now I am not seeing why this is happening. ### Additional Information * **Kamailio Version** - output of `kamailio -v` seen on: ``` version: kamailio 5.1.0 (x86_64/linux) flags: STATS: Off, EXTRA_DEBUG, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, 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_LISTEN 16, 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 on 15:29:43 Feb 8 2018 with gcc 4.8.4 ``` but I think that the same behavior will be seen on master. * **Operating System**:  ``` (paste your output here) ``` -- 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/1434

3 5

git:5.0:749dd5c5: rtpengine: get again body pointer only when read_sdp_pv is set
by Daniel-Constantin Mierla 12 Feb '18

12 Feb '18

Module: kamailio Branch: 5.0 Commit: 749dd5c55ed100e9a57ece18e82fab2d1eaaab8e URL: https://github.com/kamailio/kamailio/commit/749dd5c55ed100e9a57ece18e82fab2… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-02-12T09:00:07+01:00 rtpengine: get again body pointer only when read_sdp_pv is set - reported by GH #1434 (cherry picked from commit 75c4ca0bb69bd4f1ba42c6d964697737813fa81e) (cherry picked from commit 8794d6d60549683120e22b3b150674e686a29ba0) --- Modified: src/modules/rtpengine/rtpengine.c --- Diff: https://github.com/kamailio/kamailio/commit/749dd5c55ed100e9a57ece18e82fab2… Patch: https://github.com/kamailio/kamailio/commit/749dd5c55ed100e9a57ece18e82fab2… --- diff --git a/src/modules/rtpengine/rtpengine.c b/src/modules/rtpengine/rtpengine.c index ab1c088d5d..95a4ba4bd1 100644 --- a/src/modules/rtpengine/rtpengine.c +++ b/src/modules/rtpengine/rtpengine.c @@ -2883,12 +2883,17 @@ rtpengine_offer_answer(struct sip_msg *msg, const char *flags, int op, int more) pkg_free(newbody.s); } else { - /* get the body from the message as body ptr may have changed */ - cur_body.len = 0; - cur_body.s = get_body(msg); - cur_body.len = msg->buf + msg->len - cur_body.s; - - anchor = del_lump(msg, cur_body.s - msg->buf, cur_body.len, 0); + if (read_sdp_pvar_str.len > 0) { + /* get the body from the message as body ptr may have changed + * when using read_sdp_pv */ + cur_body.len = 0; + cur_body.s = get_body(msg); + cur_body.len = msg->buf + msg->len - cur_body.s; + + anchor = del_lump(msg, cur_body.s - msg->buf, cur_body.len, 0); + } else { + anchor = del_lump(msg, body.s - msg->buf, body.len, 0); + } if (!anchor) { LM_ERR("del_lump failed\n"); goto error_free;

1 0

git:5.1:8794d6d6: rtpengine: get again body pointer only when read_sdp_pv is set
by Daniel-Constantin Mierla 12 Feb '18

12 Feb '18

Module: kamailio Branch: 5.1 Commit: 8794d6d60549683120e22b3b150674e686a29ba0 URL: https://github.com/kamailio/kamailio/commit/8794d6d60549683120e22b3b150674e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-02-12T08:58:30+01:00 rtpengine: get again body pointer only when read_sdp_pv is set - reported by GH #1434 (cherry picked from commit 75c4ca0bb69bd4f1ba42c6d964697737813fa81e) --- Modified: src/modules/rtpengine/rtpengine.c --- Diff: https://github.com/kamailio/kamailio/commit/8794d6d60549683120e22b3b150674e… Patch: https://github.com/kamailio/kamailio/commit/8794d6d60549683120e22b3b150674e… --- diff --git a/src/modules/rtpengine/rtpengine.c b/src/modules/rtpengine/rtpengine.c index 5b82988229..666cfb32ba 100644 --- a/src/modules/rtpengine/rtpengine.c +++ b/src/modules/rtpengine/rtpengine.c @@ -3358,12 +3358,17 @@ rtpengine_offer_answer(struct sip_msg *msg, const char *flags, int op, int more) pkg_free(newbody.s); } else { - /* get the body from the message as body ptr may have changed */ - cur_body.len = 0; - cur_body.s = get_body(msg); - cur_body.len = msg->buf + msg->len - cur_body.s; - - anchor = del_lump(msg, cur_body.s - msg->buf, cur_body.len, 0); + if (read_sdp_pvar_str.len > 0) { + /* get the body from the message as body ptr may have changed + * when using read_sdp_pv */ + cur_body.len = 0; + cur_body.s = get_body(msg); + cur_body.len = msg->buf + msg->len - cur_body.s; + + anchor = del_lump(msg, cur_body.s - msg->buf, cur_body.len, 0); + } else { + anchor = del_lump(msg, body.s - msg->buf, body.len, 0); + } if (!anchor) { LM_ERR("del_lump failed\n"); goto error_free;

1 0

[kamailio/kamailio] app_lua // crash // realloc(): invalid next size: (#1433)
by amoizard 12 Feb '18

12 Feb '18

### Description When starting my kamailio with my complete kamailio.cfg, I have crash reporting *** Error in `/usr/sbin/kamailio': realloc(): invalid next size: 0x0000000001060980 *** The issue looks to be related to using "app_lua" module. However, the issue don't always show up. If I remove ONE modules, it won't crash. I've been trying to get a simple config to crash, but I have to keep many modules to reproduce the crash. The crash is happening line 228 of app_lua_api.c: if(luaL_dostring(L, "sr.probe()")!=0) ### Troubleshooting Can't start kamailio. #### Reproduction I have attached my minimal kamailio4.cfg file to reproduce the crash. I have removed my DBURL credentials. [kamailio4.cfg.txt](https://github.com/kamailio/kamailio/files/1707085/kamai… I'm starting kamailio with: /usr/sbin/kamailio -D -f kamailio4.cfg -e -E -ddd #### Debugging Data Core was generated by `/usr/sbin/kamailio -D -f kamailio4.cfg -e -E -ddd'. Program terminated with signal SIGABRT, Aborted. #0 0x00007f8ecfb62067 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:56 56 ../nptl/sysdeps/unix/sysv/linux/raise.c: No such file or directory. (gdb) bt #0 0x00007f8ecfb62067 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:56 #1 0x00007f8ecfb63448 in __GI_abort () at abort.c:89 #2 0x00007f8ecfba01b4 in __libc_message (do_abort=do_abort@entry=1, fmt=fmt@entry=0x7f8ecfc95210 "*** Error in `%s': %s: 0x%s ***\n") at ../sysdeps/posix/libc_fatal.c:175 #3 0x00007f8ecfba598e in malloc_printerr (action=1, str=0x7f8ecfc9142a "realloc(): invalid next size", ptr=<optimized out>) at malloc.c:4996 #4 0x00007f8ecfba876b in _int_realloc (av=av@entry=0x7f8ecfed2620 <main_arena>, oldp=oldp@entry=0x1060970, oldsize=oldsize@entry=736, nb=nb@entry=1552) at malloc.c:4234 #5 0x00007f8ecfba9769 in __GI___libc_realloc (oldmem=0x1060980, bytes=1536) at malloc.c:3029 #6 0x00007f8ec324acae in luaM_realloc_ (L=L@entry=0x10605c0, block=block@entry=0x1060980, osize=720, nsize=nsize@entry=1536) at lmem.c:79 #7 0x00007f8ec32469aa in luaD_reallocstack (L=0x10605c0, newsize=90) at ldo.c:145 #8 0x00007f8ec324b66d in open_func (ls=ls@entry=0x7ffe459b9720, fs=fs@entry=0x7ffe459b9780) at lparser.c:350 #9 0x00007f8ec324e0b3 in luaY_parser (L=0x10605c0, z=0x7ffe459b9bf0, buff=<optimized out>, name=<optimized out>) at lparser.c:388 #10 0x00007f8ec3246cd1 in f_parser (L=L@entry=0x10605c0, ud=ud@entry=0x7ffe459b9ba0) at ldo.c:498 #11 0x00007f8ec324692e in luaD_rawrunprotected (L=L@entry=0x10605c0, f=f@entry=0x7f8ec3246c80 <f_parser>, ud=ud@entry=0x7ffe459b9ba0) at ldo.c:116 #12 0x00007f8ec32477bb in luaD_pcall (L=L@entry=0x10605c0, func=func@entry=0x7f8ec3246c80 <f_parser>, u=u@entry=0x7ffe459b9ba0, old_top=672, ef=<optimized out>) at ldo.c:464 #13 0x00007f8ec32478c2 in luaD_protectedparser (L=L@entry=0x10605c0, z=z@entry=0x7ffe459b9bf0, name=name@entry=0x7f8ec350547e "sr.probe()") at ldo.c:514 #14 0x00007f8ec3243528 in lua_load (L=0x10605c0, reader=reader@entry=0x7f8ec3252940 <getS>, data=data@entry=0x7ffe459b9c40, chunkname=0x7f8ec350547e "sr.probe()") at lapi.c:869 #15 0x00007f8ec3253d8c in luaL_loadbuffer (L=<optimized out>, buff=<optimized out>, size=<optimized out>, name=<optimized out>) at lauxlib.c:614 #16 0x00007f8ec34e87d7 in lua_sr_init_probe () from /usr/lib/x86_64-linux-gnu/kamailio/modules/app_lua.so #17 0x00007f8ec347497d in ?? () from /usr/lib/x86_64-linux-gnu/kamailio/modules/app_lua.so #18 0x0000000000549a00 in ?? () #19 0x0000000000549d92 in init_child () #20 0x00000000004214c8 in main_loop () #21 0x000000000042aae9 in main () (gdb) (gdb) info locals resultvar = 0 pid = 23353 selftid = 23353 #### Log Messages I'm copy/pasting a few logs from the app_lua module. If you wish more, let me know. 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.hdr (hdr) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.jsonrpcs (jsonrpcs) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.kex (kex) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.tm (tm) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.tmx (tmx) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.sl (sl) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.tsilo (tsilo) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.rr (rr) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.pvx (pvx) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.maxfwd (maxfwd) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.registrar (registrar) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.textops (textops) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.siputils (siputils) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.sanity (sanity) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.acc (acc) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.uri_db (uri_db) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.auth (auth) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.auth_db (auth_db) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.permissions (permissions) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.alias_db (alias_db) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.domain (domain) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.presence (presence) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.presence_xml (presence_xml) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.nathelper (nathelper) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.rtpproxy (rtpproxy) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.tls (tls) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.pike (pike) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.htable (htable) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.xhttp (xhttp) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.websocket (websocket) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.sqlops (sqlops) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.cfgutils (cfgutils) 0(23353) DEBUG: app_lua [app_lua_sr.c:1975]: lua_sr_kemi_register_libs(): initializing kemi sub-module: KSR.app_lua (app_lua) 0(23353) DEBUG: app_lua [app_lua_sr.c:1978]: lua_sr_kemi_register_libs(): module 'KSR' has been initialized #### SIP Traffic Not applicable. The proxy crash before starting. ### Possible Solutions I haven't been able to find any clue. ### Additional Information I'm using a debian running jessie with debian package from http://deb.kamailio.org/kamailio51 I have installed liblua5.1-0-dbg to have more backtrace from the core dump. $ /usr/sbin/kamailio -v version: kamailio 5.1.1 (x86_64/linux) flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, 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_LISTEN 16, 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 4.9.2 * **Operating System**: $ uname -a Linux sip.antisip.com 3.16.0-5-amd64 #1 SMP Debian 3.16.51-3+deb8u1 (2018-01-08) x86_64 GNU/Linux Tks for looking at this! Ay -- 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/1433

2 2

git:5.0:43c9ad6f: lcr: fixed checking if there is more uris available for next_gw
by Daniel-Constantin Mierla 12 Feb '18

12 Feb '18

Module: kamailio Branch: 5.0 Commit: 43c9ad6f4dc2e20dd40a2d3dfa6b03f3dd79e3d4 URL: https://github.com/kamailio/kamailio/commit/43c9ad6f4dc2e20dd40a2d3dfa6b03f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-02-12T08:50:54+01:00 lcr: fixed checking if there is more uris available for next_gw - manual backport for 732a3153a0a41d5f951fff85de607f0b46ae73da --- Modified: src/modules/lcr/lcr_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/43c9ad6f4dc2e20dd40a2d3dfa6b03f… Patch: https://github.com/kamailio/kamailio/commit/43c9ad6f4dc2e20dd40a2d3dfa6b03f… --- diff --git a/src/modules/lcr/lcr_mod.c b/src/modules/lcr/lcr_mod.c index 3644dacee5..20af1d5c7e 100644 --- a/src/modules/lcr/lcr_mod.c +++ b/src/modules/lcr/lcr_mod.c @@ -2512,8 +2512,8 @@ static int next_gw(struct sip_msg* _m, char* _s1, char* _s2) * Take Request-URI user from ruri_user_avp and generate Request * and Destination URIs. */ - if (!generate_uris(_m, r_uri, &(ruri_user_val.s), &r_uri_len, dst_uri, - &dst_uri_len, &addr, &gw_index, &flags, &tag_str)) { + if (generate_uris(_m, r_uri, &(ruri_user_val.s), &r_uri_len, dst_uri, + &dst_uri_len, &addr, &gw_index, &flags, &tag_str) <= 0) { return -1; } }

1 0

git:5.0:be5579ed: core: kemi - eol for debug message
by Daniel-Constantin Mierla 12 Feb '18

12 Feb '18

Module: kamailio Branch: 5.0 Commit: be5579eddc9cd43302e918cf12dcdd353043644a URL: https://github.com/kamailio/kamailio/commit/be5579eddc9cd43302e918cf12dcdd3… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-02-12T08:47:17+01:00 core: kemi - eol for debug message (cherry picked from commit 0476993b7c341bcc7a994ab52986b89f240bd3b0) (cherry picked from commit 0e1fdeea9cc5f6948f8b3e19c1350718288278ba) --- Modified: src/core/kemi.c --- Diff: https://github.com/kamailio/kamailio/commit/be5579eddc9cd43302e918cf12dcdd3… Patch: https://github.com/kamailio/kamailio/commit/be5579eddc9cd43302e918cf12dcdd3… --- diff --git a/src/core/kemi.c b/src/core/kemi.c index 145cb9a926..99fcc34f1f 100644 --- a/src/core/kemi.c +++ b/src/core/kemi.c @@ -895,7 +895,7 @@ int sr_kemi_eng_register(str *ename, sr_kemi_eng_route_f froute) _sr_kemi_eng_list[_sr_kemi_eng_list_size].froute = froute; _sr_kemi_eng_list_size++; - LM_DBG("registered config routing enginge [%.*s]", + LM_DBG("registered config routing enginge [%.*s]\n", ename->len, ename->s); return 0;

1 0

git:5.0:a03a6ab7: app_lua: do lua_checkstack() to ensure enough capacity on Lua stack
by Daniel-Constantin Mierla 12 Feb '18

12 Feb '18

Module: kamailio Branch: 5.0 Commit: a03a6ab7d7a1ab6e688ad3aeb7089803a695202c URL: https://github.com/kamailio/kamailio/commit/a03a6ab7d7a1ab6e688ad3aeb708980… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-02-12T08:47:00+01:00 app_lua: do lua_checkstack() to ensure enough capacity on Lua stack - Lua doesn't do automatic management of the stack * http://www.lua.org/manual/5.1/manual.html#3.2 - do exit on insufficient capacity - based on the report by GH #1433 (cherry picked from commit f6d69d87abdc48df22135c79ee2309248248ca8e) (cherry picked from commit df9389a613e16f15b860af60222abe43ca367d09) --- Modified: src/modules/app_lua/app_lua_sr.c --- Diff: https://github.com/kamailio/kamailio/commit/a03a6ab7d7a1ab6e688ad3aeb708980… Patch: https://github.com/kamailio/kamailio/commit/a03a6ab7d7a1ab6e688ad3aeb708980… --- diff --git a/src/modules/app_lua/app_lua_sr.c b/src/modules/app_lua/app_lua_sr.c index 86380ec6ad..64ebafae40 100644 --- a/src/modules/app_lua/app_lua_sr.c +++ b/src/modules/app_lua/app_lua_sr.c @@ -1970,10 +1970,14 @@ void lua_sr_kemi_register_libs(lua_State *L) } n++; } + if(!lua_checkstack(L, i+8)) { + LM_ERR("not enough Lua stack capacity\n"); + exit(-1); + } luaL_openlib(L, mname, _sr_crt_KSRMethods, 0); - LM_DBG("initializing kemi sub-module: %s (%s)\n", mname, - emods[k].kexp[0].mname.s); + LM_DBG("initializing kemi sub-module: %s (%s) (%d/%d/%d)\n", mname, + emods[k].kexp[0].mname.s, i, k, n); } } - LM_DBG("module 'KSR' has been initialized\n"); + LM_DBG("module 'KSR' has been initialized (%d/%d)\n", emods_size, n); }

1 0

git:5.1:0e1fdeea: core: kemi - eol for debug message
by Daniel-Constantin Mierla 12 Feb '18

12 Feb '18

Module: kamailio Branch: 5.1 Commit: 0e1fdeea9cc5f6948f8b3e19c1350718288278ba URL: https://github.com/kamailio/kamailio/commit/0e1fdeea9cc5f6948f8b3e19c135071… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2018-02-12T08:42:47+01:00 core: kemi - eol for debug message (cherry picked from commit 0476993b7c341bcc7a994ab52986b89f240bd3b0) --- Modified: src/core/kemi.c --- Diff: https://github.com/kamailio/kamailio/commit/0e1fdeea9cc5f6948f8b3e19c135071… Patch: https://github.com/kamailio/kamailio/commit/0e1fdeea9cc5f6948f8b3e19c135071… --- diff --git a/src/core/kemi.c b/src/core/kemi.c index 55630c28f5..5631f4b909 100644 --- a/src/core/kemi.c +++ b/src/core/kemi.c @@ -1342,7 +1342,7 @@ int sr_kemi_eng_register(str *ename, sr_kemi_eng_route_f froute) _sr_kemi_eng_list[_sr_kemi_eng_list_size].froute = froute; _sr_kemi_eng_list_size++; - LM_DBG("registered config routing enginge [%.*s]", + LM_DBG("registered config routing enginge [%.*s]\n", ename->len, ename->s); return 0;

1 0

← Newer
1
...
22
23
24
25
26
27
28
...
36
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev February 2018