sr-dev

sr-dev@lists.kamailio.org

3 participants
45196 discussions

possible bug on kemi dlg_ontimeout
by David Escartin 15 May '20

15 May '20

Hello all Testing dialog timeout event callback for kemi lua setup on kamailio 5.4.0-dev3 (x86_64/linux) d74157-dirty, i had some problems to perform the dlg timeroute behaviour the same as we could on native version. If i used KSR.dialog.var_sets to set $dlg_ctx(timeout_route), I saw that on a dialog timeout, the dlg_ontimeout event route was not triggered, and only the dialog callback was executed with evname dialog:end. (only executing event_route -1 on state 5) So i'm guessing that the kemi dialog var_sets is for $dlg_var only (¿?) Anycase, If i used KSR.pv.sets, i saw the kamailio was crashing ..... 16(5579) DEBUG: dialog [dlg_hash.c:1013]: dlg_ref_helper(): ref dlg 0x7f9a961f5f90 with 1 -> 3 16(5579) DEBUG: dialog [dlg_handlers.c:1589]: dlg_ontimeout(): executing route -1 on timeout 22(5645) CRITICAL: <core> [core/pass_fd.c:277]: receive_fd(): EOF on 8 22(5645) DEBUG: <core> [core/tcp_main.c:3762]: handle_ser_child(): dead child 16, pid 5579 (shutting down?) .... Adding some debug, i could see that the call was not attempting the kemi function of line 1598 of dlg_handlers.c, and was trying the code i think was for native on line 1593. We think the issue is that the toroute is defined as unsigned dlg_hash.h: unsigned int toroute; /*!< index of route that is executed on timeout */ but after that when setting $dlg_ctx(timeout_route) with pv module, it tries to compare an integer being unsigned, so it's always true and never uses kemi with this change we made it work + if ((int)(dlg->toroute) > 0) { + /*if(dlg->toroute>0) {*/ but i don't know maybe there is a better way to do it. best regards david

2 5

[kamailio/kamailio] tm core.when free ack cancel_buf. (#2321)
by zhangyang 15 May '20

15 May '20

### Description wh have a too long 500msg,when the build ack,it abort we use ka listen two port ,and use port 1 send port 2 after port2 send port1,it loop in kamailio.cfg.its my wrong,but kamailio core.... ### Troubleshooting #### Debugging Data this is core bt #0 0x00002ae1c343c5f7 in raise () from /lib64/libc.so.6 #1 0x00002ae1c343dce8 in abort () from /lib64/libc.so.6 #2 0x000000000068155a in qm_debug_check_frag (qm=0x2ae1c8fb4000, f=0x2ae1c93bf558, file=0x2ae1c4a1b9ff "tm: t_msgbuilder.c", line=518, efile=0x7f1ca7 "core/mem/q_malloc.c", eline=504) at core/mem/q_malloc.c:139 #3 0x0000000000684bf5 in qm_free (qmp=0x2ae1c8fb4000, p=0x2ae1c93bf590, file=0x2ae1c4a1b9ff "tm: t_msgbuilder.c", func=0x2ae1c4a1fb50 <__FUNCTION__.11261> "build_local_reparse", line=518, mname=0x2ae1c4a1aa30 "tm") at core/mem/q_malloc.c:504 #4 0x000000000068e83f in qm_shm_free (qmp=0x2ae1c8fb4000, p=0x2ae1c93bf590, file=0x2ae1c4a1b9ff "tm: t_msgbuilder.c", func=0x2ae1c4a1fb50 <__FUNCTION__.11261> "build_local_reparse", line=518, mname=0x2ae1c4a1aa30 "tm") at core/mem/q_malloc.c:1268 #5 0x00002ae1c4951e76 in build_local_reparse (Trans=0x2ae1c94524d8, branch=0, len=0x7ffe977ff370, method=0x2ae1c4a2399b "ACK", method_len=3, to=0x7ffe977ff130, reason=0x0) at t_msgbuilder.c:518 #6 0x00002ae1c497336e in build_ack (rpl=0x2ae1c3d3bb08, trans=0x2ae1c94524d8, branch=0, ret_len=0x7ffe977ff370) at t_reply.c:354 #7 0x00002ae1c4986066 in reply_received (p_msg=0x2ae1c3d3bb08) at t_reply.c:2292 #8 0x000000000053da31 in do_forward_reply (msg=0x2ae1c3d3bb08, mode=0) at core/forward.c:748 #9 0x000000000053f5a9 in forward_reply (msg=0x2ae1c3d3bb08) at core/forward.c:853 #10 0x000000000059c633 in receive_msg ( buf=0xa6ee20 <buf.6868> "SIP/2.0 500 Internal Server Error\r\nVia: SIP/2.0/UDP 172.18.67.32;branch=z9hG4bK4301.6c6183869659ad37775254042e1ec6f4.0\r\nVia: SIP/2.0/UDP 172.18.67.32;branch=z9hG4bK4301.a149a0620b507cd0ba760eeb1cd32b3"..., len=27098, rcv_info=0x7ffe977ffae0) at core/receive.c:434 #11 0x000000000048e42c in udp_rcv_loop () at core/udp_server.c:541 #12 0x000000000042509d in main_loop () at main.c:1669 #13 0x000000000042c788 in main (argc=9, argv=0x7ffe97800028) at main.c:2710 (gdb) up #1 0x00002ae1c343dce8 in abort () from /lib64/libc.so.6 (gdb) up #2 0x000000000068155a in qm_debug_check_frag (qm=0x2ae1c8fb4000, f=0x2ae1c93bf558, file=0x2ae1c4a1b9ff "tm: t_msgbuilder.c", line=518, efile=0x7f1ca7 "core/mem/q_malloc.c", eline=504) at core/mem/q_malloc.c:139 139 core/mem/q_malloc.c: 没有那个文件或目录. (gdb) up #3 0x0000000000684bf5 in qm_free (qmp=0x2ae1c8fb4000, p=0x2ae1c93bf590, file=0x2ae1c4a1b9ff "tm: t_msgbuilder.c", func=0x2ae1c4a1fb50 <__FUNCTION__.11261> "build_local_reparse", line=518, mname=0x2ae1c4a1aa30 "tm") at core/mem/q_malloc.c:504 504 in core/mem/q_malloc.c (gdb) up #4 0x000000000068e83f in qm_shm_free (qmp=0x2ae1c8fb4000, p=0x2ae1c93bf590, file=0x2ae1c4a1b9ff "tm: t_msgbuilder.c", func=0x2ae1c4a1fb50 <__FUNCTION__.11261> "build_local_reparse", line=518, mname=0x2ae1c4a1aa30 "tm") at core/mem/q_malloc.c:1268 1268 in core/mem/q_malloc.c (gdb) up #5 0x00002ae1c4951e76 in build_local_reparse (Trans=0x2ae1c94524d8, branch=0, len=0x7ffe977ff370, method=0x2ae1c4a2399b "ACK", method_len=3, to=0x7ffe977ff130, reason=0x0) at t_msgbuilder.c:518 518 t_msgbuilder.c: 没有那个文件或目录. (gdb) p cancel_buf $1 = 0x2ae1c93bf590 "ACK sip:2002@172.18.67.42 SIP/2.0\r\n\253" (gdb) p reason $2 = (struct cancel_reason *) 0x0 #### Log Messages [2020-05-11 11:10:27] (124796) INFO: <script>: IN_REPLY_PBX [2020-05-11 11:10:27] (124796) INFO: <script>: IN_REPLY_PBX IN Call <null>: Status 100 3001@172.18.67.42:5060 -----> 2002@172.18.67.32:5060 [2020-05-11 11:10:27] (124796) ERROR: tm [t_msgbuilder.c:516]: build_local_reparse(): HDR_EOH_T was not found [2020-05-11 11:10:27] (124796) CRITICAL: <core> [core/mem/q_malloc.c:137]: qm_debug_check_frag(): BUG: qm: fragm. 0x2ae1c93bf558 (address 0x2ae1c93bf590) end overwritten (2e322f5049532032, ab0a0d30)! Memory allocator was called from tm: t_msgbuilder.c:518. Fragment marked by tm: t_msgbuilder.c:345. Exec from core/mem/q_malloc.c:504. #### SIP Traffic we use ka listen two port ,and use port 1 send port 2 after port2 send port1,it loop in kamailio.cfg.its my wrong,but kamailio core.... ### Possible Solutions we see t_msgbuilder.c shm_free(cancel_buf); faild ... ### Additional Information version: kamailio 5.2.5 (x86_64/linux) 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: unknown compiled on 15:28:15 Nov 4 2019 with gcc 4.8.5 * **Operating System**: CentOS Linux release 7.2.1511 (Core) Linux localhost.localdomain 3.10.0-327.el7.x86_64 #1 SMP Thu Nov 19 22:10:57 UTC 2015 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/2321

3 8

git:master:7ae675bd: pua_reginfo: free xml doc in case of error
by Daniel-Constantin Mierla 14 May '20

14 May '20

Module: kamailio Branch: master Commit: 7ae675bd90b4420ebaa17768c97aa09219f53116 URL: https://github.com/kamailio/kamailio/commit/7ae675bd90b4420ebaa17768c97aa09… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-14T21:21:51+02:00 pua_reginfo: free xml doc in case of error - a few whitespacing fixes --- Modified: src/modules/pua_reginfo/usrloc_cb.c --- Diff: https://github.com/kamailio/kamailio/commit/7ae675bd90b4420ebaa17768c97aa09… Patch: https://github.com/kamailio/kamailio/commit/7ae675bd90b4420ebaa17768c97aa09… --- diff --git a/src/modules/pua_reginfo/usrloc_cb.c b/src/modules/pua_reginfo/usrloc_cb.c index f8a55480bb..f128680122 100644 --- a/src/modules/pua_reginfo/usrloc_cb.c +++ b/src/modules/pua_reginfo/usrloc_cb.c @@ -16,8 +16,8 @@ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License for more details. * - * You should have received a copy of the GNU General Public License - * along with this program; if not, write to the Free Software + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ @@ -57,7 +57,7 @@ void pua_reginfo_update_self_op(int v) } str* build_reginfo_full(urecord_t * record, str uri, ucontact_t* c, int type) { - xmlDocPtr doc = NULL; + xmlDocPtr doc = NULL; xmlNodePtr root_node = NULL; xmlNodePtr registration_node = NULL; xmlNodePtr contact_node = NULL; @@ -78,11 +78,12 @@ str* build_reginfo_full(urecord_t * record, str uri, ucontact_t* c, int type) { root_node = xmlNewNode(NULL, BAD_CAST "reginfo"); if(root_node==0) { LM_ERR("Unable to create reginfo-XML-Element\n"); + xmlFreeDoc(doc); return NULL; } /* This is our Root-Element: */ - xmlDocSetRootElement(doc, root_node); - + xmlDocSetRootElement(doc, root_node); + xmlNewProp(root_node, BAD_CAST "xmlns", BAD_CAST "urn:ietf:params:xml:ns:reginfo"); /* we set the version to 0 but it should be set to the correct value in the pua module */ @@ -163,7 +164,7 @@ str* build_reginfo_full(urecord_t * record, str uri, ucontact_t* c, int type) { memset(buf, 0, sizeof(buf)); snprintf(buf, sizeof(buf), "%.*s", ptr->received.len, ptr->received.s); xmlNewProp(contact_node, BAD_CAST "received", BAD_CAST buf); - + /* path Attribute */ memset(buf, 0, sizeof(buf)); snprintf(buf, sizeof(buf), "%.*s", ptr->path.len, ptr->path.s); @@ -216,7 +217,7 @@ str* build_reginfo_full(urecord_t * record, str uri, ucontact_t* c, int type) { } if(doc) xmlFreeDoc(doc); return NULL; -} +} void reginfo_usrloc_cb(ucontact_t* c, int type, void* param) { str* body= NULL; @@ -240,7 +241,7 @@ void reginfo_usrloc_cb(ucontact_t* c, int type, void* param) { content_type.s = "application/reginfo+xml"; content_type.len = 23; - + /* Debug Output: */ LM_DBG("AOR: %.*s (%.*s)\n", c->aor->len, c->aor->s, c->domain->len, c->domain->s); if(type & UL_CONTACT_INSERT) LM_DBG("type= UL_CONTACT_INSERT\n"); @@ -292,7 +293,7 @@ void reginfo_usrloc_cb(ucontact_t* c, int type, void* param) { } uri.len = snprintf(uri.s, uri.len, "sip:%.*s", record->aor.len, record->aor.s); } - + /* Build the XML-Body: */ body = build_reginfo_full(record, uri, c, type); @@ -315,8 +316,8 @@ void reginfo_usrloc_cb(ucontact_t* c, int type, void* param) { publ.id.len = id_buf_len; publ.content_type = content_type; publ.expires = 3600; - - /* make UPDATE_TYPE, as if this "publish dialog" is not found + + /* make UPDATE_TYPE, as if this "publish dialog" is not found by pua it will fallback to INSERT_TYPE anyway */ publ.flag|= UPDATE_TYPE; publ.source_flag |= REGINFO_PUBLISH; @@ -328,7 +329,7 @@ void reginfo_usrloc_cb(ucontact_t* c, int type, void* param) { if(pua.send_publish(&publ) < 0) { LM_ERR("Error while sending publish\n"); - } + } error: if (uri.s) pkg_free(uri.s); if(body) { @@ -338,4 +339,4 @@ void reginfo_usrloc_cb(ucontact_t* c, int type, void* param) { if(record) ul.release_urecord(record); return; -} +}

1 0

git:master:4ad790f6: evapi: cast to void* when printing logs with pointer value
by Daniel-Constantin Mierla 14 May '20

14 May '20

Module: kamailio Branch: master Commit: 4ad790f697554332ea590793d3ec56e13437e82f URL: https://github.com/kamailio/kamailio/commit/4ad790f697554332ea590793d3ec56e… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-14T21:15:51+02:00 evapi: cast to void* when printing logs with pointer value --- Modified: src/modules/evapi/evapi_dispatch.c --- Diff: https://github.com/kamailio/kamailio/commit/4ad790f697554332ea590793d3ec56e… Patch: https://github.com/kamailio/kamailio/commit/4ad790f697554332ea590793d3ec56e… --- diff --git a/src/modules/evapi/evapi_dispatch.c b/src/modules/evapi/evapi_dispatch.c index 52e8f1d23e..e16d4c5ad2 100644 --- a/src/modules/evapi/evapi_dispatch.c +++ b/src/modules/evapi/evapi_dispatch.c @@ -606,7 +606,7 @@ void evapi_recv_notify(struct ev_loop *loop, struct ev_io *watcher, int revents) return; } - LM_DBG("received [%p] [%.*s] (%d)\n", emsg, + LM_DBG("received [%p] [%.*s] (%d)\n", (void*)emsg, emsg->data.len, emsg->data.s, emsg->data.len); evapi_dispatch_notify(emsg); shm_free(emsg); @@ -774,8 +774,8 @@ int _evapi_relay(str *evdata, str *ctag, int unicast) emsg->unicast = unicast; } - LM_DBG("sending [%p] [%.*s] (%d)\n", emsg, emsg->data.len, emsg->data.s, - emsg->data.len); + LM_DBG("sending [%p] [%.*s] (%d)\n", (void*)emsg, emsg->data.len, + emsg->data.s, emsg->data.len); if(_evapi_notify_sockets[1]!=-1) { len = write(_evapi_notify_sockets[1], &emsg, sizeof(evapi_msg_t*)); if(len<=0) { @@ -785,7 +785,7 @@ int _evapi_relay(str *evdata, str *ctag, int unicast) } } else { cfg_update(); - LM_DBG("dispatching [%p] [%.*s] (%d)\n", emsg, + LM_DBG("dispatching [%p] [%.*s] (%d)\n", (void*)emsg, emsg->data.len, emsg->data.s, emsg->data.len); if(evapi_dispatch_notify(emsg) == 0) { shm_free(emsg); @@ -855,7 +855,7 @@ int evapi_relay(str *event, str *data) LM_ERR("failed to pass the pointer to evapi dispatcher\n"); return -1; } - LM_DBG("sent [%p] [%.*s] (%d)\n", sbuf, sbuf->len, sbuf->s, sbuf->len); + LM_DBG("sent [%p] [%.*s] (%d)\n", (void*)sbuf, sbuf->len, sbuf->s, sbuf->len); return 0; } #endif

1 0

git:master:c4591349: tm: safety checks for branch request len
by Daniel-Constantin Mierla 14 May '20

14 May '20

Module: kamailio Branch: master Commit: c45913497f222c67bcccdbc8724b25bb083c556f URL: https://github.com/kamailio/kamailio/commit/c45913497f222c67bcccdbc8724b25b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2020-05-14T20:38:22+02:00 tm: safety checks for branch request len --- Modified: src/modules/tm/t_cancel.c Modified: src/modules/tm/t_fwd.c Modified: src/modules/tm/t_msgbuilder.c Modified: src/modules/tm/t_reply.c Modified: src/modules/tm/uac.c --- Diff: https://github.com/kamailio/kamailio/commit/c45913497f222c67bcccdbc8724b25b… Patch: https://github.com/kamailio/kamailio/commit/c45913497f222c67bcccdbc8724b25b…

1 0

git:master:a5722aa7: ctl: small spelling fix in comment
by Henning Westerholt 14 May '20

14 May '20

Module: kamailio Branch: master Commit: a5722aa7773eb4153e8c3b241cae6e3f5f7a9894 URL: https://github.com/kamailio/kamailio/commit/a5722aa7773eb4153e8c3b241cae6e3… Author: Henning Westerholt <hw(a)skalatan.de> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-05-14T16:38:31Z ctl: small spelling fix in comment --- Modified: src/modules/ctl/binrpc.h --- Diff: https://github.com/kamailio/kamailio/commit/a5722aa7773eb4153e8c3b241cae6e3… Patch: https://github.com/kamailio/kamailio/commit/a5722aa7773eb4153e8c3b241cae6e3… --- diff --git a/src/modules/ctl/binrpc.h b/src/modules/ctl/binrpc.h index 29e06670ed..3642a0015b 100644 --- a/src/modules/ctl/binrpc.h +++ b/src/modules/ctl/binrpc.h @@ -136,7 +136,7 @@ more bytes available */ #define E_BINRPC_EOP -5 /* end of packet reached */ #define E_BINRPC_NOTINIT -6 /* parse ctx not initialized */ -#define E_BINRPC_TYPE -7 /* unkown type for record, or requested +#define E_BINRPC_TYPE -7 /* unknown type for record, or requested type doesn't match record type */ #define E_BINRPC_RECORD -8 /* bad record (unexpected, bad struct a.s.o)*/ #define E_BINRPC_BUG -9 /* internal error, bug */

1 0

git:master:cbb070b7: cdp: small spelling fix in log message
by Henning Westerholt 14 May '20

14 May '20

Module: kamailio Branch: master Commit: cbb070b74e63017a23355dd8d5266c2d57b762f9 URL: https://github.com/kamailio/kamailio/commit/cbb070b74e63017a23355dd8d5266c2… Author: Henning Westerholt <hw(a)skalatan.de> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-05-14T16:37:34Z cdp: small spelling fix in log message --- Modified: src/modules/cdp/diameter_peer.c --- Diff: https://github.com/kamailio/kamailio/commit/cbb070b74e63017a23355dd8d5266c2… Patch: https://github.com/kamailio/kamailio/commit/cbb070b74e63017a23355dd8d5266c2… --- diff --git a/src/modules/cdp/diameter_peer.c b/src/modules/cdp/diameter_peer.c index 043d4f3d72..38386be5d9 100644 --- a/src/modules/cdp/diameter_peer.c +++ b/src/modules/cdp/diameter_peer.c @@ -257,7 +257,7 @@ int diameter_peer_start(int blocking) /* fork receiver for unknown peers */ seed = random(); - pid = fork_process(1001+k,"cdp_receiver_peer_unkown",1); + pid = fork_process(1001+k,"cdp_receiver_peer_unknown",1); if (pid==-1){ LM_CRIT("init_diameter_peer(): Error on fork() for unknown peer receiver!\n");

1 0

git:master:fc09015b: core: small spelling fix in comment
by Henning Westerholt 14 May '20

14 May '20

Module: kamailio Branch: master Commit: fc09015bbad342b435de7ddb3300e19a415c340d URL: https://github.com/kamailio/kamailio/commit/fc09015bbad342b435de7ddb3300e19… Author: Henning Westerholt <hw(a)skalatan.de> Committer: Henning Westerholt <hw(a)skalatan.de> Date: 2020-05-14T16:38:00Z core: small spelling fix in comment --- Modified: src/core/dns_cache.h --- Diff: https://github.com/kamailio/kamailio/commit/fc09015bbad342b435de7ddb3300e19… Patch: https://github.com/kamailio/kamailio/commit/fc09015bbad342b435de7ddb3300e19… --- diff --git a/src/core/dns_cache.h b/src/core/dns_cache.h index 3552e8392f..b09c79be8a 100644 --- a/src/core/dns_cache.h +++ b/src/core/dns_cache.h @@ -70,7 +70,7 @@ enum dns_errors{ -- returned only by the dns_resolve* functions when called iteratively,; it signals the end of the ip/records list */ - E_DNS_UNKNOWN /**< unkown error */, + E_DNS_UNKNOWN /**< unknown error */, E_DNS_INTERNAL_ERR /**< internal error */, E_DNS_BAD_SRV_ENTRY, E_DNS_NO_SRV /**< unresolvable srv record */,

1 0

[kamailio/kamailio] Debian package version don't support distribution upgrades (#2160)
by Bas Couwenberg 14 May '20

14 May '20

When upgrading from stretch to buster the kamailio packages aren't upgraded because the 5.3.1+stretch version is greater than 5.3.1+buster: ``` $ dpkg --compare-versions 5.3.1+stretch gt 5.3.1+buster ; echo $? 0 ``` The package version should include the release version instead of its codename, e.g. 5.3.1+deb9u1 & 5.3.1+deb10u1 like done for [stable updates](https://www.debian.org/doc/manuals/developers-reference/pkgs.en.ht…: ``` $ dpkg --compare-versions 5.3.1+deb10u1 gt 5.3.1+deb9u1 ; echo $? 0 ``` -- 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/2160

3 5

[kamailio/kamailio] jansson_set() "result has json error" on long json strings (#2327)
by Victor Seva 14 May '20

14 May '20

### Description We noticed random errors from jansson module when: ``` xlog("L_NOTICE", "**** PRE '«$avp(i:901)»' - [% logres %]\n"); jansson_set("str", "cc_sub", "$avp(s:caller_concurrent_calls_count)", "$(avp(i:901)[*])"); ``` ``` May 14 10:11:37 sp1 proxy[23633]: ERROR: jansson [jansson_funcs.c:219]: janssonmod_set(): result has json error at line 1: end of file expected near 'xMTAwMiIsImxucF' May 14 10:11:37 sp1 proxy[23633]: ERROR: jansson [jansson_funcs.c:219]: janssonmod_set(): result has json error at line 1: end of file expected near '0' May 14 10:11:37 sp1 proxy[23633]: ERROR: jansson [jansson_funcs.c:219]: janssonmod_set(): result has json error at line 1: end of file expected near 'TEwMDEiLCJkX' ``` ### Troubleshooting #### Reproduction `` jansson_set(type, key/path, value, result)`` Using a long json result, in my tests longer than 400 chars fires the ERROR #### Log Messages ``` May 14 10:11:37 sp1 proxy[23633]: NOTICE: <script>: Setting acc destination-leg for uuid '«0d5845e9-2cd1-4b1e-bbc4-6ca9102f864e»': '«{"gpp":["long long long long","long long long long","long long long long","long long long long","long long long long","l ong long long long","long long long long","long long long long","long long long long","long long long long"],"plu":0,"s_id":"","c_id":"","a_id":315,"dialed":"4311001","uuid":"0d5845e9-2cd1-4b1e-bbc4-6ca9102f864e","u":"testuser1001","d":"spce.test","u_in ":"4311001","d_in":"spce.test","lcr":0,"lnp_p":"","u_out":"testuser1001","lnp_t":"","fci":"","cc_quota":"","cc_sub":"","cc_cust":""}»' - R=«sip:conf=4311001@conference.local» ID=«NGCP%invite_conference_update%///1-23982(a)127.126.0.xn--1_pbx-1-yna UA='<null>' May 14 10:11:37 sp1 proxy[23633]: NOTICE: <script>: **** PRE '«{"gpp":[null,null,null,null,null,null,null,null,null,null],"clir":0,"uuid":"adeff08b-bd66-4578-8a01-194185d06827","u":"testuser1002","d":"spce.test","cli":"43110011002","s_id":"","c_id":""," a_id":315,"pau":"","par":"","s":"call","ip":"127.126.0.1","t":1589443897.908021,"lnp_p":"","cli_out":"43110011002","lnp_t":"","pai":"","div":"","cid":"","u2u":"","lcr":0,"cc_quota":"","cc_sub":"","cc_cust":""}»' - ID=«NGCP%invite_conference_update%///1- 23982(a)127.126.0.xn--1_pbx-1-yna UA='<null>' May 14 10:11:37 sp1 proxy[23633]: ERROR: jansson [jansson_funcs.c:219]: janssonmod_set(): result has json error at line 1: end of file expected near 'xMTAwMiIsImxucF' May 14 10:11:37 sp1 proxy[23633]: ERROR: jansson [jansson_funcs.c:219]: janssonmod_set(): result has json error at line 1: end of file expected near '0' ``` ### Possible Solutions In my fast check I notice the result is forced to end in '\0'. Preparing a PR with a solution. ### Additional Information * **Kamailio Version** - output of `kamailio -v` This is Sipwise's version but with no jansson patches at all ``` version: kamailio 5.3.4 (x86_64/linux) flags: USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, USE_RAW_SOCKS, DISABLE_NAGLE, USE_MCAST, NO_SIG_DEBUG, 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_BLACKLIST, 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 ``` -- 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/2327

1 1

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev