[sr-dev] new compiler warnings (on debian stretch) in latest master

2017-04-10 Thread Juha Heinanen
I built latest master on Debian stretch and got the compiler warnings below. -- Juha --- CC (gcc) [M ipops.so] ip_parser.o ip_parser.c:18:18: warning: 'ip_parser_en_main' defined but not used

Re: [sr-dev] testing function result in if-statement

2017-04-10 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > I think I looked at it several years ago and the situation is a bit > complex given the way the config script interpreter is doing the > evaluation -- there are the rules that return 0 means exit, negative > values are evaluated to false and positive to true.

[sr-dev] git:master:ac2572b8: modules/registrar: apply method_filtering only on initial requests w/o to-tag

2017-04-08 Thread Juha Heinanen
Module: kamailio Branch: master Commit: ac2572b880c5373864a55ebe1b9e538dd9337e63 URL: https://github.com/kamailio/kamailio/commit/ac2572b880c5373864a55ebe1b9e538dd9337e63 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-04-08T15:22:54+03

[sr-dev] testing function result in if-statement

2017-04-08 Thread Juha Heinanen
I noticed that function result cannot be directly tested in if-statement: if (func_x() == y) ... Why is that? Should I open a feature request or is it a no-go? -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:master:8e037f62: modules/siputils: made tel2sip to return 2 if no conversion was needed

2017-04-08 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 8e037f623ae6daf58e092d8a41430fd13d1d38ec URL: https://github.com/kamailio/kamailio/commit/8e037f623ae6daf58e092d8a41430fd13d1d38ec Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-04-08T10:31:23+03

[sr-dev] git:5.0:dccca195: pv: declare the pv_empty as buffer with size

2017-04-08 Thread Juha Heinanen
Module: kamailio Branch: 5.0 Commit: dccca1959b6f96bb508baa9dab0eacba7c69f20c URL: https://github.com/kamailio/kamailio/commit/dccca1959b6f96bb508baa9dab0eacba7c69f20c Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-04-08T09

[sr-dev] git:5.0:2705a9d3: core: use buffers instead of static empty string

2017-04-08 Thread Juha Heinanen
Module: kamailio Branch: 5.0 Commit: 2705a9d36481d66ea86f79b574760f3009118775 URL: https://github.com/kamailio/kamailio/commit/2705a9d36481d66ea86f79b574760f3009118775 Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-04-08T09

[sr-dev] git:5.0:f9fbd7a9: pv: renamed empty buf variable

2017-04-08 Thread Juha Heinanen
Module: kamailio Branch: 5.0 Commit: f9fbd7a97cae4431a64ab350c2eed7bbc0586049 URL: https://github.com/kamailio/kamailio/commit/f9fbd7a97cae4431a64ab350c2eed7bbc0586049 Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-04-08T09

Re: [sr-dev] registrar method_filtering

2017-04-07 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > A quicker solution will be to disable method matching based on presence > of To-tag, because this is anyhow part of an established dialog and this > can be end-to-end without involving the proxy/registrar. Sounds good to me, although in case of gruu, registrar

Re: [sr-dev] set_ruri_host crash

2017-04-07 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > It was the parser setting the value of host part to "", not the > transformation, so the change had to be done in other file. Can you try > again with latest master? thanks for the fix. now the crash is gone. statements: $var(test_uri) =

Re: [sr-dev] set_ruri_host crash

2017-04-07 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > What OS do you have and which version of gdb? The list command should > list the code around the line where the crash happens, but it doesn't do > it for you. this is debian jessie with GNU gdb (Debian 7.7.1+dfsg-5) 7.7.1. > Anyhow, you can extract the code

Re: [sr-dev] set_ruri_host crash

2017-04-07 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > Is it latest master? The line is matching an LM_ERR() with only static > string. master as it was on april 3. > You should list the code, in gdb: > > frame 0 > > list Program terminated with signal SIGSEGV, Segmentation fault. #0 0x7f9d268703e0 in

[sr-dev] set_ruri_host crash

2017-04-07 Thread Juha Heinanen
k 5.1 crashes when executing this: $var(test_uri) = "tel:+358447058060"; xlog("L_INFO", "** extracting domain from <$var(test_uri)>\n"); $rd = $(var(test_uri){uri.host}); xlog("L_INFO", "** request uri is <$ru>\n"); Apr 7 15:15:17 lohi /usr/bin/sip-proxy[14477]: INFO:

[sr-dev] registrar method_filtering

2017-04-07 Thread Juha Heinanen
if registrar method_filtering param is set, lookup produces -2 and $du is not set if method is not listed in allow header of register request. this is fine with initial requests, but in case of an invite dialog, initial invite may list another set of methods that the ua supports in in-dialog

[sr-dev] compiler warnings (latest master)

2017-04-03 Thread Juha Heinanen
CC (gcc) [M pv.so] pv_core.o pv_core.c: In function 'pv_get_hdrc': pv_core.c:1811:9: warning: unused variable 'p_size' [-Wunused-variable] int n, p_size; ^ pv_core.c:1811:6: warning: unused variable 'n' [-Wunused-variable] int n, p_size; ^ pv_core.c:1810:12:

[sr-dev] git:master:b2e8a193: modules/lcr: added lcr.load_gws rpc function

2017-03-18 Thread Juha Heinanen
Module: kamailio Branch: master Commit: b2e8a193d086df58437c5b4d4df5dce684d88c61 URL: https://github.com/kamailio/kamailio/commit/b2e8a193d086df58437c5b4d4df5dce684d88c61 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-03-18T11:40:56+02

Re: [sr-dev] [kamailio/kamailio] substr transformation error does not include config line number (#1019)

2017-03-14 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > I pushed a patch to attempt to print the line for string transformation > errors, as they are commonly used. However, this can be also accomplished for > all error logs by setting config parameters: > > ``` > log_prefix_mode=1 > log_prefix="(cfg line:

[sr-dev] [kamailio/kamailio] websocket: X-Forwarded-For and X-Real-IP support (#1002)

2017-02-19 Thread Juha Heinanen
Michael Ricordeau writes: > New feature to get the real source IP when kamailio websocket is > behind a load balancer transmitting X-Forwarded-For and/or X-Real-IP > headers: I have used $hdr(X-Forwarded-For) to get the source address. What benefit does a pv have over that? -- Juha

Re: [sr-dev] larger dialplan/repl_exp field

2017-02-16 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > I am fine to increase the size. The version number should not be > increased, it is not a change that breaks any query and c code should > have checks/or allocations based on the size. Should be also ok to > backport. OK, done, Juha

[sr-dev] git:5.0:5a9716f9: lib/srdb1: increased size of dialplan table repl_exp field to 256 chars

2017-02-16 Thread Juha Heinanen
Module: kamailio Branch: 5.0 Commit: 5a9716f9f4426375b0f99f499ade4d1785e0b600 URL: https://github.com/kamailio/kamailio/commit/5a9716f9f4426375b0f99f499ade4d1785e0b600 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-02-16T10:28:45+02:0

[sr-dev] git:master:fc358c8d: lib/srdb1: increased size of dialplan table repl_exp field to 256 chars

2017-02-16 Thread Juha Heinanen
Module: kamailio Branch: master Commit: fc358c8dabc227e86e4299d54ef6d3d29a46b096 URL: https://github.com/kamailio/kamailio/commit/fc358c8dabc227e86e4299d54ef6d3d29a46b096 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2017-02-16T10:25:30+02:0

[sr-dev] larger dialplan/repl_exp field

2017-02-15 Thread Juha Heinanen
I would need larger dialplan table repl_exp field. Its size is currently 64 chars: repl_exp string 64 Replacement expresion (sed like). Would it be OK to increase its size to 256 chars? If so, should table version number be also increased? -- Juha

Re: [sr-dev] Branch 5.0 created

2017-02-13 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > Where do you set the QUIET? When building the usual source code or when > building the debs? When building the debs. I added to debian/rules: # Uncomment this to turn on verbose mode. # export QUIET=verbose Might be a good idea to add those lines to official

Re: [sr-dev] Branch 5.0 created

2017-02-11 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > They were reported in the past, iirc, but apparently only by a specific > version of compiler, or maybe some specific flags to gcc. I set QUIET=verbose and got the exact command (below). -- Juha gcc -funroll-loops -Wcast-align -m64 -minline-all-stringops

Re: [sr-dev] Branch 5.0 created

2017-02-10 Thread Juha Heinanen
LD_EXTRA_OPTS="$(LD_EXTRA_OPTS)" \ group_include="opensipg" > > Cheers, > Daniel > > On 09/02/2017 22:48, Juha Heinanen wrote: > > Daniel-Constantin Mierla writes: > > > >> the GIT branch 5.0 has just been created, it will host the rel

[sr-dev] Branch 5.0 created

2017-02-09 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > the GIT branch 5.0 has just been created, it will host the release > series 5.0.x. To get this branch from GIT, you can use: > > git clone https://github.com/kamailio/kamailio.git kamailio > cd kamailio > git checkout -b 5.0 origin/5.0 > > Hopefully before the

Re: [sr-dev] select error without line number

2017-01-25 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > I pushed another commit for it. Thanks, now I got the line number: Jan 24 06:45:39 lohi /usr/bin/sip-proxy[8809]: BUG: [core/select.c:376]: resolve_select(): final node has SEL_PARAM_EXPECTED set (no more parameters available) Jan 24 06:45:39 lohi

Re: [sr-dev] select error without line number

2017-01-20 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > I just pushed a patch that should print cfg line and column in such > case. Can you test and see if works ok? The commit didn't change the syslog output. The commit checks if fix_rval returns negative result: case RVE_RVAL_OP:

[sr-dev] select error without line number

2017-01-19 Thread Juha Heinanen
while testing latest master, my sip proxy didn't start and produced these errors to syslog: Jan 19 21:32:18 lohi /usr/bin/sip-proxy[5382]: BUG: [core/select.c:376]: resolve_select(): final node has SEL_PARAM_EXPECTED set (no more parameters available) Jan 19 21:32:18 lohi

[sr-dev] unused function in rtpengine module

2017-01-04 Thread Juha Heinanen
CC (gcc) [M rtpengine.so] rtpengine.o rtpengine.c:1024:12: warning: 'rtpp_test_ping' defined but not used [-Wunused-function] static int rtpp_test_ping(struct rtpp_node *node) ^ ___ sr-dev mailing list

[sr-dev] unused vars in tm module

2017-01-04 Thread Juha Heinanen
CC (gcc) [M tm.so] rpc_uac.o rpc_uac.c: In function 't_uac_send': rpc_uac.c:679:7: warning: unused variable 'err_buf' [-Wunused-variable] char err_buf[MAX_REASON_LEN]; ^ rpc_uac.c:678:22: warning: unused variable 'err_ret' [-Wunused-variable] int ret, sip_error, err_ret,

[sr-dev] mi fields in struct module_exports

2017-01-04 Thread Juha Heinanen
What to do with mi field in struct module_exports? Should it be removed too? -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] new websocket warnings

2017-01-03 Thread Juha Heinanen
CC (gcc) [M websocket.so] ws_frame.o ws_frame.c:131:12: warning: 'str_status_empty_param' defined but not used [-Wunused-variable] static str str_status_empty_param = str_init("Empty connection ID parameter"); ^ ws_frame.c:132:12: warning: 'str_status_too_many_params'

[sr-dev] git:master:18631d09: modules/pua_rpc: added doc directory

2016-12-13 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 18631d093c9f189ea35db118db80ee954637d3ed URL: https://github.com/kamailio/kamailio/commit/18631d093c9f189ea35db118db80ee954637d3ed Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-12-13T10:01:51+02

[sr-dev] git:master:5429d68e: modules/pua_rpc: no need to include status code/reason phrase to 200 OK

2016-12-12 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 5429d68eaa6ad54586d428f34ec31c674cf8c060 URL: https://github.com/kamailio/kamailio/commit/5429d68eaa6ad54586d428f34ec31c674cf8c060 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-12-13T02:12:26+02

Re: [sr-dev] make appears to be in infinite loop

2016-12-12 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > You can change even now the line NAME?=kamcmd in the IFEQ blovk if you > want to edit the makefile for your local build system. That is exactly what I have done. -- Juha ___ sr-dev mailing list

Re: [sr-dev] make appears to be in infinite loop

2016-12-12 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > I pushed a small patch that allows specifying the name from command > line, like: > > cd utils/kamcmd/ > > make NAME=mykamcmd ... Looks like that make is called from for loop in src/Makefile and it is not that easy to add the param. Perhaps easier solution

[sr-dev] git:master:07310c63: modules/pua_rpc: return result as struct

2016-12-12 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 07310c634d322f137cff0103f776f0e397f963bb URL: https://github.com/kamailio/kamailio/commit/07310c634d322f137cff0103f776f0e397f963bb Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-12-12T11:28:11+02

[sr-dev] why utils dir is not in src dir?

2016-12-10 Thread Juha Heinanen
why utils dir is not in src dir? utils certainly contains important source code. -- juha ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] make appears to be in infinite loop

2016-12-09 Thread Juha Heinanen
Found the reason for the cleaning loop. It was caused by local renaming of kamcmd command. I need to do the rename, since I can have several kamailios running on the same host and each needs its own kamcmd command. It would be nice if $(MAIN_NAME) would also have effect on the name of kamcmd

Re: [sr-dev] make appears to be in infinite loop

2016-12-09 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > > Anyhow, it guess you try to build deb packages. In this case the rules > are from the debian specs, not from kamailio Makefiles, and they were > not updated, hopefully Victor Seva (or someone else with deb packaging > experience) will get the time to work on

Re: [sr-dev] make appears to be in infinite loop

2016-12-09 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > If you don't give the exact command you run and where, it's hard to > guess and impossible to help. The command I run at root is dpkg-buildpackage -rfakeroot -b -us -uc i.e., I try to build debian package. It produces /usr/bin/make maintainer-clean make[1]:

[sr-dev] make appears to be in infinite loop

2016-12-09 Thread Juha Heinanen
i tried to build latest master and make appears to be in infinite loop of rm commands (example below). -- juha ... Package json-c was not found in the pkg-config search path. Perhaps you should add the directory containing `json-c.pc' to the PKG_CONFIG_PATH environment variable No package

[sr-dev] git:master:4360021c: modules/pua_rpc: added pua_rpc module

2016-12-08 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 4360021cced3d81ab6588c5cecc17ff39063819e URL: https://github.com/kamailio/kamailio/commit/4360021cced3d81ab6588c5cecc17ff39063819e Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-12-09T09:50:42+02

Re: [sr-dev] how to compile a module

2016-12-08 Thread Juha Heinanen
Mikko Lehto writes: > > Why does modules dir also exist at root? > > Try running "git clean --dry-run". > You will propably get some hints of undeleted files. 'git clean -d' deleted modules and lib dirs at root. after that 'make modules=modules/ modules' worked from root. wonder why the dirs

Re: [sr-dev] how to compile a module

2016-12-08 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > What's the error you get? No error. Make from root claims that modules is up to date (see below). -- Juha /usr/src/orig/kamailio$ ls src/modules/lcr/ doc/hash.h lcr_mod.h lcr_rpc.hMakefile utils/ hash.c lcr_mod.c lcr_rpc.c makecfg.lst

Re: [sr-dev] how to compile a module

2016-12-08 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > > $ make modules=modules/ modules > > > This should still work, either executed in root folder or in src/. Most > of commands should just work as in the past. I tried on Debian and MacOS > during the restructuring and worked for me. OK, works from src, not

[sr-dev] how to compile a module

2016-12-07 Thread Juha Heinanen
stupid question: how to compile a module? this used to work earlier when issued at root: $ make modules=modules/ modules -- juha ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] Restructuring source code tree for v5.0

2016-12-05 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > The result will look like: > > /Makefile (only-forwarding make commands to various subdirs) > /README|INSTAL|LICENSE|... > /src/Makefile* (build makefiles) > /src/main.c (main c file to generate the binary) Why not also place main.c in core/? -- Juha

Re: [sr-dev] [SR-Users] VERY IMPORTANT: deciding when to remove the MI code

2016-12-01 Thread Juha Heinanen
Brandon Armstead writes: > +1 - I'm all for cleaning up any technical debt and moving on with more > normalized concept. I'm also in favor of getting rid on MI interface. I'm myself relying only on one MI command (pua_publish) for which an RPC command does not exist (or did not exist last time

[sr-dev] git:master:f27d7c1f: tcpops: update doc with new event routes

2016-10-11 Thread Juha Heinanen
There is tcp_enable_closed_event([conid]) function and the example shows how it is called upon registration. How about tcp_disable_closed_event? Would it be useful upon un-registration? -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org

[sr-dev] git:4.4:ce3d1dd8: presence: test if event is dialog for checking terminated state in presentity update

2016-09-17 Thread Juha Heinanen
Module: kamailio Branch: 4.4 Commit: ce3d1dd8360567fdf2c8e36fbb4a4238dab8f036 URL: https://github.com/kamailio/kamailio/commit/ce3d1dd8360567fdf2c8e36fbb4a4238dab8f036 Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-09-17T09

[sr-dev] presence/mwi error

2016-09-16 Thread Juha Heinanen
Juha Heinanen writes: > Sep 16 21:43:09 lohi /usr/bin/pres-serv[29250]: INFO: presence > [publish.c:305]: handle_publish(): * event name/content_type/type > 'message-summary'/'application/simple-message-summary'/2 > > Otherwise OK, but type does not match event name

[sr-dev] presence/mwi error

2016-09-16 Thread Juha Heinanen
Juha Heinanen writes: > I don't remember making any changes to the pua_publish request since > 4.5, where it works. Typo: 4.5 -> 4.4. -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/li

[sr-dev] presence/mwi error

2016-09-16 Thread Juha Heinanen
Using latest master, I'm getting error Sep 16 20:16:36 lohi /usr/bin/pres-serv[1744]: ERROR: presence [presentity.c:319]: parse_dialog_state_from_body(): failed to parse xml document 'Messages-Waiting: no#015#012Message-Account: sip:j...@vm.test.tutpro.com#015#012Voice-Message: 0/0

[sr-dev] unused var

2016-07-17 Thread Juha Heinanen
latest master gives this: CC (gcc) [M dialplan.so]dialplan.o dialplan.c: In function 'dp_reload_f': dialplan.c:460:18: warning: unused variable 'rpl_tree' [-Wunused-variable] struct mi_root* rpl_tree= NULL; ^ -- juha

