Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Joshua Colp
On Sat, Jan 14, 2017, at 06:20 PM, Daniel-Constantin Mierla wrote: > If you want a solution to avoid the issue via config: in the > request_route, store the RPID in an avp and update the acc parameter to > use the avp instead of the rpid variable. > > As for code, a solution could be resetting

Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Daniel-Constantin Mierla
If you want a solution to avoid the issue via config: in the request_route, store the RPID in an avp and update the acc parameter to use the avp instead of the rpid variable. As for code, a solution could be resetting the 'hdr->parsed' pointers to NULL that are in private memory after memcpy(,

Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Joshua Colp
On Sat, Jan 14, 2017, at 04:09 PM, Joshua Colp wrote: > On Sat, Jan 14, 2017, at 02:43 PM, Joshua Colp wrote: > > > > > Testing is in progress and so far so good. I do think the code in > > acc_onreply that cleans up the parsed header is not correct, though. > > It's referencing the shared

[sr-dev] git:master:1dc6da87: modules: readme files regenerated - tls ...

2017-01-14 Thread Kamailio Dev
Module: kamailio Branch: master Commit: 1dc6da87180a01f21554f2e857ec280814ba05e5 URL: https://github.com/kamailio/kamailio/commit/1dc6da87180a01f21554f2e857ec280814ba05e5 Author: Kamailio Dev Committer: Kamailio Dev Date:

[sr-dev] git:master:b6458e9d: tls Update README - update links to openssl.org, add ref to BCP, remove mention of "SSL"

2017-01-14 Thread Olle E . Johansson
Module: kamailio Branch: master Commit: b6458e9d1436936101b1fe3a3895c9857969d810 URL: https://github.com/kamailio/kamailio/commit/b6458e9d1436936101b1fe3a3895c9857969d810 Author: Olle E. Johansson Committer: Olle E. Johansson Date: 2017-01-14T21:51:47+01:00

[sr-dev] git:master:29922546: modules: readme files regenerated - utils ...

2017-01-14 Thread Kamailio Dev
Module: kamailio Branch: master Commit: 29922546644613648a30fc469b57085ca6e9a38e URL: https://github.com/kamailio/kamailio/commit/29922546644613648a30fc469b57085ca6e9a38e Author: Kamailio Dev Committer: Kamailio Dev Date:

Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Joshua Colp
On Sat, Jan 14, 2017, at 02:43 PM, Joshua Colp wrote: > > Testing is in progress and so far so good. I do think the code in > acc_onreply that cleans up the parsed header is not correct, though. > It's referencing the shared memory memory instead of the locally scoped > one where any parsed

[sr-dev] git:master:4ad19d01: utils Remove http_query function that now is replaced by the http_client module

2017-01-14 Thread Olle E . Johansson
Module: kamailio Branch: master Commit: 4ad19d01dce96c846a18f7f51619420a78e22bc7 URL: https://github.com/kamailio/kamailio/commit/4ad19d01dce96c846a18f7f51619420a78e22bc7 Author: Olle E. Johansson Committer: Olle E. Johansson Date: 2017-01-14T21:02:05+01:00

[sr-dev] git:master:7ddb20da: INSTALL updating compiler requirements

2017-01-14 Thread Olle E . Johansson
Module: kamailio Branch: master Commit: 7ddb20da95d35033eea00c2d4400db9a725b7e7b URL: https://github.com/kamailio/kamailio/commit/7ddb20da95d35033eea00c2d4400db9a725b7e7b Author: Olle E. Johansson Committer: Olle E. Johansson Date: 2017-01-14T20:46:31+01:00

Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Joshua Colp
On Sat, Jan 14, 2017, at 01:55 PM, Daniel-Constantin Mierla wrote: > >> The issues was with many processing handling the same transaction, which > >> has the sip_msg in shared memory, but then parsing of some headers > >> created pointers to private memory of the process doing the parsing. > >>

Re: [sr-dev] [kamailio/kamailio] kamailio-basic-kemi.cfg: add WITH_DB_TEXT (#930)

2017-01-14 Thread Daniel-Constantin Mierla
No new file should go in etc/ . The folder should be cleaned up anyhow. It is supposed to include only default config files to be used in packages. Any other config examples have to go in misc/examples/ ... or we should create a dedicated repo not to end up with a ton of configs that need to be

Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Daniel-Constantin Mierla
On 14/01/2017 18:45, Joshua Colp wrote: > On Sat, Jan 14, 2017, at 01:25 PM, Daniel-Constantin Mierla wrote: >> Hello, >> >> I think you were hit by an issue solved with commit >> 15fc8b9c59aaf31f005e38f54d363f1e9d0a068e : >> >>

Re: [sr-dev] [kamailio/kamailio] kamailio-basic-kemi.cfg: add WITH_DB_TEXT (#930)

2017-01-14 Thread Stefan Mititelu
@miconda I agree, generating the db_text tables instead of manually managing them is definitely a better approach. That 'bkp' table is just a leftover file that I forgot to remove. Maybe an approach would be to add a new kamailio-basic-dbtext.cfg in ``etc/``, with dbtext only pointing to

Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Joshua Colp
On Sat, Jan 14, 2017, at 01:25 PM, Daniel-Constantin Mierla wrote: > Hello, > > I think you were hit by an issue solved with commit > 15fc8b9c59aaf31f005e38f54d363f1e9d0a068e : > > https://github.com/kamailio/kamailio/commit/15fc8b9c59aaf31f005e38f54d363f1e9d0a068e > > The 4.1.3 was released

Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Daniel-Constantin Mierla
Hello, I think you were hit by an issue solved with commit 15fc8b9c59aaf31f005e38f54d363f1e9d0a068e : https://github.com/kamailio/kamailio/commit/15fc8b9c59aaf31f005e38f54d363f1e9d0a068e The 4.1.3 was released before, in April 2014: - https://www.kamailio.org/pub/kamailio/4.1.3/README I am

Re: [sr-dev] Remote-Party-ID Parsed Header Crash

2017-01-14 Thread Matthew Jordan
On Fri, Jan 13, 2017 at 2:57 PM, Joshua Colp wrote: > Greetings all, > > I'm currently investigating a crash in a Kamailio 4.1.3 deployment. The > backtrace is as follows: > > (I've commented out phone numbers and IP addresses using X but have > retained the correct length for

Re: [sr-dev] [kamailio/kamailio] kamailio-basic-kemi.cfg: add WITH_DB_TEXT (#930)

2017-01-14 Thread Daniel-Constantin Mierla
The examples in this folder are intended to reflect the equivalent of etc/kamailio-basic.cfg. Then I think than embedding the db_text tables is a wrong approach. They can be installed using kamdbctl and that will ensure they are having the proper structure. Now, for example, a change in

[sr-dev] git:master:3ad3438a: rtpengine: kemi rename to rtpengine_manage

2017-01-14 Thread Stefan Mititelu
Module: kamailio Branch: master Commit: 3ad3438ad6769ac403b17dc2399b60457fef9da8 URL: https://github.com/kamailio/kamailio/commit/3ad3438ad6769ac403b17dc2399b60457fef9da8 Author: Stefan Mititelu Committer: Stefan Mititelu Date:

[sr-dev] git:master:152f46dd: rtpengine: Add kemi rtpengine_manage0/1

2017-01-14 Thread Stefan Mititelu
Module: kamailio Branch: master Commit: 152f46dd03c179f2fc63cb69498ad43db51fb1ae URL: https://github.com/kamailio/kamailio/commit/152f46dd03c179f2fc63cb69498ad43db51fb1ae Author: Stefan Mititelu Committer: Stefan Mititelu Date:

Re: [sr-dev] [kamailio/kamailio] rtpengine: Add kemi rtpengine_manage0/1 (#929)

2017-01-14 Thread Stefan Mititelu
Merged #929. -- 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/929#event-923207262___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:master:e98e8a86: Merge pull request #929 from smititelu/kemi-rtpengine

2017-01-14 Thread GitHub
Module: kamailio Branch: master Commit: e98e8a86f33fab2a3969044d9eef327072709601 URL: https://github.com/kamailio/kamailio/commit/e98e8a86f33fab2a3969044d9eef327072709601 Author: Stefan Mititelu Committer: GitHub Date:

[sr-dev] [kamailio/kamailio] kamailio-basic-kemi.cfg: add WITH_DB_TEXT (#930)

2017-01-14 Thread Stefan Mititelu
This is for the people out there (me included) who don't want to install mysql to play with kamailio: - comment out ``#!define WITH_DB_MYSQL`` (still used by default) - set the right ``DB_TEXT_DIR``, to match ``kamailio_path/misc/examples/kemi/db_text`` (absolute path is needed; I tried with

[sr-dev] git:master:2064efea: modules: readme files regenerated - benchmark ...

2017-01-14 Thread Kamailio Dev
Module: kamailio Branch: master Commit: 2064efeace7d974b2767f3ba26895433ca84231e URL: https://github.com/kamailio/kamailio/commit/2064efeace7d974b2767f3ba26895433ca84231e Author: Kamailio Dev Committer: Kamailio Dev Date:

[sr-dev] git:master:2ba989de: nathelper: removed function names from log messages

2017-01-14 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 2ba989deb0d60f6b0570f3cc47f9c363ab9df024 URL: https://github.com/kamailio/kamailio/commit/2ba989deb0d60f6b0570f3cc47f9c363ab9df024 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:53677181: Merge pull request #926 from mslehto/doc

2017-01-14 Thread GitHub
Module: kamailio Branch: master Commit: 53677181381bc0cd026f2ef2398532e47f0cefb2 URL: https://github.com/kamailio/kamailio/commit/53677181381bc0cd026f2ef2398532e47f0cefb2 Author: Daniel-Constantin Mierla Committer: GitHub Date: 2017-01-14T13:02:24+01:00

[sr-dev] git:master:06a0a1f6: modules/pdt: restore dots and indentation for better layout in README

2017-01-14 Thread Mikko Lehto
Module: kamailio Branch: master Commit: 06a0a1f6e91e4d1c0f87afb33519f1de4584a918 URL: https://github.com/kamailio/kamailio/commit/06a0a1f6e91e4d1c0f87afb33519f1de4584a918 Author: Mikko Lehto Committer: Mikko Lehto Date: 2017-01-13T22:30:57+02:00 modules/pdt:

[sr-dev] git:master:1b7bc211: modules: update documentation to mention RPC instead of MI

2017-01-14 Thread Mikko Lehto
Module: kamailio Branch: master Commit: 1b7bc21145c91aef1f9fcc32cca8fe7418ca0d17 URL: https://github.com/kamailio/kamailio/commit/1b7bc21145c91aef1f9fcc32cca8fe7418ca0d17 Author: Mikko Lehto Committer: Mikko Lehto Date: 2017-01-13T12:15:59+02:00 modules: update

Re: [sr-dev] [kamailio/kamailio] documentation updates (#926)

2017-01-14 Thread Daniel-Constantin Mierla
Merged #926. -- 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/926#event-923186992___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:master:7bda0ce9: modules/dispatcher: remove duplicate text

2017-01-14 Thread Mikko Lehto
Module: kamailio Branch: master Commit: 7bda0ce9525c1b7e04de2879f7645dd097b23d02 URL: https://github.com/kamailio/kamailio/commit/7bda0ce9525c1b7e04de2879f7645dd097b23d02 Author: Mikko Lehto Committer: Mikko Lehto Date: 2017-01-10T12:31:11+02:00

Re: [sr-dev] [kamailio/kamailio] documentation updates (#926)

2017-01-14 Thread Daniel-Constantin Mierla
Thanks! -- 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/926#issuecomment-272619897___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] [kamailio/kamailio] rtpengine: Add kemi rtpengine_manage0/1 (#929)

2017-01-14 Thread Daniel-Constantin Mierla
Thanks! Can be merged, but then I would rename rtpengine_manage1() to just rtpengine_manage(), to be in pair with rtpproxy kemi exports. Variable number of arguments was in the plan, but iirc I couldn't find an easy way in python embed api to get the number of parameters (and didn't get much

[sr-dev] [kamailio/kamailio] rtpengine: Add kemi rtpengine_manage0/1 (#929)

2017-01-14 Thread Stefan Mititelu
Basically tested it from python script, by grepping for incoming commands on rtpengine side. IMHO, a nice improvement to the kemi interface would be to detect the number of input parameters given from the script and call the proper function straight ahead. You can view, comment on, or merge