Re: [sr-dev] [kamailio/kamailio] Crashing Kamailio: store_reply. replacing stored reply (#1744)

2018-11-30 Thread Luis Martin Gil
@miconda By the way, do you always recommend using "release" mode better than "debug" mode? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] Crashing Kamailio: store_reply. replacing stored reply (#1744)

2018-11-30 Thread Luis Martin Gil
Thanks for such a quick response. I tried to reproduce it today with version **5.1**, no luck yet. Also, I'll test the patch and keep you posted. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] rtpengine : crash is fixed and codec flags added to doc (#1742)

2018-11-30 Thread Yasin CANER
ycaner06 commented on this pull request. > @@ -1992,6 +1992,10 @@ static int parse_flags(struct ng_flags_parse > *ng_flags, struct sip_msg *msg, enu if (str_key_val_prefix(, "transcode", , ) || str_key_val_prefix(, "codec-transcode", , ))

Re: [sr-dev] [kamailio/kamailio] acc_json: Supporting multi_leg_info parameter (#1745)

2018-11-30 Thread Julien Chavanton
Sure, I will have a look when I have a chance. Very soon hopefully. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] Segfaults in 5.1.4 (maybe after one rtpengine disappeared) (#1613)

2018-11-30 Thread Daniel-Constantin Mierla
Anyone able to test with the commit referenced above (I just pushed the patch to master)? Might have been an issue caused by a delayed processing (eg., waiting for rtp engine command execution timeout) that also induced some response retransmissions. I want to be sure it doesn't have side

Re: [sr-dev] [kamailio/kamailio] Crashing Kamailio: store_reply. replacing stored reply (#1744)

2018-11-30 Thread Daniel-Constantin Mierla
I just pushed a patch, maybe you can test with it as well (or using master branch). >From another perspective, the code where it crashed is an extra check in an >EXTRA_DEBUG define, which is enabled in debug mode, not in release mode. Just >saying, it has to be fixed anyhow. -- You are

Re: [sr-dev] [kamailio/kamailio] rtpengine : crash is fixed and codec flags added to doc (#1742)

2018-11-30 Thread Richard Fuchs
rfuchs commented on this pull request. > @@ -1992,6 +1992,10 @@ static int parse_flags(struct ng_flags_parse > *ng_flags, struct sip_msg *msg, enu if (str_key_val_prefix(, "transcode", , ) || str_key_val_prefix(, "codec-transcode", , ))

Re: [sr-dev] [kamailio/kamailio] Suddenly found kamailio dead (#1717)

2018-11-30 Thread Daniel-Constantin Mierla
I pushed a safety check to avoid getting to log message printing with some invalid local variables. Maybe you can test with the patch and report if any issue. Then I will backport to 5.2 and 5.1 branches. -- You are receiving this because you are subscribed to this thread. Reply to this email

[sr-dev] git:master:46401e46: dialog: safety checks for local cseq and contact vars

2018-11-30 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 46401e465d88495d510d2cb36dc36ec86978c53e URL: https://github.com/kamailio/kamailio/commit/46401e465d88495d510d2cb36dc36ec86978c53e Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-11-30T16:31:22+01:00 dialog: safety checks

[sr-dev] git:master:39b89a18: tm: reply_received() - simplify locking for processing sip response

2018-11-30 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 39b89a18a8c357151a173ab02dc95dff1f02715d URL: https://github.com/kamailio/kamailio/commit/39b89a18a8c357151a173ab02dc95dff1f02715d Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-11-30T16:05:30+01:00 tm: reply_received()

Re: [sr-dev] [kamailio/kamailio] Private memory leaking in case of topos(redis) and topoh modules enable together (#1682)

2018-11-30 Thread Daniel-Constantin Mierla
Closing this one, given the solution above and not expecting to use the two modules at the same time. That could be an enhancement in the future, for now it is not a bug. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] Private memory leaking in case of topos(redis) and topoh modules enable together (#1682)

2018-11-30 Thread Daniel-Constantin Mierla
Closed #1682. -- 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/1682#event-1998163822___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] Suddenly found kamailio dead (#1717)

2018-11-30 Thread Mathias Schneuwly
Alright, I had to specify where gdb shall search for sources ``` (gdb) list 246 } 247 248 if(leg==DLG_CALLER_LEG) 249 dlg->from_rr_nb = skip_recs; 250 251 LM_DBG("leg(%d) route_set [%.*s], contact [%.*s], cseq [%.*s]" 252

Re: [sr-dev] [kamailio/kamailio] Dispatcher fails to work with NAPTR/SRV as destination (#1743)

2018-11-30 Thread Daniel-Constantin Mierla
Closed #1743. -- 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/1743#event-1997786116___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] Dispatcher fails to work with NAPTR/SRV as destination (#1743)

2018-11-30 Thread Daniel-Constantin Mierla
Iirc, AX is active state, which should be the expected in this case, because NAPTR+SRV can return many records of A//CNAME/IP type, and the module is not keeping track of them. Sometimes not all of them are returned, just couple, being rotated for the purpose of DNS based load balancer.

Re: [sr-dev] [kamailio/kamailio] Dispatcher fails to work with NAPTR/SRV as destination (#1743)

2018-11-30 Thread Patrick Murphy
NAPTRs/SRVs and A records are working as expected, but if a gateway with flag=16 is not resolvable by DNS, it is incorrectly kept in the ```FLAGS: AX``` state and not ```IP```. Whereas if an A-record (flag!=16) is not resolvable, it is correctly set to ```IP```. -- You are receiving this

Re: [sr-dev] [kamailio/kamailio] textops: new function regex_substring(itext, regex, index, mcount, dpv) (ca8967c)

2018-11-30 Thread Yasin CANER
mistype? -- 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/ca8967cfb1dc4b5f97ace4c3a32607503a364f69#commitcomment-31511704___ Kamailio

Re: [sr-dev] [kamailio/kamailio] textops : added a new function for getting substring with regular expression (#1733)

2018-11-30 Thread Daniel-Constantin Mierla
Closed #1733. -- 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/1733#event-1997435370___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] textops : added a new function for getting substring with regular expression (#1733)

2018-11-30 Thread Daniel-Constantin Mierla
Refactored the code to properly export to KEMI interface, allow variables in some of the parameters, so the values can be dynamic. The function was named regex_substring() and the order of the parameters was changed to follow the pattern: input value, match expression (regex, match index, max

[sr-dev] git:master:1a72cd21: textops: docs for regex_substring()

2018-11-30 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 1a72cd21cf3a4ed245a20cd21adeaf8c7e75a1c5 URL: https://github.com/kamailio/kamailio/commit/1a72cd21cf3a4ed245a20cd21adeaf8c7e75a1c5 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-11-30T10:20:23+01:00 textops: docs for

Re: [sr-dev] [kamailio/kamailio] cdp deadlock (#1739)

2018-11-30 Thread Daniel-Constantin Mierla
Closed #1739. -- 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/1739#event-1997330766___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] Allow startup with offline database. See issue https://github.com/kam… (#1706)

2018-11-30 Thread Daniel-Constantin Mierla
Closed #1706. -- 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/1706#event-1997329042___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] Allow startup with offline database. See issue https://github.com/kam… (#1706)

2018-11-30 Thread Daniel-Constantin Mierla
Pushed a different patch, referenced above. It continues to try to connect the other sql connections, even one fails as well as the reconnect at runtime is attempted only for the connection needed at that moment. The parameter is named connect_mode. I haven't tested, it would be good if you

[sr-dev] git:master:1b014d65: lib/srdb1: memset db connection structure earlier in db_bind_mod()

2018-11-30 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 1b014d65e26a99f9f5d4aea1b27949776f083c2d URL: https://github.com/kamailio/kamailio/commit/1b014d65e26a99f9f5d4aea1b27949776f083c2d Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-11-30T09:19:16+01:00 lib/srdb1: memset db

[sr-dev] git:master:6c1c70bb: sqlops: docs for connect_mode parameter

2018-11-30 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 6c1c70bbceeba87d6d87c852006c92f7d120ddb9 URL: https://github.com/kamailio/kamailio/commit/6c1c70bbceeba87d6d87c852006c92f7d120ddb9 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-11-30T09:25:53+01:00 sqlops: docs for

[sr-dev] git:master:2df43b99: sqlops: option to start even when connecting to db fails

2018-11-30 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 2df43b992b1e6932974bd0c928c0465a839b75da URL: https://github.com/kamailio/kamailio/commit/2df43b992b1e6932974bd0c928c0465a839b75da Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: 2018-11-30T09:24:59+01:00 sqlops: option to