[sr-dev] Kamailio v5.1.1 Released

2018-01-22 Thread Daniel-Constantin Mierla 
Hello,

Kamailio SIP Server v5.1.1 stable release is out.

This is a maintenance release of the latest stable branch, 5.1, that
includes fixes since the release of v5.1.0. There is no change to
database schema or configuration language structure that you have to do
on previous installations of v5.1.x. Deployments running previous v5.1.x
versions are strongly recommended to be upgraded to v5.1.1.

For more details about version 5.1.1 (including links and guidelines to
download the tarball or from GIT repository), visit:

  * https://www.kamailio.org/w/2018/01/kamailio-v5-1-1-released/

RPM, Debian/Ubuntu packages will be available soon as well.

Many thanks to all contributing and using Kamailio!

Cheers,
Daniel

-- 
Daniel-Constantin Mierla
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio Advanced Training - www.asipto.com
Kamailio World Conference - May 14-16, 2018 - www.kamailioworld.com

___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:33cb1f8b: ChangeLog: content updated for v5.1.1

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: 33cb1f8b1224bedcd49d2bce9fa71cef30404b68
URL: 
https://github.com/kamailio/kamailio/commit/33cb1f8b1224bedcd49d2bce9fa71cef30404b68

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T15:37:59+01:00

ChangeLog: content updated for v5.1.1

---

Modified: ChangeLog

---

Diff:  
https://github.com/kamailio/kamailio/commit/33cb1f8b1224bedcd49d2bce9fa71cef30404b68.diff
Patch: 
https://github.com/kamailio/kamailio/commit/33cb1f8b1224bedcd49d2bce9fa71cef30404b68.patch


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:d3c6227d: Makefile.defs: version set to 5.1.1

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: d3c6227d998f46388338de5a5be8684e02c529a2
URL: 
https://github.com/kamailio/kamailio/commit/d3c6227d998f46388338de5a5be8684e02c529a2

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T15:31:46+01:00

Makefile.defs: version set to 5.1.1

---

Modified: src/Makefile.defs

---

Diff:  
https://github.com/kamailio/kamailio/commit/d3c6227d998f46388338de5a5be8684e02c529a2.diff
Patch: 
https://github.com/kamailio/kamailio/commit/d3c6227d998f46388338de5a5be8684e02c529a2.patch

---

diff --git a/src/Makefile.defs b/src/Makefile.defs
index 1238e4f99d..5bfc0bf792 100644
--- a/src/Makefile.defs
+++ b/src/Makefile.defs
@@ -106,7 +106,7 @@ INSTALL_FLAVOUR=$(FLAVOUR)
 # version number
 VERSION = 5
 PATCHLEVEL = 1
-SUBLEVEL = 0
+SUBLEVEL = 1
 EXTRAVERSION =
 
 # memory manager switcher


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:b631f7e7: pkg/distros: updated version to 5.1.1

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: b631f7e715658fcff15bee6daa889519aa687adf
URL: 
https://github.com/kamailio/kamailio/commit/b631f7e715658fcff15bee6daa889519aa687adf

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T15:30:33+01:00

pkg/distros: updated version to 5.1.1

---

Modified: pkg/kamailio/alpine/APKBUILD
Modified: pkg/kamailio/freebsd/Makefile
Modified: pkg/kamailio/openbsd/Makefile
Modified: pkg/kamailio/solaris/pkginfo

---

Diff:  
https://github.com/kamailio/kamailio/commit/b631f7e715658fcff15bee6daa889519aa687adf.diff
Patch: 
https://github.com/kamailio/kamailio/commit/b631f7e715658fcff15bee6daa889519aa687adf.patch

---

diff --git a/pkg/kamailio/alpine/APKBUILD b/pkg/kamailio/alpine/APKBUILD
index eebe8d64c3..1e98751427 100644
--- a/pkg/kamailio/alpine/APKBUILD
+++ b/pkg/kamailio/alpine/APKBUILD
@@ -4,7 +4,7 @@
 # Maintainer: Nathan Angelacos 
 
 pkgname=kamailio
-pkgver=5.1.0
+pkgver=5.1.1
 pkgrel=0
 
 # If building from a git snapshot, specify the gitcommit
diff --git a/pkg/kamailio/freebsd/Makefile b/pkg/kamailio/freebsd/Makefile
index 54aa0b760f..03f821c568 100644
--- a/pkg/kamailio/freebsd/Makefile
+++ b/pkg/kamailio/freebsd/Makefile
@@ -6,7 +6,7 @@
 #
 
 PORTNAME=  kamailio
-PORTVERSION=   5.1.0
+PORTVERSION=   5.1.1
 CATEGORIES=net
 MASTER_SITES=  http://kamailio.org/pub/kamailio/${PORTVERSION}/src/
 DISTNAME=  ${PORTNAME}-${PORTVERSION}-tls_src
diff --git a/pkg/kamailio/openbsd/Makefile b/pkg/kamailio/openbsd/Makefile
index 7f9e292054..3c8053f984 100644
--- a/pkg/kamailio/openbsd/Makefile
+++ b/pkg/kamailio/openbsd/Makefile
@@ -12,7 +12,7 @@ COMMENT-carrierroute =kamailio carrierroute module
 COMMENT-snmpstats =kamailio snmpstats module
 COMMENT-perl = kamailio perl modules
 
-VERSION =  5.1.0
+VERSION =  5.1.1
 DISTNAME = kamailio-${VERSION}
 PKGNAME-main = kamailio-${VERSION}
 PKGNAME-mysql =kamailio-mysql-${VERSION}
diff --git a/pkg/kamailio/solaris/pkginfo b/pkg/kamailio/solaris/pkginfo
index c1a1960fa7..66f5c4cbc7 100644
--- a/pkg/kamailio/solaris/pkginfo
+++ b/pkg/kamailio/solaris/pkginfo
@@ -1,6 +1,6 @@
 PKG=Kamailio
 ARCH=sparc
-VERSION=5.1.0
+VERSION=5.1.1
 CATEGORY=application
 EMAIL=mico...@gmail.com
 NAME=Kamailio is a very fast and flexible SIP (RFC3261) proxy server


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:246b40b5: pkg/rpms: updated version in spec files

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: 246b40b5048a6c751747fe827f046bb8e0034bed
URL: 
https://github.com/kamailio/kamailio/commit/246b40b5048a6c751747fe827f046bb8e0034bed

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T15:30:09+01:00

pkg/rpms: updated version in spec files

---

Modified: pkg/kamailio/centos/6/kamailio.spec
Modified: pkg/kamailio/centos/7/kamailio.spec
Modified: pkg/kamailio/fedora/17/kamailio.spec
Modified: pkg/kamailio/obs/kamailio.spec
Modified: pkg/kamailio/oracle/el6/kamailio.spec
Modified: pkg/kamailio/oracle/el7/kamailio.spec
Modified: pkg/kamailio/rpm/kamailio.spec-4.1
Modified: pkg/kamailio/rpm/kamailio.spec.CenOS
Modified: pkg/kamailio/rpm/kamailio.spec.SuSE

---

Diff:  
https://github.com/kamailio/kamailio/commit/246b40b5048a6c751747fe827f046bb8e0034bed.diff
Patch: 
https://github.com/kamailio/kamailio/commit/246b40b5048a6c751747fe827f046bb8e0034bed.patch

---

diff --git a/pkg/kamailio/centos/6/kamailio.spec 
b/pkg/kamailio/centos/6/kamailio.spec
index df4419929e..d6df4c05a6 100644
--- a/pkg/kamailio/centos/6/kamailio.spec
+++ b/pkg/kamailio/centos/6/kamailio.spec
@@ -1,5 +1,5 @@
 %define name   kamailio
-%define ver5.1.0
+%define ver5.1.1
 %define rel0%{dist}
 
 
diff --git a/pkg/kamailio/centos/7/kamailio.spec 
b/pkg/kamailio/centos/7/kamailio.spec
index 2d0099b901..147908d046 100644
--- a/pkg/kamailio/centos/7/kamailio.spec
+++ b/pkg/kamailio/centos/7/kamailio.spec
@@ -1,5 +1,5 @@
 %define namekamailio
-%define ver 5.1.0
+%define ver 5.1.1
 %define rel dev6%{dist}
 %bcond_with dnssec
 
diff --git a/pkg/kamailio/fedora/17/kamailio.spec 
b/pkg/kamailio/fedora/17/kamailio.spec
index 446ac8bfb3..224094e16b 100644
--- a/pkg/kamailio/fedora/17/kamailio.spec
+++ b/pkg/kamailio/fedora/17/kamailio.spec
@@ -1,5 +1,5 @@
 %define name   kamailio
-%define ver5.1.0
+%define ver5.1.1
 %define rel0%{dist}
 
 
diff --git a/pkg/kamailio/obs/kamailio.spec b/pkg/kamailio/obs/kamailio.spec
index d1c5466cf4..68be69c6c8 100644
--- a/pkg/kamailio/obs/kamailio.spec
+++ b/pkg/kamailio/obs/kamailio.spec
@@ -1,5 +1,5 @@
 %define namekamailio
-%define ver 5.1.0
+%define ver 5.1.1
 %define rel 0%{dist}
 
 %if 0%{?fedora} == 25
diff --git a/pkg/kamailio/oracle/el6/kamailio.spec 
b/pkg/kamailio/oracle/el6/kamailio.spec
index e5807b5ada..23f6de0c0c 100644
--- a/pkg/kamailio/oracle/el6/kamailio.spec
+++ b/pkg/kamailio/oracle/el6/kamailio.spec
@@ -1,5 +1,5 @@
 %define namekamailio
-%define ver 5.1.0
+%define ver 5.1.1
 %define rel 0
 %define _sharedir %{_prefix}/share
 
diff --git a/pkg/kamailio/oracle/el7/kamailio.spec 
b/pkg/kamailio/oracle/el7/kamailio.spec
index 0547e8b176..a0da154b3c 100644
--- a/pkg/kamailio/oracle/el7/kamailio.spec
+++ b/pkg/kamailio/oracle/el7/kamailio.spec
@@ -1,5 +1,5 @@
 %define namekamailio
-%define ver 5.1.0
+%define ver 5.1.1
 %define rel 0
 %define _sharedir %{_prefix}/share
 
diff --git a/pkg/kamailio/rpm/kamailio.spec-4.1 
b/pkg/kamailio/rpm/kamailio.spec-4.1
index 3acccefe52..b6e3ef0282 100644
--- a/pkg/kamailio/rpm/kamailio.spec-4.1
+++ b/pkg/kamailio/rpm/kamailio.spec-4.1
@@ -1,5 +1,5 @@
 %define namekamailio
-%define ver 5.1.0
+%define ver 5.1.1
 %define rel 0
 
 %define EXCLUDED_MODULES   mysql jabber cpl-c avp_radius auth_radius 
group_radius uri_radius pa postgres osp tlsops unixodbc
diff --git a/pkg/kamailio/rpm/kamailio.spec.CenOS 
b/pkg/kamailio/rpm/kamailio.spec.CenOS
index dd4d992106..dd5325ee76 100644
--- a/pkg/kamailio/rpm/kamailio.spec.CenOS
+++ b/pkg/kamailio/rpm/kamailio.spec.CenOS
@@ -1,5 +1,5 @@
 %define namekamailio
-%define ver 5.1.0
+%define ver 5.1.1
 %define rel 0
 %define _sharedir %{_prefix}/share
 
diff --git a/pkg/kamailio/rpm/kamailio.spec.SuSE 
b/pkg/kamailio/rpm/kamailio.spec.SuSE
index ef0eb41161..c679d99f22 100644
--- a/pkg/kamailio/rpm/kamailio.spec.SuSE
+++ b/pkg/kamailio/rpm/kamailio.spec.SuSE
@@ -1,5 +1,5 @@
 %define namekamailio
-%define ver 5.1.0
+%define ver 5.1.1
 %define rel 0
 
 %define EXCLUDED_MODULES   mysql jabber cpl-c auth_radius misc_radius 
peering postgress pa unixodbc osp tlsops


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] Contributing platform specific code to Kamailio

2018-01-22 Thread Tsvetomir Dimitrov 
Hello Carsten,

I promised to share the code with you. I wanted to push it in a better
shape, but I've got some difficulties, which take me more time than
expected. You can check it here:

https://github.com/tdimitrov/kamailio/tree/ipsec-wip

The code is far from production ready, but my main priority for now is to
make it work with real phone. My issue in nutshell: when the UE sends IPSec
protected REGISTER I can see the message reaching P-CSCF (in wireshark),
but the REGISTER is not delivered to the kamailio process. The ipsec itself
is initialised correctly (at least I see no errors). I think the issue is
that the UDP checksum of the REGISTER is wrong due to a NAT. I see lots of
lines like this in dmesg:

[11782894.569952] UDP: bad checksum. From 192.168.178.61:6177 to
192.168.178.167:5061 ulen 1332

Currently I'm working on a fix for this. I think there is a socket option
to disable UDP checksum validation, but I want to avoid this. If you have
got experience with such issues, any help is highly appreciated :)

Besides this there is some more work on the module, but nothing serious:
- IPSec tunnels are not destroyed on deregister.
- IPSec tunnels are not cleaned up in case of ungraceful kamailio shutdown.
- Logic to allocate unique local IPSec ports to each UE is required.
- Some hardcoded params should be replaced with module config options.

I'll update this thread when I have got any significant progress.

Best regards,
Tsvetomir

On Mon, Dec 11, 2017 at 5:15 PM, Tsvetomir Dimitrov 
wrote:

> Hi Carsten,
>
> I'm working on this and I have got some progress, but unplanned two week
> sick leave delayed my work a lot. What I have done so far:
> - Fixed saving of security parameters in ims_usrloc_pcscf. My previous
> patch didn't handle SHM allocation correctly. This is can be merged so I'll
> open a pull request soon.
> - New module which handles ipsec tunnel creation. For now the module
> successfully registers SAs and policy via xfrm (no external bash scripts
> involved) on incoming REGISTER.
>
> What needs to be done:
> - Cleanup of ipsec SAs and policy on subscriber detach, PCSCF graceful
> shut down and PCSCF not so graceful shutdown.
> - Module parameters for everything - for now all params are hardcoded.
>
> The code is not very usable at the moment, mainly because there are a lot
> of hardcoded parameters. I can share it if you wish.
>
> Best regards,
> Tsvetomir
>
>
> On Mon, Dec 11, 2017 at 3:25 PM, Carsten Bock 
> wrote:
>
>> Hi Tsvetomir,
>>
>> any updates regarding this? I believe otherwise, I would assign
>> ressources from our side for this in February/March next year.
>> However, I would want to avoid double work.
>>
>> You can share it privately with me, if you don't want to publish it yet.
>>
>> Thanks,
>> Carsten
>>
>> 2017-10-19 9:49 GMT+02:00 Tsvetomir Dimitrov :
>> > Hi Carsten,
>> >
>> > Thanks for your responce and please excuse my late reply too. I'm still
>> > working on the changes and will make a pull request as soon as I am
>> ready.
>> > It will be a separate module which handles the IPSec tunnel
>> creation/tear
>> > down, so that ims_register_pcscf won't be polluted with platform
>> specific
>> > functionality. You are right, that new module can be ifdef-ed and
>> replaced
>> > with something *BSD specific or whatever OS someone wants to use.
>> >
>> > Best regards,
>> > Tsvetomir
>> >
>> > On Fri, Oct 13, 2017 at 6:21 AM, Carsten Bock 
>> wrote:
>> >>
>> >> Hi Tsvetomir,
>> >>
>> >> sorry for the late reply. I assume this mail got lost a bit in the
>> >> days of Astricon. I even asked Daniel about this mail during Astricon,
>> >> but he hadn't seen it yet. Right now, I'm officially on holiday
>> >>
>> >> Can you please provide a Pull-Request for the changes?
>> >>
>> >> From my perspective, it is likely fine to have a Linux-Only module, it
>> >> might not be the first one. If you can encapsulate your extensions
>> >> with some IFDEF's, so the functionality can be disabled on non-Linux,
>> >> then that would be fine with me.
>> >>
>> >> It would be great, if Daniel or anyone else from the Management-Group
>> >> could answer or comment this one as well??
>> >>
>> >> Thanks,
>> >> Carsten
>> >>
>> >> 2017-10-04 10:14 GMT-04:00 Tsvetomir Dimitrov > >:
>> >> > Hello,
>> >> >
>> >> > I am working on a functionality which handles ipsec tunel creation
>> for
>> >> > VoLTE
>> >> > registration and I'd like to contribute it to the project. However
>> the
>> >> > code
>> >> > is heavily Linux specific - uses xfrm framework, so it won't compile
>> on
>> >> > distribution with older kernels and definitely won't compile on *BSD.
>> >> >
>> >> > How problematic is this? How to handle this implementation so that it
>> >> > gets
>> >> > merged?
>> >> >
>> >> > Right now I can see two options:
>> >> > 1. Implement the functionality in ims_register_pcscf.
>> >> > 2.

[sr-dev] git:5.1:7f8a398d: modules/sipcapture fixed LM_ERR to LM_DBG

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: 7f8a398dd18d240efd8eb4377251a8752b321a1e
URL: 
https://github.com/kamailio/kamailio/commit/7f8a398dd18d240efd8eb4377251a8752b321a1e

Author: Alexandr Dubovikov 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T12:16:00+01:00

modules/sipcapture fixed LM_ERR to LM_DBG

(cherry picked from commit c2949fe2dc4d1156657cd6bc1683da83eda2a809)

---

Modified: src/modules/sipcapture/sipcapture.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/7f8a398dd18d240efd8eb4377251a8752b321a1e.diff
Patch: 
https://github.com/kamailio/kamailio/commit/7f8a398dd18d240efd8eb4377251a8752b321a1e.patch

---

diff --git a/src/modules/sipcapture/sipcapture.c 
b/src/modules/sipcapture/sipcapture.c
index 171e1d035c..6e6faa1ad4 100644
--- a/src/modules/sipcapture/sipcapture.c
+++ b/src/modules/sipcapture/sipcapture.c
@@ -3030,7 +3030,7 @@ static int pv_parse_hep_name(pv_spec_p sp, str *in)
if(sp == NULL || in == NULL || in->len <= 0)
return -1;
 
-   LM_ERR("REQUEST, PRE, %.*s\n", in->len, in->s);
+   LM_DBG("REQUEST, PRE, %.*s\n", in->len, in->s);
 
switch(in->len) {
case 5: {


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:501fdfe9: sctp: removed unused goto label

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: 501fdfe94211eb525d775738a2f6c77499033447
URL: 
https://github.com/kamailio/kamailio/commit/501fdfe94211eb525d775738a2f6c77499033447

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T12:15:13+01:00

sctp: removed unused goto label

(cherry picked from commit f6232ee6d5d7fed67b6510c8c5ab8e7f6273274e)

---

Modified: src/modules/sctp/sctp_server.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/501fdfe94211eb525d775738a2f6c77499033447.diff
Patch: 
https://github.com/kamailio/kamailio/commit/501fdfe94211eb525d775738a2f6c77499033447.patch

---

diff --git a/src/modules/sctp/sctp_server.c b/src/modules/sctp/sctp_server.c
index 0e0b6eab97..66c71b7889 100644
--- a/src/modules/sctp/sctp_server.c
+++ b/src/modules/sctp/sctp_server.c
@@ -797,9 +797,6 @@ static int sctp_init_sock_opt_common(int s, int af)
sctp_err++;
goto error; /* critical */
}
-#ifdef __OS_linux
-ev_success:
-#endif
 #else
 #error no sctp lib support for SCTP_EVENTS, consider upgrading
 #endif /* SCTP_EVENTS */


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:f6232ee6: sctp: removed unused goto label

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: master
Commit: f6232ee6d5d7fed67b6510c8c5ab8e7f6273274e
URL: 
https://github.com/kamailio/kamailio/commit/f6232ee6d5d7fed67b6510c8c5ab8e7f6273274e

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:57:05+01:00

sctp: removed unused goto label

---

Modified: src/modules/sctp/sctp_server.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/f6232ee6d5d7fed67b6510c8c5ab8e7f6273274e.diff
Patch: 
https://github.com/kamailio/kamailio/commit/f6232ee6d5d7fed67b6510c8c5ab8e7f6273274e.patch

---

diff --git a/src/modules/sctp/sctp_server.c b/src/modules/sctp/sctp_server.c
index 0e0b6eab97..66c71b7889 100644
--- a/src/modules/sctp/sctp_server.c
+++ b/src/modules/sctp/sctp_server.c
@@ -797,9 +797,6 @@ static int sctp_init_sock_opt_common(int s, int af)
sctp_err++;
goto error; /* critical */
}
-#ifdef __OS_linux
-ev_success:
-#endif
 #else
 #error no sctp lib support for SCTP_EVENTS, consider upgrading
 #endif /* SCTP_EVENTS */


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:589e8a83: Merge pull request #1406 from adubovikov/master

2018-01-22 Thread GitHub 
Module: kamailio
Branch: master
Commit: 589e8a8345da5160c1693c1c6808be4588b625aa
URL: 
https://github.com/kamailio/kamailio/commit/589e8a8345da5160c1693c1c6808be4588b625aa

Author: Alexandr Dubovikov 
Committer: GitHub 
Date: 2018-01-22T11:32:43+01:00

Merge pull request #1406 from adubovikov/master

modules/sipcapture fixed LM_ERR to LM_DBG

---

Modified: src/modules/sipcapture/sipcapture.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/589e8a8345da5160c1693c1c6808be4588b625aa.diff
Patch: 
https://github.com/kamailio/kamailio/commit/589e8a8345da5160c1693c1c6808be4588b625aa.patch

---

diff --git a/src/modules/sipcapture/sipcapture.c 
b/src/modules/sipcapture/sipcapture.c
index 171e1d035c..6e6faa1ad4 100644
--- a/src/modules/sipcapture/sipcapture.c
+++ b/src/modules/sipcapture/sipcapture.c
@@ -3030,7 +3030,7 @@ static int pv_parse_hep_name(pv_spec_p sp, str *in)
if(sp == NULL || in == NULL || in->len <= 0)
return -1;
 
-   LM_ERR("REQUEST, PRE, %.*s\n", in->len, in->s);
+   LM_DBG("REQUEST, PRE, %.*s\n", in->len, in->s);
 
switch(in->len) {
case 5: {


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] [kamailio/kamailio] modules/sipcapture fixed LM_ERR to LM_DBG (#1406)

2018-01-22 Thread Alexandr Dubovikov 
Merged #1406.

-- 
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/1406#event-1434828615___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:c2949fe2: modules/sipcapture fixed LM_ERR to LM_DBG

2018-01-22 Thread GitHub 
Module: kamailio
Branch: master
Commit: c2949fe2dc4d1156657cd6bc1683da83eda2a809
URL: 
https://github.com/kamailio/kamailio/commit/c2949fe2dc4d1156657cd6bc1683da83eda2a809

Author: Alexandr Dubovikov 
Committer: GitHub 
Date: 2018-01-22T10:52:15+01:00

modules/sipcapture fixed LM_ERR to LM_DBG

---

Modified: src/modules/sipcapture/sipcapture.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/c2949fe2dc4d1156657cd6bc1683da83eda2a809.diff
Patch: 
https://github.com/kamailio/kamailio/commit/c2949fe2dc4d1156657cd6bc1683da83eda2a809.patch

---

diff --git a/src/modules/sipcapture/sipcapture.c 
b/src/modules/sipcapture/sipcapture.c
index 171e1d035c..6e6faa1ad4 100644
--- a/src/modules/sipcapture/sipcapture.c
+++ b/src/modules/sipcapture/sipcapture.c
@@ -3030,7 +3030,7 @@ static int pv_parse_hep_name(pv_spec_p sp, str *in)
if(sp == NULL || in == NULL || in->len <= 0)
return -1;
 
-   LM_ERR("REQUEST, PRE, %.*s\n", in->len, in->s);
+   LM_DBG("REQUEST, PRE, %.*s\n", in->len, in->s);
 
switch(in->len) {
case 5: {


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:70bc83c5: siptrace: fixed unused variable warnings

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: 70bc83c52c6bfe38fd5daa60c3aa449521a063fa
URL: 
https://github.com/kamailio/kamailio/commit/70bc83c52c6bfe38fd5daa60c3aa449521a063fa

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:27:19+01:00

siptrace: fixed unused variable warnings

(cherry picked from commit 7dae4ec9e4be87d5a3406fe2ddeb98328beb1125)

---

Modified: src/modules/siptrace/siptrace.c
Modified: src/modules/siptrace/siptrace_send.h

---

Diff:  
https://github.com/kamailio/kamailio/commit/70bc83c52c6bfe38fd5daa60c3aa449521a063fa.diff
Patch: 
https://github.com/kamailio/kamailio/commit/70bc83c52c6bfe38fd5daa60c3aa449521a063fa.patch

---

diff --git a/src/modules/siptrace/siptrace.c b/src/modules/siptrace/siptrace.c
index 924c4f52e7..851bf6e0f8 100644
--- a/src/modules/siptrace/siptrace.c
+++ b/src/modules/siptrace/siptrace.c
@@ -1016,7 +1016,6 @@ static void trace_onreq_out(struct cell *t, int type, 
struct tmcb_params *ps)
siptrace_data_t sto;
sip_msg_t *msg;
ip_addr_t to_ip;
-   int len;
dest_info_t *dst;
 
if(t == NULL || ps == NULL) {
@@ -1334,7 +1333,6 @@ static void trace_sl_onreply_out(sl_cbp_t *slcbp)
siptrace_data_t sto;
sip_msg_t *msg;
ip_addr_t to_ip;
-   int len;
char statusbuf[5];
 
if(slcbp == NULL || slcbp->req == NULL) {
diff --git a/src/modules/siptrace/siptrace_send.h 
b/src/modules/siptrace/siptrace_send.h
index 30c33e53b5..867d3995b0 100644
--- a/src/modules/siptrace/siptrace_send.h
+++ b/src/modules/siptrace/siptrace_send.h
@@ -68,8 +68,9 @@ int trace_send_duplicate(char *buf, int len, struct dest_info 
*dst2);
 #define siptrace_copy_proto(vproto, vbuf) do { \
int __olen; \
siptrace_copy_proto_olen(vproto, vbuf, __olen); \
+   (void)__olen; \
} while(0)
 
 char* siptrace_proto_name(int vproto);
 
-#endif
\ No newline at end of file
+#endif


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:e9549e27: rls: removed local variable contact, unused inside rls_handle_subscribe()

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: e9549e27efaf920e6f0b3d4c7a81129b705326e9
URL: 
https://github.com/kamailio/kamailio/commit/e9549e27efaf920e6f0b3d4c7a81129b705326e9

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:26:32+01:00

rls: removed local variable contact, unused inside rls_handle_subscribe()

(cherry picked from commit 2aebb2bf1f3d71a0dc7eca847af2aa127ff54453)

---

Modified: src/modules/rls/subscribe.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/e9549e27efaf920e6f0b3d4c7a81129b705326e9.diff
Patch: 
https://github.com/kamailio/kamailio/commit/e9549e27efaf920e6f0b3d4c7a81129b705326e9.patch

---

diff --git a/src/modules/rls/subscribe.c b/src/modules/rls/subscribe.c
index 9222dc72b8..a96d1674f7 100644
--- a/src/modules/rls/subscribe.c
+++ b/src/modules/rls/subscribe.c
@@ -479,7 +479,6 @@ int rls_handle_subscribe(struct sip_msg* msg, str 
watcher_user, str watcher_doma
pres_ev_t* event = NULL;
int err_ret = -1;
int ret = to_presence_code;
-   str* contact = NULL;
xmlDocPtr doc = NULL;
xmlNodePtr service_node = NULL;
unsigned int hash_code=0;
@@ -802,13 +801,6 @@ int rls_handle_subscribe(struct sip_msg* msg, str 
watcher_user, str watcher_doma
ret = 1;
 stop:
 forpresence:
-   if(contact!=NULL)
-   {
-   if(contact->s!=NULL)
-   pkg_free(contact->s);
-   pkg_free(contact);
-   }
-
if(subs.pres_uri.s!=NULL)
pkg_free(subs.pres_uri.s);
if(subs.record_route.s!=NULL)
@@ -829,13 +821,6 @@ int rls_handle_subscribe(struct sip_msg* msg, str 
watcher_user, str watcher_doma
 
 error:
LM_ERR("occurred in rls_handle_subscribe\n");
-
-   if(contact!=NULL)
-   {
-   if(contact->s!=NULL)
-   pkg_free(contact->s);
-   pkg_free(contact);
-   }
if(subs.pres_uri.s!=NULL)
pkg_free(subs.pres_uri.s);
 


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:5c19d9bc: uri_db: ki_check_uri_realm() has three string params for kemi exports

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: 5c19d9bcf87a4c5595aa4b0706c45917ca75858d
URL: 
https://github.com/kamailio/kamailio/commit/5c19d9bcf87a4c5595aa4b0706c45917ca75858d

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:26:08+01:00

uri_db: ki_check_uri_realm() has three string params for kemi exports

(cherry picked from commit 4fa92be97bca8da3ce5bf5181ef9262006b4fb9d)

---

Modified: src/modules/uri_db/uri_db.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/5c19d9bcf87a4c5595aa4b0706c45917ca75858d.diff
Patch: 
https://github.com/kamailio/kamailio/commit/5c19d9bcf87a4c5595aa4b0706c45917ca75858d.patch

---

diff --git a/src/modules/uri_db/uri_db.c b/src/modules/uri_db/uri_db.c
index 583303c9ee..aa8f1ac99a 100644
--- a/src/modules/uri_db/uri_db.c
+++ b/src/modules/uri_db/uri_db.c
@@ -232,7 +232,7 @@ static sr_kemi_t sr_kemi_uri_db_exports[] = {
},
{ str_init("uri_db"), str_init("check_uri_realm"),
SR_KEMIP_INT, ki_check_uri_realm,
-   { SR_KEMIP_STR, SR_KEMIP_NONE, SR_KEMIP_NONE,
+   { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_STR,
SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE }
},
 
@@ -244,4 +244,4 @@ int mod_register(char *path, int *dlflags, void *p1, void 
*p2)
 {
sr_kemi_modules_add(sr_kemi_uri_db_exports);
return 0;
-}
\ No newline at end of file
+}


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:730fad39: rls: rls_get_service_list() - set to NULL the content of rootdoc pointer

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: 730fad39599c39b1a5c3faa1d9bb9c49db61439b
URL: 
https://github.com/kamailio/kamailio/commit/730fad39599c39b1a5c3faa1d9bb9c49db61439b

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:26:19+01:00

rls: rls_get_service_list() - set to NULL the content of rootdoc pointer

- its a function parameter and expects to be set by the function

(cherry picked from commit 5e58d48c980f5e413f47271dd342ee0630722bbb)

---

Modified: src/modules/rls/subscribe.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/730fad39599c39b1a5c3faa1d9bb9c49db61439b.diff
Patch: 
https://github.com/kamailio/kamailio/commit/730fad39599c39b1a5c3faa1d9bb9c49db61439b.patch

---

diff --git a/src/modules/rls/subscribe.c b/src/modules/rls/subscribe.c
index c895bd1a93..9222dc72b8 100644
--- a/src/modules/rls/subscribe.c
+++ b/src/modules/rls/subscribe.c
@@ -272,12 +272,10 @@ int rls_get_service_list(str *service_uri, str *user, str 
*domain,
LM_DBG("service uri %.*s not found in rl document for user"
" sip:%.*s@%.*s\n", service_uri->len, 
service_uri->s,
user->len, user->s, domain->len, domain->s);
-   rootdoc = NULL;
if(xmldoc!=NULL)
xmlFreeDoc(xmldoc);
-   }
-   else
-   {
+   *rootdoc = NULL;
+   } else {
*rootdoc = xmldoc;
}
 
@@ -290,8 +288,6 @@ int rls_get_service_list(str *service_uri, str *user, str 
*domain,
 error:
if(result!=NULL)
rls_xcap_dbf.free_result(rls_xcap_db, result);
-   if(xmldoc!=NULL)
-   xmlFreeDoc(xmldoc);
if(xcapdoc!=NULL)
pkg_free(xcapdoc);
 


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:96c2294e: textops: fixed mixmatching types for exported kemi functions

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: 96c2294eb1a56b67c11d32e8fab55dd75a5fd26d
URL: 
https://github.com/kamailio/kamailio/commit/96c2294eb1a56b67c11d32e8fab55dd75a5fd26d

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:25:47+01:00

textops: fixed mixmatching types for exported kemi functions

(cherry picked from commit fb904abd968eb44e797ee7bc08c5705e69de2fc7)

---

Modified: src/modules/textops/textops.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/96c2294eb1a56b67c11d32e8fab55dd75a5fd26d.diff
Patch: 
https://github.com/kamailio/kamailio/commit/96c2294eb1a56b67c11d32e8fab55dd75a5fd26d.patch

---

diff --git a/src/modules/textops/textops.c b/src/modules/textops/textops.c
index 8bd7d883fe..97f1ecff12 100644
--- a/src/modules/textops/textops.c
+++ b/src/modules/textops/textops.c
@@ -4096,7 +4096,7 @@ static sr_kemi_t sr_kemi_textops_exports[] = {
},
{ str_init("textops"), str_init("append_body_part_cd"),
SR_KEMIP_INT, ki_append_multibody_cd,
-   { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_NONE,
+   { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_STR,
SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE }
},
{ str_init("textops"), str_init("remove_body_part"),


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:f243c5d2: textops: proper parameter types for kemi exports

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: f243c5d2da214bad59adcdc7d702d8da61c5d69f
URL: 
https://github.com/kamailio/kamailio/commit/f243c5d2da214bad59adcdc7d702d8da61c5d69f

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:25:58+01:00

textops: proper parameter types for kemi exports

(cherry picked from commit 23450310fe6095f612d328871380012f189ac1c3)

---

Modified: src/modules/tcpops/tcpops_mod.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/f243c5d2da214bad59adcdc7d702d8da61c5d69f.diff
Patch: 
https://github.com/kamailio/kamailio/commit/f243c5d2da214bad59adcdc7d702d8da61c5d69f.patch

---

diff --git a/src/modules/tcpops/tcpops_mod.c b/src/modules/tcpops/tcpops_mod.c
index 8e7fcdcacd..303dbf055a 100644
--- a/src/modules/tcpops/tcpops_mod.c
+++ b/src/modules/tcpops/tcpops_mod.c
@@ -330,7 +330,7 @@ static int ki_tcp_conid_state(sip_msg_t* msg, int i_conid)
LM_DBG("Connection id %d does not exist.\n", i_conid);
ret = -1;
goto done;
-   } 
+   }
/* Connection structure exists, now check what Kamailio thinks of it */
if (s_con->state == S_CONN_OK) {
/* All is fine, return happily */
@@ -383,7 +383,7 @@ static int w_tcp_conid_alive(sip_msg_t* msg, char* conid, 
char *p2)
int ret = w_tcp_conid_state(msg, conid, p2);
if (ret >= 1) {
return 1;   /* TRUE */
-   } 
+   }
/* We have some kind of problem */
return -1;
 }
@@ -393,7 +393,7 @@ static int ki_tcp_conid_alive(sip_msg_t* msg, int i_conid)
int ret = ki_tcp_conid_state(msg, i_conid);
if (ret >= 1) {
return 1;   /* TRUE */
-   } 
+   }
/* We have some kind of problem */
return -1;
 }
@@ -550,22 +550,22 @@ static sr_kemi_t sr_kemi_tcpops_exports[] = {
},
{ str_init("tcpops"), str_init("tcp_enable_closed_event"),
SR_KEMIP_INT, ki_tcpops_enable_closed_event,
-   { SR_KEMIP_INT, SR_KEMIP_NONE, SR_KEMIP_NONE,
+   { SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE,
SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE }
},
{ str_init("tcpops"), str_init("tcp_enable_closed_event_cid"),
SR_KEMIP_INT, ki_tcpops_enable_closed_event_cid,
-   { SR_KEMIP_INT, SR_KEMIP_INT, SR_KEMIP_NONE,
+   { SR_KEMIP_INT, SR_KEMIP_NONE, SR_KEMIP_NONE,
SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE }
},
{ str_init("tcpops"), str_init("tcp_conid_alive"),
SR_KEMIP_INT, ki_tcp_conid_alive,
-   { SR_KEMIP_INT, SR_KEMIP_INT, SR_KEMIP_NONE,
+   { SR_KEMIP_INT, SR_KEMIP_NONE, SR_KEMIP_NONE,
SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE }
},
{ str_init("tcpops"), str_init("tcp_conid_state"),
SR_KEMIP_INT, ki_tcp_conid_state,
-   { SR_KEMIP_INT, SR_KEMIP_INT, SR_KEMIP_NONE,
+   { SR_KEMIP_INT, SR_KEMIP_NONE, SR_KEMIP_NONE,
SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE }
},
 
@@ -577,4 +577,4 @@ int mod_register(char *path, int *dlflags, void *p1, void 
*p2)
 {
sr_kemi_modules_add(sr_kemi_tcpops_exports);
return 0;
-}
\ No newline at end of file
+}


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:bc8dd926: cnxcc: fixed kemi export for signature for ki_get_channel_count()

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: bc8dd9263715479f682062b1fb5a16e265fc4ae0
URL: 
https://github.com/kamailio/kamailio/commit/bc8dd9263715479f682062b1fb5a16e265fc4ae0

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:25:27+01:00

cnxcc: fixed kemi export for signature for ki_get_channel_count()

(cherry picked from commit c668854d05e548bf9d138778307891e914cc565e)

---

Modified: src/modules/cnxcc/cnxcc_mod.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/bc8dd9263715479f682062b1fb5a16e265fc4ae0.diff
Patch: 
https://github.com/kamailio/kamailio/commit/bc8dd9263715479f682062b1fb5a16e265fc4ae0.patch

---

diff --git a/src/modules/cnxcc/cnxcc_mod.c b/src/modules/cnxcc/cnxcc_mod.c
index 2cd8a87c6a..d429a50ee5 100644
--- a/src/modules/cnxcc/cnxcc_mod.c
+++ b/src/modules/cnxcc/cnxcc_mod.c
@@ -2133,7 +2133,7 @@ static sr_kemi_t sr_kemi_cnxcc_exports[] = {
},
{ str_init("cnxcc"), str_init("get_channel_count"),
SR_KEMIP_INT, ki_get_channel_count,
-   { SR_KEMIP_STR, SR_KEMIP_INT, SR_KEMIP_NONE,
+   { SR_KEMIP_STR, SR_KEMIP_STR, SR_KEMIP_NONE,
SR_KEMIP_NONE, SR_KEMIP_NONE, SR_KEMIP_NONE }
},
{ str_init("cnxcc"), str_init("terminate_all"),
@@ -2150,4 +2150,4 @@ int mod_register(char *path, int *dlflags, void *p1, void 
*p2)
 {
sr_kemi_modules_add(sr_kemi_cnxcc_exports);
return 0;
-}
\ No newline at end of file
+}


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:a0de4bce: msilo: set parameter names in prototypes

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: 5.1
Commit: a0de4bce7c09d361e2d912ebe8cfcb76a72dac5c
URL: 
https://github.com/kamailio/kamailio/commit/a0de4bce7c09d361e2d912ebe8cfcb76a72dac5c

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T11:25:38+01:00

msilo: set parameter names in prototypes

- used in genrating the index for kemi exports

(cherry picked from commit 4fbbc544c2c860c24ab3e8d4a4eb362f7f626ee8)

---

Modified: src/modules/msilo/msilo.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/a0de4bce7c09d361e2d912ebe8cfcb76a72dac5c.diff
Patch: 
https://github.com/kamailio/kamailio/commit/a0de4bce7c09d361e2d912ebe8cfcb76a72dac5c.patch

---

diff --git a/src/modules/msilo/msilo.c b/src/modules/msilo/msilo.c
index 2f261d69bd..19c1beb1fd 100644
--- a/src/modules/msilo/msilo.c
+++ b/src/modules/msilo/msilo.c
@@ -146,8 +146,8 @@ static int ms_skip_notification_flag = -1;
 static int mod_init(void);
 static int child_init(int);
 
-static int m_store(struct sip_msg*, str*);
-static int m_dump(struct sip_msg*, str*);
+static int m_store(sip_msg_t *masg, str *owner_s);
+static int m_dump(sip_msg_t *msg, str *owner_s);
 
 static int m_store_2(struct sip_msg*, char*, char*);
 static int m_dump_2(struct sip_msg*, char*, char*);
@@ -544,7 +544,7 @@ static int get_non_mandatory_headers(struct sip_msg *msg, 
char *buf, int buf_len
  * = "2" -- look for outgoing URI only at to header
  */
 
-static int m_store(struct sip_msg* msg, str *owner_s)
+static int m_store(sip_msg_t* msg, str *owner_s)
 {
str body, str_hdr, ctaddr;
struct to_body *pto, *pfrom;
@@ -935,7 +935,7 @@ static int ki_m_store(sip_msg_t* msg)
 /**
  * dump message
  */
-static int m_dump(struct sip_msg* msg, str* owner_s)
+static int m_dump(sip_msg_t* msg, str* owner_s)
 {
struct to_body *pto = NULL;
db_key_t db_keys[3];
@@ -1652,4 +1652,4 @@ int mod_register(char *path, int *dlflags, void *p1, void 
*p2)
 {
sr_kemi_modules_add(sr_kemi_msilo_exports);
return 0;
-}
\ No newline at end of file
+}


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.0:470f1885: http_async_client: cleanup avps/xavps after exec route on Http worker

2018-01-22 Thread Victor Seva 
Module: kamailio
Branch: 5.0
Commit: 470f1885a63518a42b921a93e2dedb604044dd41
URL: 
https://github.com/kamailio/kamailio/commit/470f1885a63518a42b921a93e2dedb604044dd41

Author: Victor Seva 
Committer: Victor Seva 
Date: 2018-01-22T11:15:15+01:00

http_async_client: cleanup avps/xavps after exec route on Http worker
when suspend_transaction is false

(cherry picked from commit ad46b115bedec01c52c5a9dcde0756db85ee61ec)

---

Modified: src/modules/http_async_client/async_http.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/470f1885a63518a42b921a93e2dedb604044dd41.diff
Patch: 
https://github.com/kamailio/kamailio/commit/470f1885a63518a42b921a93e2dedb604044dd41.patch

---

diff --git a/src/modules/http_async_client/async_http.c 
b/src/modules/http_async_client/async_http.c
index 8386985412..0f1242338b 100644
--- a/src/modules/http_async_client/async_http.c
+++ b/src/modules/http_async_client/async_http.c
@@ -45,6 +45,7 @@
 #include "../../core/dprint.h"
 #include "../../core/ut.h"
 #include "../../core/cfg/cfg_struct.h"
+#include "../../core/receive.h"
 #include "../../core/fmsg.h"
 #include "../../modules/tm/tm_load.h"
 
@@ -226,6 +227,7 @@ void async_http_cb(struct http_m_reply *reply, void *param)
fmsg = faked_msg_next();
if (run_top_route(act, fmsg, 0)<0)
LM_ERR("failure inside run_top_route\n");
+   ksr_msg_env_reset();
}
 
free_sip_msg(ah_reply);


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] [SR-Users] Fosdem dinner

2018-01-22 Thread Alexandr Dubovikov 
Hi,

count on me as well +2

Wbr,
Alexandr

On 17 January 2018 at 00:34, Julien Chavanton  wrote:

> Hi Daniel, I will be happy to join !
>
> On Mon, Jan 15, 2018 at 12:28 AM, Daniel-Constantin Mierla <
> mico...@gmail.com> wrote:
>
>> Hello,
>>
>> in the past 10 years or so, many of us met at Fosdem conference in
>> Brussels (the edition this year happens during February 3-4) and had a
>> dinner on Saturday evening. I know already couple of people from
>> community that will go there, so the question is if there is any
>> interest to organize again such event.
>>
>> If yes, it will be again on Saturday, on the evening of the 3rd of
>> February.
>>
>> At the past editions, it turned into an all-RTC dinner, but we can
>> consider to have it alone again (eventually with few other invited
>> friends), if there are too many or people want a smaller meeting.
>>
>> Reply this week if you want to join, so we can estimate how many seats
>> we need and have enough time to find a place that can accommodate us.
>>
>> More about Fosdem can be found at:
>>
>>   - https://fosdem.org
>>
>> There is a devroom on Sunday with topics covering real time
>> communications. I will have a presentation there.
>>
>> Cheers,
>> Daniel
>>
>> --
>> Daniel-Constantin Mierla
>> www.twitter.com/miconda -- www.linkedin.com/in/miconda
>> Kamailio Advanced Training - March 5-7, 2018, Berlin - www.asipto.com
>> Kamailio World Conference - May 14-16, 2018 - www.kamailioworld.com
>>
>>
>> ___
>> Kamailio (SER) - Users Mailing List
>> sr-us...@lists.kamailio.org
>> https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users
>>
>
>
> ___
> Kamailio (SER) - Development Mailing List
> sr-dev@lists.kamailio.org
> https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev
>
>
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] [kamailio/kamailio] modules/sipcapture fixed LM_ERR to LM_DBG (#1406)

2018-01-22 Thread Alexandr Dubovikov 
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/1406#issuecomment-359376506___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] [kamailio/kamailio] modules/sipcapture fixed LM_ERR to LM_DBG (#1406)

2018-01-22 Thread Daniel-Constantin Mierla 
you can merge and backport if needed.

-- 
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/1406#issuecomment-359376321___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] [kamailio/kamailio] modules/sipcapture fixed LM_ERR to LM_DBG (#1406)

2018-01-22 Thread Alexandr Dubovikov 




 Pre-Submission Checklist



- [x] Commit message has the format required by CONTRIBUTING guide
- [x] Commits are split per component (core, individual modules, libs, utils, 
...)
- [x] Each component has a single commit (if not, squash them into one commit)
- [x] No commits to README files for modules (changes must be done to docbook 
files
in `doc/` subfolder, the README file is autogenerated)

 Type Of Change
- [x] Small bug fix (non-breaking change which fixes an issue)

 Checklist:

- [x] PR should be backported to stable branches

 Description
fixing LM_ERR logging to LM_DBG in the pv_parse_hep_name() function.
You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/1406

-- Commit Summary --

  * modules/sipcapture fixed LM_ERR to LM_DBG

-- File Changes --

M src/modules/sipcapture/sipcapture.c (2)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/1406.patch
https://github.com/kamailio/kamailio/pull/1406.diff

-- 
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/1406
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:7dae4ec9: siptrace: fixed unused variable warnings

2018-01-22 Thread Daniel-Constantin Mierla 
Module: kamailio
Branch: master
Commit: 7dae4ec9e4be87d5a3406fe2ddeb98328beb1125
URL: 
https://github.com/kamailio/kamailio/commit/7dae4ec9e4be87d5a3406fe2ddeb98328beb1125

Author: Daniel-Constantin Mierla 
Committer: Daniel-Constantin Mierla 
Date: 2018-01-22T10:53:05+01:00

siptrace: fixed unused variable warnings

---

Modified: src/modules/siptrace/siptrace.c
Modified: src/modules/siptrace/siptrace_send.h

---

Diff:  
https://github.com/kamailio/kamailio/commit/7dae4ec9e4be87d5a3406fe2ddeb98328beb1125.diff
Patch: 
https://github.com/kamailio/kamailio/commit/7dae4ec9e4be87d5a3406fe2ddeb98328beb1125.patch

---

diff --git a/src/modules/siptrace/siptrace.c b/src/modules/siptrace/siptrace.c
index 924c4f52e7..851bf6e0f8 100644
--- a/src/modules/siptrace/siptrace.c
+++ b/src/modules/siptrace/siptrace.c
@@ -1016,7 +1016,6 @@ static void trace_onreq_out(struct cell *t, int type, 
struct tmcb_params *ps)
siptrace_data_t sto;
sip_msg_t *msg;
ip_addr_t to_ip;
-   int len;
dest_info_t *dst;
 
if(t == NULL || ps == NULL) {
@@ -1334,7 +1333,6 @@ static void trace_sl_onreply_out(sl_cbp_t *slcbp)
siptrace_data_t sto;
sip_msg_t *msg;
ip_addr_t to_ip;
-   int len;
char statusbuf[5];
 
if(slcbp == NULL || slcbp->req == NULL) {
diff --git a/src/modules/siptrace/siptrace_send.h 
b/src/modules/siptrace/siptrace_send.h
index 30c33e53b5..867d3995b0 100644
--- a/src/modules/siptrace/siptrace_send.h
+++ b/src/modules/siptrace/siptrace_send.h
@@ -68,8 +68,9 @@ int trace_send_duplicate(char *buf, int len, struct dest_info 
*dst2);
 #define siptrace_copy_proto(vproto, vbuf) do { \
int __olen; \
siptrace_copy_proto_olen(vproto, vbuf, __olen); \
+   (void)__olen; \
} while(0)
 
 char* siptrace_proto_name(int vproto);
 
-#endif
\ No newline at end of file
+#endif


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:54f3a3d9: pkg/kamailio/deb: set version to 5.1.1

2018-01-22 Thread Victor Seva 
Module: kamailio
Branch: 5.1
Commit: 54f3a3d98682e159988c57fa3e3cf28f6b41a121
URL: 
https://github.com/kamailio/kamailio/commit/54f3a3d98682e159988c57fa3e3cf28f6b41a121

Author: Victor Seva 
Committer: Victor Seva 
Date: 2018-01-22T10:37:52+01:00

pkg/kamailio/deb: set version to 5.1.1

---

Modified: pkg/kamailio/deb/buster/changelog
Modified: pkg/kamailio/deb/debian/changelog
Modified: pkg/kamailio/deb/jessie/changelog
Modified: pkg/kamailio/deb/precise/changelog
Modified: pkg/kamailio/deb/sid/changelog
Modified: pkg/kamailio/deb/stretch/changelog
Modified: pkg/kamailio/deb/trusty/changelog
Modified: pkg/kamailio/deb/wheezy/changelog
Modified: pkg/kamailio/deb/xenial/changelog

---

Diff:  
https://github.com/kamailio/kamailio/commit/54f3a3d98682e159988c57fa3e3cf28f6b41a121.diff
Patch: 
https://github.com/kamailio/kamailio/commit/54f3a3d98682e159988c57fa3e3cf28f6b41a121.patch

---

diff --git a/pkg/kamailio/deb/buster/changelog 
b/pkg/kamailio/deb/buster/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/buster/changelog
+++ b/pkg/kamailio/deb/buster/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0
diff --git a/pkg/kamailio/deb/debian/changelog 
b/pkg/kamailio/deb/debian/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/debian/changelog
+++ b/pkg/kamailio/deb/debian/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0
diff --git a/pkg/kamailio/deb/jessie/changelog 
b/pkg/kamailio/deb/jessie/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/jessie/changelog
+++ b/pkg/kamailio/deb/jessie/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0
diff --git a/pkg/kamailio/deb/precise/changelog 
b/pkg/kamailio/deb/precise/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/precise/changelog
+++ b/pkg/kamailio/deb/precise/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0
diff --git a/pkg/kamailio/deb/sid/changelog b/pkg/kamailio/deb/sid/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/sid/changelog
+++ b/pkg/kamailio/deb/sid/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0
diff --git a/pkg/kamailio/deb/stretch/changelog 
b/pkg/kamailio/deb/stretch/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/stretch/changelog
+++ b/pkg/kamailio/deb/stretch/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0
diff --git a/pkg/kamailio/deb/trusty/changelog 
b/pkg/kamailio/deb/trusty/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/trusty/changelog
+++ b/pkg/kamailio/deb/trusty/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0
diff --git a/pkg/kamailio/deb/wheezy/changelog 
b/pkg/kamailio/deb/wheezy/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/wheezy/changelog
+++ b/pkg/kamailio/deb/wheezy/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0
diff --git a/pkg/kamailio/deb/xenial/changelog 
b/pkg/kamailio/deb/xenial/changelog
index 18e2a36357..798ee4ecce 100644
--- a/pkg/kamailio/deb/xenial/changelog
+++ b/pkg/kamailio/deb/xenial/changelog
@@ -1,3 +1,9 @@
+kamailio (5.1.1) unstable; urgency=medium
+
+  * version set to 5.1.1
+
+ -- Victor Seva   Mon, 22 Jan 2018 10:37:19 +0100
+
 kamailio (5.1.0) unstable; urgency=medium
 
   * version set to 5.1.0


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org

Re: [sr-dev] [kamailio/kamailio] http_async_client: coredump on 5.0.5 (#1391)

2018-01-22 Thread Victor Seva 
Closed #1391.

-- 
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/1391#event-1434728143___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:5.1:a88346c6: http_async_client: cleanup avps/xavps after exec route on Http worker

2018-01-22 Thread Federico Cabiddu 
Module: kamailio
Branch: 5.1
Commit: a88346c6106653a9ffa60ad41e143f62dd8af1ea
URL: 
https://github.com/kamailio/kamailio/commit/a88346c6106653a9ffa60ad41e143f62dd8af1ea

Author: Victor Seva 
Committer: Federico Cabiddu 
Date: 2018-01-22T10:11:29+01:00

http_async_client: cleanup avps/xavps after exec route on Http worker
when suspend_transaction is false

(cherry picked from commit ad46b115bedec01c52c5a9dcde0756db85ee61ec)

---

Modified: src/modules/http_async_client/async_http.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/a88346c6106653a9ffa60ad41e143f62dd8af1ea.diff
Patch: 
https://github.com/kamailio/kamailio/commit/a88346c6106653a9ffa60ad41e143f62dd8af1ea.patch

---

diff --git a/src/modules/http_async_client/async_http.c 
b/src/modules/http_async_client/async_http.c
index 59106a3bae..87a62ea8f6 100644
--- a/src/modules/http_async_client/async_http.c
+++ b/src/modules/http_async_client/async_http.c
@@ -45,6 +45,7 @@
 #include "../../core/dprint.h"
 #include "../../core/ut.h"
 #include "../../core/cfg/cfg_struct.h"
+#include "../../core/receive.h"
 #include "../../core/fmsg.h"
 #include "../../core/kemi.h"
 #include "../../modules/tm/tm_load.h"
@@ -263,6 +264,7 @@ void async_http_cb(struct http_m_reply *reply, void *param)
LM_ERR("error running event route kemi 
callback\n");
}
}
+   ksr_msg_env_reset();
}
 
 done:


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] [kamailio/kamailio] http_async_client: cleanup avps/xavps after exec route on Http worker (#1405)

2018-01-22 Thread Federico Cabiddu 
Merged #1405.

-- 
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/1405#event-1434689440___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] git:master:f6066a02: Merge pull request #1405 from kamailio/vseva/http_async_client_fix

2018-01-22 Thread GitHub 
Module: kamailio
Branch: master
Commit: f6066a022db81ae4d5f6c5cb5500fdcfdb15e33e
URL: 
https://github.com/kamailio/kamailio/commit/f6066a022db81ae4d5f6c5cb5500fdcfdb15e33e

Author: Federico Cabiddu 
Committer: GitHub 
Date: 2018-01-22T10:10:51+01:00

Merge pull request #1405 from kamailio/vseva/http_async_client_fix

http_async_client: cleanup avps/xavps after exec route on Http worker

---

Modified: src/modules/http_async_client/async_http.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/f6066a022db81ae4d5f6c5cb5500fdcfdb15e33e.diff
Patch: 
https://github.com/kamailio/kamailio/commit/f6066a022db81ae4d5f6c5cb5500fdcfdb15e33e.patch

---

diff --git a/src/modules/http_async_client/async_http.c 
b/src/modules/http_async_client/async_http.c
index 59106a3bae..87a62ea8f6 100644
--- a/src/modules/http_async_client/async_http.c
+++ b/src/modules/http_async_client/async_http.c
@@ -45,6 +45,7 @@
 #include "../../core/dprint.h"
 #include "../../core/ut.h"
 #include "../../core/cfg/cfg_struct.h"
+#include "../../core/receive.h"
 #include "../../core/fmsg.h"
 #include "../../core/kemi.h"
 #include "../../modules/tm/tm_load.h"
@@ -263,6 +264,7 @@ void async_http_cb(struct http_m_reply *reply, void *param)
LM_ERR("error running event route kemi 
callback\n");
}
}
+   ksr_msg_env_reset();
}
 
 done:


___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] [kamailio/kamailio] http_async_client: cleanup avps/xavps after exec route on Http worker (#1405)

2018-01-22 Thread Federico Cabiddu 
grumvalski approved this 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/1405#pullrequestreview-90403545___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Re: [sr-dev] [kamailio/kamailio] http_async_client: cleanup avps/xavps after exec route on Http worker (#1405)

2018-01-22 Thread Federico Cabiddu 
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/1405#issuecomment-359360870___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

[sr-dev] [kamailio/kamailio] http_async_client: cleanup avps/xavps after exec route on Http worker (#1405)

2018-01-22 Thread Victor Seva 
when suspend_transaction is false

As suggested at #1391 to avoid shared memory leak
You can view, comment on, or merge this pull request online at:

  https://github.com/kamailio/kamailio/pull/1405

-- Commit Summary --

  * http_async_client: cleanup avps/xavps after exec route on Http worker

-- File Changes --

M src/modules/http_async_client/async_http.c (2)

-- Patch Links --

https://github.com/kamailio/kamailio/pull/1405.patch
https://github.com/kamailio/kamailio/pull/1405.diff

-- 
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/1405
___
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev