Re: [sr-dev] [kamailio/kamailio] Kamailio 5 coredump 11 (SEGV) upcon receiving MESSAGE via jsonrpcs (#1030)

2017-03-16 Thread Anthony Messina
Just in case, these are some of the script settings: ``` # - jsonrpcs params - modparam("jsonrpcs", "pretty_format", 1) /* set the path to RPC fifo control file */ # modparam("jsonrpcs", "fifo_name", "/run/kamailio/kamailio_rpc.fifo") /* set the path to RPC unix socket control file */ #

Re: [sr-dev] [kamailio/kamailio] Kamailio 5 coredump 11 (SEGV) upcon receiving MESSAGE via jsonrpcs (#1030)

2017-03-16 Thread Anthony Messina
The new log is from a rebuild of the 5.0 branch as before with 0350885c076fa691de6106eb821ec5890572cee6, fa50c85a9e61dd6b91ce88f3f74c433af5d414e2, and a97701575863993f8a19f9dd611156b37ffca247 cherry-picked onto it. Still dumps.

Re: [sr-dev] [kamailio/kamailio] Kamailio 5 coredump 11 (SEGV) upcon receiving MESSAGE via jsonrpcs (#1030)

2017-03-16 Thread Daniel-Constantin Mierla
The log messages are not from syslog, so they are missing date/time and which PID printed them. Can you redo the test and grab via syslog? You can also test with a977015 . -- 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] Kamailio 5 coredump 11 (SEGV) upcon receiving MESSAGE via jsonrpcs (#1030)

2017-03-16 Thread Anthony Messina
@miconda attached is the debug log using the 5.0 branch at f5889fcc9e42cae01c58357a622d2270f994b8d7. [kamailio.script.debug.3.txt](https://github.com/kamailio/kamailio/files/849081/kamailio.script.debug.3.txt) -- You are receiving this because you are subscribed to this thread. Reply to this

Re: [sr-dev] [kamailio/kamailio] Kamailio 5 coredump 11 (SEGV) upcon receiving MESSAGE via jsonrpcs (#1030)

2017-03-16 Thread Anthony Messina
I see a97701575863993f8a19f9dd611156b37ffca247 in the master branch, but not the 5.0 branch. Do you want me to include that in the testing as well? -- 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] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Federico Cabiddu
Merged #1024. -- 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/1024#event-1002969820___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:master:b186dc66: core: enable reuse of tcp sockets

2017-03-16 Thread grumvalski
Module: kamailio Branch: master Commit: b186dc66b3c0fc7ba2cbbc88a1fd3112d4d7a492 URL: https://github.com/kamailio/kamailio/commit/b186dc66b3c0fc7ba2cbbc88a1fd3112d4d7a492 Author: grumvalski Committer: grumvalski Date:

[sr-dev] git:master:6fcd9f22: Merge pull request #1024 from grumvalski/tcp_reuseport

2017-03-16 Thread GitHub
Module: kamailio Branch: master Commit: 6fcd9f2200e218239492f387e9054e9414fe498a URL: https://github.com/kamailio/kamailio/commit/6fcd9f2200e218239492f387e9054e9414fe498a Author: Federico Cabiddu Committer: GitHub Date: 2017-03-16T16:44:08+01:00

Re: [sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Daniel-Constantin Mierla
Thanks, it can be merged. -- 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/1024#issuecomment-287086658___ sr-dev mailing list

Re: [sr-dev] [kamailio/kamailio] Fix for no cdr after Kamailio 5.0 restart (#1036)

2017-03-16 Thread Daniel-Constantin Mierla
Maybe DLGCB_LOADED callback can be used for this situation -- I guess it is no need populate again the variables in the dlg structure as they were already done when the dialog was effectively created. If you create a new solution, make a commit per module and prefix the first log message with

[sr-dev] [kamailio/kamailio] Fix for no cdr after Kamailio 5.0 restart (#1036)

2017-03-16 Thread Lucian Balaceanu
Hi Daniel, This is a sketch of how I would force the execution of the acc/cdr dialog callbacks for dialogues created from the database. This solution needs more testing and I don't expect it to be accepted as is. I think that the run_create_callbacks was omitted because at a later step, the

[sr-dev] about presence expires_offset

2017-03-16 Thread Luis Azedo
Hi, the description of "expires_offset" says "The value in seconds that should be subtracted from the expires value when sending a 200OK for a publish" but this value is used for subscriptions and AFAICT its behaviour is not correct. when checking expired subscriptions it subtracts the offset

Re: [sr-dev] availability of commits in master

2017-03-16 Thread Luis Azedo
Hi Daniel, thanks for the reply. i made some fixes myself to presence and db_text. should i create a PR for 5.0 branch ? about the small additions like pres_has_subscribers, since they are new features, when do you think we can have a 5.1 ? just trying to figure out if we should build it

Re: [sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Federico Cabiddu
Just updated the PR with your suggestion. -- 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/1024#issuecomment-287024005___ sr-dev mailing

Re: [sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Federico Cabiddu
@grumvalski pushed 1 commit. d2c3f38 core: compile by default support for SO_REUSEPORT when available on the OS -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Daniel-Constantin Mierla
I think ifdef can be used to check if SO_REUSEPORT is defined, on macos: ``` /usr/include/sys/socket.h:136:#define SO_REUSEPORT0x0200 /* allow local address & port reuse */ ``` On Linux: ``` /usr/include/asm-generic/socket.h:25:#define SO_REUSEPORT 15 ``` So it appears it

Re: [sr-dev] [kamailio/kamailio] Kamailio 5 coredump 11 (SEGV) upcon receiving MESSAGE via jsonrpcs (#1030)

2017-03-16 Thread Daniel-Constantin Mierla
Can you try with latest branch 5.0 and grab again the log messages with the debug=3 in case the issue is still there. -- 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:a9770157: xhttp: use stateless reply for sending http response

2017-03-16 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: a97701575863993f8a19f9dd611156b37ffca247 URL: https://github.com/kamailio/kamailio/commit/a97701575863993f8a19f9dd611156b37ffca247 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:fa50c85a: jsonrpcs: extra debug messages when rpc response is sent

2017-03-16 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: fa50c85a9e61dd6b91ce88f3f74c433af5d414e2 URL: https://github.com/kamailio/kamailio/commit/fa50c85a9e61dd6b91ce88f3f74c433af5d414e2 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:0350885c: xhttp: more debug messages for sending the response

2017-03-16 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 0350885c076fa691de6106eb821ec5890572cee6 URL: https://github.com/kamailio/kamailio/commit/0350885c076fa691de6106eb821ec5890572cee6 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

Re: [sr-dev] [kamailio/kamailio] core: enable reuse of tcp sockets (#1024)

2017-03-16 Thread Federico Cabiddu
Thanks for the feedback Daniel. About the first point, I think that we still need the #define because if SO_REUSEPORT is not defined for the running OS the compilation will fail. As far as I know the option is currently supported in FreeBSD/OpenBSD/NetBSD/Mac OSX, not in Solaris. Maybe we could

Re: [sr-dev] [kamailio/kamailio] Makefile.defs: default locks review (#1035)

2017-03-16 Thread Daniel-Constantin Mierla
Very likely the ports system set the ARCH variable in environment or command line for building kamailio, that's why a simple assignment didn't work. Also, the Makefile should have done the amd64=>x86_64 even without my changes if the ARCH was not enforced from outside. -- You are receiving

Re: [sr-dev] [kamailio/kamailio] Makefile.defs: default locks review (#1035)

2017-03-16 Thread SubnetsRU
Yes, now arch changed properly. 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/issues/1035#issuecomment-286990909___ sr-dev mailing

[sr-dev] git:4.4:65891fa1: Makefile.defs: override the ARCH for amd64 to x86_64

2017-03-16 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 4.4 Commit: 65891fa1051a08a261846db8926937dfcad9f518 URL: https://github.com/kamailio/kamailio/commit/65891fa1051a08a261846db8926937dfcad9f518 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:5.0:f5889fcc: Makefile.defs: override the ARCH for amd64 to x86_64

2017-03-16 Thread Daniel-Constantin Mierla
Module: kamailio Branch: 5.0 Commit: f5889fcc9e42cae01c58357a622d2270f994b8d7 URL: https://github.com/kamailio/kamailio/commit/f5889fcc9e42cae01c58357a622d2270f994b8d7 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:1adf88e7: Makefile.defs: override the ARCH for amd64 to x86_64

2017-03-16 Thread Daniel-Constantin Mierla
Module: kamailio Branch: master Commit: 1adf88e7211400b8d56a6dc7c463e60a2b697b80 URL: https://github.com/kamailio/kamailio/commit/1adf88e7211400b8d56a6dc7c463e60a2b697b80 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:32e1e504: utils/pdbt: FreeBSD fix, gcc is hardcoded, header is missing

2017-03-16 Thread Rick Barenthin
Module: kamailio Branch: master Commit: 32e1e5040d2ff5e13d5bd0373e7edd6cef56f172 URL: https://github.com/kamailio/kamailio/commit/32e1e5040d2ff5e13d5bd0373e7edd6cef56f172 Author: Rick Barenthin Committer: Rick Barenthin Date: 2017-03-14T22:35:52+01:00