[sr-dev] log_engine_type|data

2016-06-22 Thread Juha Heinanen
i read log_custom readme and found statement: It requires that core parameters log_engine_type to be set to 'udp' and log_engine_data to be set to target 'address:port'. but did not find these params in core cookbook. -- juha ___ sr-dev mailing

Re: [sr-dev] xmlrpc: missing declaration of vsnprintf

2016-06-08 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > A commit from master needed to be backported. Try again and see if works > ok now. Thanks, it works now. -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org

Re: [sr-dev] xmlrpc: missing declaration of vsnprintf

2016-06-08 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > are you giving special parameters to gcc version? Nothing special. > I just tested on debian jessie and all seems ok: > > # make modules modules=modules/xmlrpc > Makefile:56: modules.lst: No such file or directory > saving modules list... > CC (gcc) [M

[sr-dev] xmlrpc: missing declaration of vsnprintf

2016-06-08 Thread Juha Heinanen
If redeclaration of snprintf and vsnprintf causes trouble for clang, then they should be removed only when compiler is clang. -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] xmlrpc: missing declaration of vsnprintf

2016-06-08 Thread Juha Heinanen
Also 4.4 xmlrpc module fails to compile cleanly (Debian Jessie). -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] xmlrpc: missing declaration of vsnprintf

2016-06-08 Thread Juha Heinanen
Looks like cause of the warnings is this commit done 5 days ago: https://github.com/kamailio/kamailio/commit/efe5903cd17de9c5d94610e2345254e347e3aeb0 So I'm in big trouble: due to tls incompatibility, Kamailio 4.3 does not start without rebuild and I clean rebuild is not possible due to xmlrpc

[sr-dev] xmlrpc: missing declaration of vsnprintf

2016-06-08 Thread Juha Heinanen
While compiling Kamailio 4.3 on Debian Jessie, I got this: CC (gcc) [M xmlrpc.so] xmlrpc.o xmlrpc.c: In function 'set_fault': xmlrpc.c:886:2: warning: implicit declaration of function 'vsnprintf' [-Wimplicit-function-declaration] vsnprintf(buf, REASON_BUF_LEN, fmt, ap); ^ xmlrpc.c:

[sr-dev] jessie systemd is highjacking /etc/init.d/kamailio status

2016-05-21 Thread Juha Heinanen
i noticed that debian jessie systemd is highjacking /etc/init.d/kamailio status action. It can be prevented by including _SYSTEMCTL_SKIP_REDIRECT=true in the beginning of the script. that does not matter except that console messages are missing, but if someone wants to have a more

[sr-dev] git:4.3:45060ef7: xmlrpc: user libxml function to get content of node if type not provided for param

2016-05-03 Thread Juha Heinanen
Module: kamailio Branch: 4.3 Commit: 45060ef72edf5fdac9376150e60d56ff9458e4ff URL: https://github.com/kamailio/kamailio/commit/45060ef72edf5fdac9376150e60d56ff9458e4ff Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-05-03T18

[sr-dev] git:4.4:ea20338f: xmlrpc: user libxml function to get content of node if type not provided for param

2016-05-03 Thread Juha Heinanen
Module: kamailio Branch: 4.4 Commit: ea20338f8786fac2bb290a610f938fec8e7ece54 URL: https://github.com/kamailio/kamailio/commit/ea20338f8786fac2bb290a610f938fec8e7ece54 Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-05-03T18

[sr-dev] git:master:c395ae00: core: common check for ifa->ifa_addr in add_intefaces

2016-04-07 Thread Juha Heinanen
Could the crash fix be backported to 4.4, since that is where I noticed it? -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:4.4:d11cc80c: srdb1/schema: removed unique constraint from connection_idx

2016-03-23 Thread Juha Heinanen
Module: kamailio Branch: 4.4 Commit: d11cc80c3a4a2fdeb446837c550517e4f4a511a9 URL: https://github.com/kamailio/kamailio/commit/d11cc80c3a4a2fdeb446837c550517e4f4a511a9 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-03-23T16:37:38+02:00

Re: [sr-dev] git:master:4f9e0ffc: srdb1/schema: added connection_idx index to location table

2016-03-23 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > The index on (server_id, connection_id) was made unique with this, but > that doesn't work for udp/sctp, where connection id is always 0, > resulting in duplicate pairs for this constraint. The > constraint needs to be removed. OK, Juha

[sr-dev] git:master:d72ac166: srdb1/schema: removed unique constraint from connection_idx

2016-03-23 Thread Juha Heinanen
Module: kamailio Branch: master Commit: d72ac166b152ca6906248d67e2605d5b283fa01e URL: https://github.com/kamailio/kamailio/commit/d72ac166b152ca6906248d67e2605d5b283fa01e Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-03-23T16:32:06+

[sr-dev] git:master:00f108df: websocket: added $ws_conid pseudo variable

2016-03-10 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 00f108dfb38ed1267929b0cbead1ec9dc81b34ec URL: https://github.com/kamailio/kamailio/commit/00f108dfb38ed1267929b0cbead1ec9dc81b34ec Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-03-11T09:51:38+02:0

[sr-dev] LCR Commands Kamctl

2016-03-01 Thread Juha Heinanen
Alberto Sagredo provided changes to kamctl that implement lcr commands: http://lists.sip-router.org/pipermail/sr-users/2016-February/092011.html Can these be committed to current master or do they need to wait next release? -- Juha ___ sr-dev mailing

[sr-dev] getting id of closed ws connection using $conid pv

2016-03-01 Thread Juha Heinanen
before event_route[websocket:closed] is run, i can assign id of the closed ws connection to rcv.proto_reserved1 field of the faked message: static void wsconn_run_route(ws_connection_t *wsc) { ... fmsg = faked_msg_next(); wsc->rcv.proto_reserved1 = wsc->id; fmsg->rcv =

[sr-dev] git:master:b4e3c04b: Merge branch 'master' of https://github.com/kamailio/kamailio

2016-02-29 Thread Juha Heinanen
Module: kamailio Branch: master Commit: b4e3c04b601bef6841eb4616a17109296cd8aba1 URL: https://github.com/kamailio/kamailio/commit/b4e3c04b601bef6841eb4616a17109296cd8aba1 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-03-01T09:50:18+

[sr-dev] git:master:4f9e0ffc: srdb1/schema: added connection_idx index to location table

2016-02-29 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 4f9e0ffc7c320666de1d139658c6b645561e217d URL: https://github.com/kamailio/kamailio/commit/4f9e0ffc7c320666de1d139658c6b645561e217d Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-03-01T09:46:52+

Re: [sr-dev] rtpengine module notice

2016-02-09 Thread Juha Heinanen
smititelu writes: > The NOTICE message indicates that there is no entry in the hastable for > the *(callid, viabranch)* pair, and the node selection algorithm will > start running (select_rtpp_node_new()). This means that a new entry will > be added in the hastable, containing the selected

[sr-dev] rtpengine module notice

2016-02-08 Thread Juha Heinanen
Also rtpengine_delete() generates the same kind of message. Here are all of them from a new test: rtpengine_offer(...): Feb 9 03:38:24 lohi /usr/bin/sip-proxy[11835]: NOTICE: rtpengine [rtpengine.c:2792]: select_rtpp_node_old(): rtpengine hash table lookup failed to find node for calllen=16

[sr-dev] rtpengine module notice

2016-02-08 Thread Juha Heinanen
With latest rtpengine module, rtpengine_offer generates to syslog: Feb 9 03:05:03 lohi /usr/bin/sip-proxy[11835]: NOTICE: rtpengine [rtpengine.c:2792]: select_rtpp_node_old(): rtpengine hash table lookup failed to find node for calllen=16 callid=48e2ba8176be0f4a

[sr-dev] git:master:b4970c92: modules/tmx: regenerated README

2016-02-06 Thread Juha Heinanen
Module: kamailio Branch: master Commit: b4970c92cfed389d221782b091eb3984b7d6224f URL: https://github.com/kamailio/kamailio/commit/b4970c92cfed389d221782b091eb3984b7d6224f Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-02-07T08:28:51+02

[sr-dev] git:master:6e9f017d: tmx: Consistent naming of counters

2016-02-06 Thread Juha Heinanen
Juha Heinanen writes: > should these new names be updated in tmx readme section 7? currently > it has: README was not regenerated. -- Juha ___ sr-dev mailing list sr-dev@lists.sip-router.org http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:6e9f017d: tmx: Consistent naming of counters

2016-02-06 Thread Juha Heinanen
Stefan Mititelu writes: > tmx: Consistent naming of counters > > Simplify the names of the tmx counters to "rpl_*". should these new names be updated in tmx readme section 7? currently it has: 7. Statistics 7.1. received_replies 7.2. relayed_replies 7.3. local_replies 7.4.

Re: [sr-dev] active_watchers flags field

2016-02-05 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > It should keep the branch flags at the time of handling the subscribe - > that was the plan discussed with Luis, not sure it was here on sr-dev or > irc channel. OK, in that case the feature was not that I was looking for, i.e., be able to set arbitrary flags

[sr-dev] git:master:6297a071: core: avoid compiler warning about variable cl being set but not used

2016-02-03 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 6297a0718f6f04d7b018d1046d78985826271a68 URL: https://github.com/kamailio/kamailio/commit/6297a0718f6f04d7b018d1046d78985826271a68 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2016-02-04T04:43:40

[sr-dev] active_watchers flags field

2016-02-03 Thread Juha Heinanen
I noticed that active_watchers table now has a new flags field. The only reference I found to it in README is this: 7.1. $subs(attr) Access the attributes of handled subscription. It must be used after a successful call of “handle_subscription()” or in the following events. *

[sr-dev] git:master:93b5d2e5: usrloc: option to clean up database on write back/through modes

2016-01-30 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > usrloc: option to clean up database on write back/through modes > > - enabled by db_timer_clean parameter > - deletes expired records from database using secondary timer > - can be helpful if there was an error on db client after the db > operation was done

Re: [sr-dev] Panning next major release - v4.4

2016-01-08 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > replying on this announcement to get it fresh in mind for everyone and, > if it is needed, start relevant discussions for upcoming major release > 4.4. I too would like to have capability to reload tls certificates without restarting Kamailio. -- Juha

[sr-dev] git:4.2:5093871c: db_mysql: explicitely set the reconenct flag via mysql cloent api

2015-12-21 Thread Juha Heinanen
Module: kamailio Branch: 4.2 Commit: 5093871c5e612c6a949e2b2ab043fbc24551a2ba URL: https://github.com/kamailio/kamailio/commit/5093871c5e612c6a949e2b2ab043fbc24551a2ba Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2015-12-22T00

[sr-dev] git:4.2:74918723: db_mysql: extended the list of error code when to try to resend query

2015-12-21 Thread Juha Heinanen
Module: kamailio Branch: 4.2 Commit: 7491872348e5e3769ad8c7038fe1c7d15fe1e00c URL: https://github.com/kamailio/kamailio/commit/7491872348e5e3769ad8c7038fe1c7d15fe1e00c Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2015-12-22T00

[sr-dev] git:4.2:269e4455: db_mysql: print error number when failing to do mysql query

2015-12-21 Thread Juha Heinanen
Module: kamailio Branch: 4.2 Commit: 269e445550ee55b48ff1b92817a46d5084abad5b URL: https://github.com/kamailio/kamailio/commit/269e445550ee55b48ff1b92817a46d5084abad5b Author: Daniel-Constantin Mierla <mico...@gmail.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2015-12-22T00

[sr-dev] git:4.3:05ec88af: modules/lcr: backported ruri_user_avp fix

2015-12-14 Thread Juha Heinanen
Module: kamailio Branch: 4.3 Commit: 05ec88af66344c2897f34b51b0ca7fc6d66ee2a1 URL: https://github.com/kamailio/kamailio/commit/05ec88af66344c2897f34b51b0ca7fc6d66ee2a1 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2015-12-15T00:15:07+02:00

[sr-dev] git:master:d8991897: modules/lcr: delete possible earlier ruri_user_avp value

2015-12-04 Thread Juha Heinanen
Module: kamailio Branch: master Commit: d8991897275b5681bbe79cd4552b516ffa57894c URL: https://github.com/kamailio/kamailio/commit/d8991897275b5681bbe79cd4552b516ffa57894c Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2015-12-04T13:21:47+02

[sr-dev] git:master:a67bb24f: modules/lcr: changed location of ruri_user_avp delete

2015-12-04 Thread Juha Heinanen
Module: kamailio Branch: master Commit: a67bb24f24960e726523810da197cc7c816864f5 URL: https://github.com/kamailio/kamailio/commit/a67bb24f24960e726523810da197cc7c816864f5 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2015-12-05T00:21:59+02

[sr-dev] git:master:b56fae6d: Merge branch 'master' of https://github.com/kamailio/kamailio

2015-12-04 Thread Juha Heinanen
Module: kamailio Branch: master Commit: b56fae6d00f92628055301c4ac50a2929b49d3b4 URL: https://github.com/kamailio/kamailio/commit/b56fae6d00f92628055301c4ac50a2929b49d3b4 Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2015-12-05T00:23:42+

[sr-dev] git:master:8940cf95: modules/sdpops: added sloppy option to sdp_content function

2015-11-12 Thread Juha Heinanen
Module: kamailio Branch: master Commit: 8940cf950cb763ba5145a35d80ffc71f64f450ca URL: https://github.com/kamailio/kamailio/commit/8940cf950cb763ba5145a35d80ffc71f64f450ca Author: Juha Heinanen <j...@tutpro.com> Committer: Juha Heinanen <j...@tutpro.com> Date: 2015-11-13T08:50:01+02

Re: [sr-dev] faster sdp_content() test

2015-11-12 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > I think still having the ability to parse is good, because it makes sure > that the sdp is valid and it is quite common that later it will be > parsed anyhow (e.g., for rtp proxying). based on my reading of rtpengine_offer/answer, they do not parse the body and

[sr-dev] faster sdp_content() test

2015-11-12 Thread Juha Heinanen
i wrote new version of sdp_content() test that does not parse the whole sdp like the current one does. instead it checks if request has a body and content type is either missing or it is application/sdp or content type is multipart/mixed and boxy has string application/sdp. could this one

Re: [sr-dev] faster sdp_content() test

2015-11-12 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > Also, there is another function that checks the type of the body: > > has_body("application/sdp") > > Not sure if that works for multi-part body. i checked that. it makes its decision basedon content-type header. -- juha

Re: [sr-dev] a new master crash in tm module

2015-09-23 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > Can you the test repeat with last master branch? I pushed a cleanup > patch there. it now took longer before i got crash. -- juha (gdb) where #0 0x00641d80 in fm_insert_free (qm=0x7f70379f9000, frag=0x7f70386a8a18) at mem/f_malloc.c:242 #1

Re: [sr-dev] a new master crash in tm module

2015-09-23 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > This one looks like a double free of the transaction structure. > > I pushed a patch to master branch that will print a WARN message for > each of transaction free operations. Can you test with it, wait for > crash and then send me all the logs containing: > >

Re: [sr-dev] a new master crash in tm module

2015-09-23 Thread Juha Heinanen
Daniel-Constantin Mierla writes: > Have you start it with -x qm or without? > > >From the core of the next bt, can you get: > > frame 0 > p *t > p *p_msg > info locals when i start k with -x qm, i have not managed to get a crash. however, if i start k without -x qm, it crashes reliably, for

  1   2   3   4   5   6   7   8   9   10   >