[OpenSIPS-Devel] SF.net SVN: opensips:[8394] trunk/parser/parse_to.c

2011-09-27 Thread Vlad Paiu
Revision: 8394
  http://opensips.svn.sourceforge.net/opensips/?rev=8394&view=rev
Author:   vladut-paiu
Date: 2011-09-27 09:32:49 + (Tue, 27 Sep 2011)
Log Message:
---
fixed to param freeing

Modified Paths:
--
trunk/parser/parse_to.c

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8395] branches/1.7/parser/parse_to.c

2011-09-27 Thread Vlad Paiu
Revision: 8395
  http://opensips.svn.sourceforge.net/opensips/?rev=8395&view=rev
Author:   vladut-paiu
Date: 2011-09-27 09:36:02 + (Tue, 27 Sep 2011)
Log Message:
---
backport from trunk (#8394)

fixed to param freeing

Modified Paths:
--
branches/1.7/parser/parse_to.c

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] Planning for OpenSIPS 1.8.0 Release

2011-09-27 Thread Vlad Paiu

Hello,

While the next OpenSIPS major release ( 1.8.0 ) is expected for early 
2012, we have already started working on the new features that will be 
integrated into 1.8.0, and decided to share all the future additions 
with the community.


By going to [1], you can find all the new things that have already been 
developed and that currently reside in the trunk branch [2] , as well as 
the features that are planned to be in 1.8.0 and that will be developed 
and integrated [3].


Opinions and suggestions are more than welcome.

[1] http://www.opensips.org/Main/Ver180
[2] http://www.opensips.org/Main/Ver180#toc2
[3] http://www.opensips.org/Main/Ver180#toc9

Regards,

--
Vlad Paiu
OpenSIPS Developer


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] Proposal for change in permissions module

2011-09-27 Thread Saúl Ibarra Corretgé
Hi,

While working with the permissions module I ran across the case where I'd do a 
check_address(...) but the group id wouldn't exist. AFAIS in the code, there 
are only 2 possible return values: 1 if a match was found, or -1 if it wasn't 
found.

I'd like to add a new return code, -2 which would be returned if the group id 
being checked doesn't exist. This way the script writer could decide if the 
fact that the group doesn't exist is the same as the ip/mask/... not matching 
in an *existing* group.

If there is no objection I'll make a patch and submit it to the tracker.


Thanks and regards,

--
Saúl Ibarra Corretgé
AG Projects




___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


Re: [OpenSIPS-Devel] [OpenSIPS-Users] Planning for OpenSIPS 1.8.0 Release

2011-09-27 Thread Saúl Ibarra Corretgé
Hi Vlad,

On Sep 27, 2011, at 12:13 PM, Vlad Paiu wrote:

> Hello,
> 
> While the next OpenSIPS major release ( 1.8.0 ) is expected for early 2012, 
> we have already started working on the new features that will be integrated 
> into 1.8.0, and decided to share all the future additions with the community.
> 
> By going to [1], you can find all the new things that have already been 
> developed and that currently reside in the trunk branch [2] , as well as the 
> features that are planned to be in 1.8.0 and that will be developed and 
> integrated [3].
> 
> Opinions and suggestions are more than welcome.
> 
> [1] http://www.opensips.org/Main/Ver180
> [2] http://www.opensips.org/Main/Ver180#toc2
> [3] http://www.opensips.org/Main/Ver180#toc9
> 

Great to hear! Looking forward to checking out GRUU :-)

I see you listed CallControl call-limit as a new 1.8.0 feature. I didn't know 
there would be a 1.8.0 release, so the code is already in the 1.7 branch, sorry 
for that.

There hasn't been a minor release for the 1.7 branch with that code yet, but 
since it's already there, I'd go for keeping it in the 1.7 branch, if that's 
not a problem.


Thanks and regards,

--
Saúl Ibarra Corretgé
AG Projects




___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8396] trunk

2011-09-27 Thread Bogdan-Andrei Iancu
Revision: 8396
  http://opensips.svn.sourceforge.net/opensips/?rev=8396&view=rev
Author:   bogdan_iancu
Date: 2011-09-27 11:31:43 + (Tue, 27 Sep 2011)
Log Message:
---
- new function in internal AVP API - add_avp_last() to add an AVP to the end of 
the list of existing AVPs
  (to be used by new Dynamic Routing version)

Modified Paths:
--
trunk/usr_avp.c
trunk/usr_avp.h

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


Re: [OpenSIPS-Devel] Proposal for change in permissions module

2011-09-27 Thread Bogdan-Andrei Iancu

Hi Saúl,

if in the check_address() you use group "0", it will search in all 
groups - group "0" is a kind of wild card. Will this help you ?


Regards,
Bogdan

On 09/27/2011 01:49 PM, Saúl Ibarra Corretgé wrote:

Hi,

While working with the permissions module I ran across the case where I'd do a 
check_address(...) but the group id wouldn't exist. AFAIS in the code, there 
are only 2 possible return values: 1 if a match was found, or -1 if it wasn't 
found.

I'd like to add a new return code, -2 which would be returned if the group id 
being checked doesn't exist. This way the script writer could decide if the 
fact that the group doesn't exist is the same as the ip/mask/... not matching 
in an *existing* group.

If there is no objection I'll make a patch and submit it to the tracker.


Thanks and regards,

--
Saúl Ibarra Corretgé
AG Projects




___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel




--
Bogdan-Andrei Iancu
OpenSIPS eBootcamp - 19th of September 2011
OpenSIPS solutions and "know-how"


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


Re: [OpenSIPS-Devel] Proposal for change in permissions module

2011-09-27 Thread Saúl Ibarra Corretgé
Hi Bogdan,

On Sep 27, 2011, at 1:36 PM, Bogdan-Andrei Iancu wrote:

> Hi Saúl,
> 
> if in the check_address() you use group "0", it will search in all groups - 
> group "0" is a kind of wild card. Will this help you ?
> 

Nope. Let me explain the case a bit better: lets say I have some sort of ACL 
mechanism, so that only the listed IP addresses are allowed to send traffic. In 
order to allow traffic from everywhere I'd have to either not do the check or 
add 0.0.0.0/0 as a value so that it matches everything.

If I'd have a -2 return code I could do something like this:

check_address(...)
switch ($retcode) {
case -2:
   # There was no ACL, so I choose to allow it anyway
  ...
  break;
case -1:
  # No match
  sl_send_reply("403", "Get out!");
  exit;
default:
# All is good

}

Does it make more sense now?


Regards,

--
Saúl Ibarra Corretgé
AG Projects




___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8397] trunk/modules/rtpproxy/rtpproxy.c

2011-09-27 Thread Razvan Crainea
Revision: 8397
  http://opensips.svn.sourceforge.net/opensips/?rev=8397&view=rev
Author:   razvancrainea
Date: 2011-09-27 13:03:00 + (Tue, 27 Sep 2011)
Log Message:
---
Fixed bug in the rtpproxy module - when a disabled rtpproxy
is chosen to handle the request, it was entering in a loop that never stopped

Modified Paths:
--
trunk/modules/rtpproxy/rtpproxy.c

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8398] branches/1.7/modules/rtpproxy/rtpproxy.c

2011-09-27 Thread Razvan Crainea
Revision: 8398
  http://opensips.svn.sourceforge.net/opensips/?rev=8398&view=rev
Author:   razvancrainea
Date: 2011-09-27 13:03:35 + (Tue, 27 Sep 2011)
Log Message:
---
Backported from trunk (8397):
Fixed bug in the rtpproxy module - when a disabled rtpproxy
is chosen to handle the request, it was entering in a loop that never stopped

Modified Paths:
--
branches/1.7/modules/rtpproxy/rtpproxy.c

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] [ opensips-Patches-3414466 ] New retcode for persmissions module

2011-09-27 Thread SourceForge . net
Patches item #3414466, was opened at 2011-09-27 16:03
Message generated for change (Tracker Item Submitted) made by saghul
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3414466&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: saghul (saghul)
Assigned to: Nobody/Anonymous (nobody)
Summary: New retcode for persmissions module

Initial Comment:
As explained in 
http://lists.opensips.org/pipermail/devel/2011-September/008956.html

The attached patch will make check_address and check_source_address return -2 
(instead of -1) if the group being checked doesn't exist.

--

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3414466&group_id=232389

___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] [ opensips-Patches-3414466 ] New retcode for persmissions module

2011-09-27 Thread SourceForge . net
Patches item #3414466, was opened at 2011-09-27 16:03
Message generated for change (Settings changed) made by saghul
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3414466&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
>Category: modules
>Group: trunk
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: saghul (saghul)
Assigned to: Nobody/Anonymous (nobody)
Summary: New retcode for persmissions module

Initial Comment:
As explained in 
http://lists.opensips.org/pipermail/devel/2011-September/008956.html

The attached patch will make check_address and check_source_address return -2 
(instead of -1) if the group being checked doesn't exist.

--

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3414466&group_id=232389

___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] [ opensips-Patches-2957313 ] [MediaProxy] Add support for ICE

2011-09-27 Thread SourceForge . net
Patches item #2957313, was opened at 2010-02-23 18:12
Message generated for change (Comment added) made by saghul
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=2957313&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: None
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: saghul (saghul)
Assigned to: Dan (dan_pascu)
Summary: [MediaProxy] Add support for ICE

Initial Comment:
Hi,

The attached patch adds support for using ICE 
(http://tools.ietf.org/html/draft-ietf-mmusic-ice-19) in scenarios involving 
OpenSIPS and MediaProxy.

Currently NAT is usually fixed by mangling the SDP in OpenSIPS and directing 
RTP to a media relay server. This SDP mangling makes ICE not to work, because 
ICE needs to have a candidate for the IP present in the c line of the SDP, and 
as the proxy changed it, ICE negotiation breaks.

With the attached patch, OpenSIPS will add the necessary candidates to the SDP 
so that the ICE negotiation doesn't break and we can have p2p audio on LAN 
environments for example. :)


Please consder it for inclusion in some later relase,

Kind regards,

PS: The design on how the issue has been solved can be seen here: 
http://mediaproxy.ag-projects.com/wiki/ICE
PS2: This patch needs this other patch to be applied previously: 
https://sourceforge.net/tracker/?func=detail&aid=2949743&group_id=232389&atid=1086412


--
Saúl Ibarra Corretgé
AG Projects

--

>Comment By: saghul (saghul)
Date: 2011-09-27 16:06

Message:
Closing this, it was applied long ago...

--

Comment By: saghul (saghul)
Date: 2010-03-09 16:48

Message:
Uploaded new version to match latest changes in the code. Now the RTCP line
fix patch is not needed.

--

Comment By: saghul (saghul)
Date: 2010-02-24 18:47

Message:
Just uploaded a patch for adding the documentation (README and XML) for the
new parameters.

--

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086412&aid=2957313&group_id=232389

___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


Re: [OpenSIPS-Devel] Proposal for change in permissions module

2011-09-27 Thread Saúl Ibarra Corretgé
Bogdan,

I made and attached the patch in the tracker: 
http://sourceforge.net/tracker/?func=detail&atid=1086412&aid=3414466&group_id=232389


Regards,

--
Saúl Ibarra Corretgé
AG Projects




___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8399] trunk/modules/drouting

2011-09-27 Thread Bogdan-Andrei Iancu
Revision: 8399
  http://opensips.svn.sourceforge.net/opensips/?rev=8399&view=rev
Author:   bogdan_iancu
Date: 2011-09-27 14:21:58 + (Tue, 27 Sep 2011)
Log Message:
---
- lists of gws replaced with "carriers". Carriers can:
- have attributes (opaque string)
- have falgs (if sorting of GWs is order or weight based, if enabled or 
not)
- have a list of GWs (with weights)
- dr rules get a mixed list of GWs and CARRIERs to use (each with weight)
- GW and CARRIER IDs can be now alphanumerical strings

Modified Paths:
--
trunk/modules/drouting/dr_load.c
trunk/modules/drouting/drouting.c
trunk/modules/drouting/parse.h
trunk/modules/drouting/prefix_tree.c
trunk/modules/drouting/prefix_tree.h
trunk/modules/drouting/routing.c
trunk/modules/drouting/routing.h

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8400] trunk/db/schema

2011-09-27 Thread Bogdan-Andrei Iancu
Revision: 8400
  http://opensips.svn.sourceforge.net/opensips/?rev=8400&view=rev
Author:   bogdan_iancu
Date: 2011-09-27 14:24:16 + (Tue, 27 Sep 2011)
Log Message:
---
- changes in DB schema to reflect DR changes

Modified Paths:
--
trunk/db/schema/dr_gateways.xml
trunk/db/schema/opensips-drouting.xml

Added Paths:
---
trunk/db/schema/dr_carriers.xml

Removed Paths:
-
trunk/db/schema/dr_gw_lists.xml

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8401] trunk/scripts

2011-09-27 Thread Bogdan-Andrei Iancu
Revision: 8401
  http://opensips.svn.sourceforge.net/opensips/?rev=8401&view=rev
Author:   bogdan_iancu
Date: 2011-09-27 14:31:15 + (Tue, 27 Sep 2011)
Log Message:
---
- updated DB schema and tables for DR

Modified Paths:
--
trunk/scripts/db_berkeley/opensips/dr_gateways
trunk/scripts/db_berkeley/opensips/version
trunk/scripts/dbtext/opensips/dr_gateways
trunk/scripts/dbtext/opensips/version
trunk/scripts/mysql/drouting-create.sql
trunk/scripts/opensipsdbctl.mysql
trunk/scripts/oracle/drouting-create.sql
trunk/scripts/osipsconsole
trunk/scripts/postgres/drouting-create.sql

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8402] trunk/modules/drouting/drouting.c

2011-09-27 Thread Bogdan-Andrei Iancu
Revision: 8402
  http://opensips.svn.sourceforge.net/opensips/?rev=8402&view=rev
Author:   bogdan_iancu
Date: 2011-09-27 15:43:17 + (Tue, 27 Sep 2011)
Log Message:
---
- GW ID AVP carries now a string values, not a interger one

Modified Paths:
--
trunk/modules/drouting/drouting.c

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] SF.net SVN: opensips:[8403] trunk/modules/drouting

2011-09-27 Thread Bogdan-Andrei Iancu
Revision: 8403
  http://opensips.svn.sourceforge.net/opensips/?rev=8403&view=rev
Author:   bogdan_iancu
Date: 2011-09-27 16:27:23 + (Tue, 27 Sep 2011)
Log Message:
---
- return in an AVP the matched prefix, so after a rule matcthing (in script) 
you can know the exact prefix that matched.
  adds new module param "rule_attrs_avp" 

Modified Paths:
--
trunk/modules/drouting/README
trunk/modules/drouting/doc/drouting_admin.xml
trunk/modules/drouting/drouting.c
trunk/modules/drouting/prefix_tree.c
trunk/modules/drouting/prefix_tree.h

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel


[OpenSIPS-Devel] [ opensips-Bugs-3413995 ] PUBLISH handling causes segfault through update_presentity

2011-09-27 Thread SourceForge . net
Bugs item #3413995, was opened at 2011-09-26 13:47
Message generated for change (Comment added) made by wdoekes
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086410&aid=3413995&group_id=232389

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: modules
Group: 1.7.x
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Walter Doekes (wdoekes)
Assigned to: Nobody/Anonymous (nobody)
Summary: PUBLISH handling causes segfault through update_presentity

Initial Comment:
Hi,

update_presentity() [modules/presence/presentity.c] calls
publ_notify(...&body...) [modules/presence/notify.c where body is allocated on 
the stack
get_p_notify_body(...publ_body...) [modules/presence/notify.c] attempts to free 
this

This results in a segfault obviously.

Ugly fix: check for publ_body while doing the Free loop.

Regards,
Walter Doekes
OSSO B.V.

P.S. This bug is not present in 1.6, but is present in trunk and 1.7.

--

>Comment By: Walter Doekes (wdoekes)
Date: 2011-09-28 08:37

Message:
I added a new patch, because after fixing the crash, a memory leak became
apparent. (publ_body overwrote an allocated var before freeing it)

The patch also cleans up a bit of the code in the neighbourhood:
- no redundant memsets
- some code formatting

--

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1086410&aid=3413995&group_id=232389

___
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel