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

2017-01-13 Thread Joshua Colp
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 each) (gdb) bt #0 0x7f017f2ac96e in memcpy () from /lib64/libc.so.6 #1

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

2017-01-13 Thread Mikko Lehto
@mslehto pushed 1 commit. 06a0a1f modules/pdt: restore dots and indentation for better layout in README -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] Kamailio failed to autostart on reboot with MySQL database (#925)

2017-01-13 Thread Daniel-Constantin Mierla
Not a kamailio code bug -- it looks like mysql server is not started at that time. Maybe @linuxmaniac can add comments here and see whether or not something else needs to be specified in the startup service script to set the order after mysql server. -- You are receiving this because you are

[sr-dev] git:master:6fc4367f: auth_ephemeral: Fix typo in RPC commands

2017-01-13 Thread Carsten Bock
Module: kamailio Branch: master Commit: 6fc4367f1aa4fd623ed9ca5271525d23d5e0ef56 URL: https://github.com/kamailio/kamailio/commit/6fc4367f1aa4fd623ed9ca5271525d23d5e0ef56 Author: Carsten Bock Committer: Carsten Bock Date: 2017-01-13T18:56:20+01:00

Re: [sr-dev] [kemi] porting module functions with input/output PV parameters

2017-01-13 Thread Daniel-Constantin Mierla
On 13/01/2017 17:30, Mititelu Stefan wrote: > >> 1. What would be the best practice for porting module functions >> that read/write data from/to a pseudo-variable? ...At the moment >> I'm thinking of using str for input and return; probably this >> will lead to partially

[sr-dev] git:master:3e1c3500: Merge pull request #928 from lbalaceanu/savefunc_retval

2017-01-13 Thread GitHub
Module: kamailio Branch: master Commit: 3e1c3500b78593c3e0e376e872e3aa3028c2be65 URL: https://github.com/kamailio/kamailio/commit/3e1c3500b78593c3e0e376e872e3aa3028c2be65 Author: Daniel-Constantin Mierla Committer: GitHub Date: 2017-01-13T17:47:24+01:00

Re: [sr-dev] [kamailio/kamailio] registrar: save() returns separate code on 503 generation (#928)

2017-01-13 Thread Daniel-Constantin Mierla
I am merging it -- in the future, you can do it once it's clarified and all is ok. -- 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] registrar: save() returns separate code on 503 generation (#928)

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

[sr-dev] git:master:f6443567: registrar: save() returns new err code

2017-01-13 Thread Lucian Balaceanu
Module: kamailio Branch: master Commit: f644356742e5967b442b5efb99f695566c99396a URL: https://github.com/kamailio/kamailio/commit/f644356742e5967b442b5efb99f695566c99396a Author: Lucian Balaceanu Committer: Lucian Balaceanu Date:

Re: [sr-dev] [kemi] porting module functions with input/output PV parameters

2017-01-13 Thread Mititelu Stefan
> > 1. What would be the best practice for porting module functions that > read/write data from/to a pseudo-variable? ...At the moment I'm thinking of > using str for input and return; probably this will lead to partially > duplicating the code of the existing module functions. > > > The idea is

Re: [sr-dev] [kamailio/kamailio] registrar: save() returns separate code on 503 generation (#928)

2017-01-13 Thread Lucian Balaceanu
Thanks for the input. I updated the pull request. -- 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/928#issuecomment-272472608___ sr-dev

Re: [sr-dev] [kemi] porting module functions with input/output PV parameters

2017-01-13 Thread Daniel-Constantin Mierla
Hello, On 13/01/2017 15:30, Mititelu Stefan wrote: > Hi, > > I've been reading about new kemi interface that will be available in > 5.0. Writing configs in lua/python is awesome!!! and javascript via app_jsdt to make it more awesome for the internet world :-) > > I am currently looking into

[sr-dev] [kemi] porting module functions with input/output PV parameters

2017-01-13 Thread Mititelu Stefan
Hi, I've been reading about new kemi interface that will be available in 5.0. Writing configs in lua/python is awesome!!! I am currently looking into and trying to add kemi support for a module and have some questions: 1. What would be the best practice for porting module functions that

Re: [sr-dev] [kamailio/kamailio] Add kpcre group (#901)

2017-01-13 Thread Fabrice Fontaine
OK thanks, I will close this PR and create 2 PRs: one for kmini and another for kpcre -- 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] Add kpcre group (#901)

2017-01-13 Thread Fabrice Fontaine
Closed #901. -- 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/901#event-922281331___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:master:b1e49b5a: textopsx: updated old log messages

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

Re: [sr-dev] [kamailio/kamailio] Add kpcre group (#901)

2017-01-13 Thread Daniel-Constantin Mierla
Yes, it's ok to add new groups that can help in what you need. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:03e6757d: cdp: fix compiler warnings

2017-01-13 Thread Victor Seva
Module: kamailio Branch: master Commit: 03e6757d78d4439ce16df3d6e7f8023993c17df7 URL: https://github.com/kamailio/kamailio/commit/03e6757d78d4439ce16df3d6e7f8023993c17df7 Author: Victor Seva Committer: Victor Seva Date:

Re: [sr-dev] [kamailio/kamailio] registrar: save() returns separate code on 503 generation (#928)

2017-01-13 Thread Daniel-Constantin Mierla
Thanks, it can be merged, but it needs also a note in the docs about the new return code. More about the return code of the save() -- returning 0 seems to be a bug right now, because there is a flag to control if the reply should be sent or not from the code -- REG_SAVE_NORPL_FL. ``` error:

[sr-dev] git:master:f018b88b: ims_registrar_scscf: fix compiler warnings

2017-01-13 Thread Victor Seva
Module: kamailio Branch: master Commit: f018b88bfe0620f22e6c76a388eeb7e3408a4496 URL: https://github.com/kamailio/kamailio/commit/f018b88bfe0620f22e6c76a388eeb7e3408a4496 Author: Victor Seva Committer: Victor Seva Date:

Re: [sr-dev] [kamailio/kamailio] ims usrloc scscf: fix inline function bin_xxx declared but never defined (#927)

2017-01-13 Thread Daniel-Constantin Mierla
Does it worth moving the functions to .h file? It is a consistent size of code there, maybe just removing the 'inline' is better, probably the compiler is ignoring it anyhow and having it inside the .h means bigger object files for each .c including the .h. -- You are receiving this because

[sr-dev] [kamailio/kamailio] registrar: save() returns separate code on 503 generation (#928)

2017-01-13 Thread Lucian Balaceanu
We would like to know when the save("location") has generated a 503 message and as such we are thinking of modifying the save() function to return a code (like -2) for this error. The default behaviour now seems to be that most errors in save() log a notice message, generate a reply via

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

2017-01-13 Thread Daniel-Constantin Mierla
miconda commented on this pull request. > @@ -501,11 +501,8 @@ pd_translate("*", "$var(a)"); RPC Command Example: - -... -# _sdomain_ _prefix_ _domain_ - kamailio.org 123 -... The dots here were on purpose to enclose the example

[sr-dev] [kamailio/kamailio] ims usrloc scscf: fix inline function bin_xxx declared but never defined (#927)

2017-01-13 Thread Victor Seva
* move inline functions to bin_utils.h * clang-format bin_utils.* files You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/927 -- Commit Summary -- * ims_usrloc_scscf: fix compiler warnings -- File Changes -- M

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

2017-01-13 Thread Mikko Lehto
You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/926 -- Commit Summary -- * modules/dispatcher: remove duplicate text * modules: update documentation to mention RPC instead of MI -- File Changes -- M

[sr-dev] git:master:b5dd54e9: ims_usrloc_scscf: fix compiler warnings

2017-01-13 Thread Victor Seva
Module: kamailio Branch: master Commit: b5dd54e980bda1ed850d4899d2b6e3ce139832bb URL: https://github.com/kamailio/kamailio/commit/b5dd54e980bda1ed850d4899d2b6e3ce139832bb Author: Victor Seva Committer: Victor Seva Date:

Re: [sr-dev] [kamailio/kamailio] Add kpcre group (#901)

2017-01-13 Thread Fabrice Fontaine
buildroot does not use deb or rpm specs from kamailio tree. To integrate a package to buildroot, I have to define a .mk that explain how to download the source from github and selects the appropriate modules depending on what packages were selected by the user. For example, if the package

[sr-dev] git:master:5d7fe2c2: sanity: main file of the module renamed to sanity_mod.c

2017-01-13 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 5d7fe2c2e9d4669003681086651041ec5e74500a URL: https://github.com/kamailio/kamailio/commit/5d7fe2c2e9d4669003681086651041ec5e74500a Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:77d88007: sanity: updated from old style of log messages

2017-01-13 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 77d8800767612d6ae198eddcd07c51a7e91174c0 URL: https://github.com/kamailio/kamailio/commit/77d8800767612d6ae198eddcd07c51a7e91174c0 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

Re: [sr-dev] [kamailio/kamailio] Add kpcre group (#901)

2017-01-13 Thread Daniel-Constantin Mierla
@ffontaine - is the buildroot using deb or rpm specs from kamailio tree? There is another set of groups, which doesn't have 'k' as prefix of the last word in the name -- for example, there is module_group_standard and module_group_kstandard. The module_group_k... were intended for packaging,

[sr-dev] git:master:50854467: erlang: fix compuler warning

2017-01-13 Thread Victor Seva
Module: kamailio Branch: master Commit: 5085446772fa43ffde30c752f955fe4a3067c826 URL: https://github.com/kamailio/kamailio/commit/5085446772fa43ffde30c752f955fe4a3067c826 Author: Victor Seva Committer: Victor Seva Date:

Re: [sr-dev] [kamailio/kamailio] Add kpcre group (#901)

2017-01-13 Thread Fabrice Fontaine
My goal was to integrate kamailio into buildroot (https://buildroot.org/). buildroot allows the user to create its own embedded linux system and the pcre package is not always selected by the user. -- You are receiving this because you are subscribed to this thread. Reply to this email

Re: [sr-dev] [kamailio/kamailio] Add kpcre group (#901)

2017-01-13 Thread Victor Seva
I do believe pcre is quite common and we can live with it on kstandard -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:36e28f83: kamcmd: removed svn ids, history

2017-01-13 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 36e28f83e7ae64409370c8163ccab749532944fc URL: https://github.com/kamailio/kamailio/commit/36e28f83e7ae64409370c8163ccab749532944fc Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date: