[Kamailio-Devel] [ openser-Patches-2505748 ] module 'Regex' tarball

SourceForge.net noreply at sourceforge.net
Wed Jan 14 11:46:42 CET 2009


Patches item #2505748, was opened at 2009-01-14 02:23
Message generated for change (Comment added) made by miconda
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=2505748&group_id=139143

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: ver devel
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: Iñaki Baz (ibc_sf)
>Assigned to: Daniel-Constantin Mierla (miconda)
Summary: module 'Regex' tarball

Initial Comment:
Hi, I upload a tarball containing a new module called 'Regex':

This module offers matching operations against regular expressions using the powerful PCRE library.  A text file containing regular expressions categorized in groups is compiled when the module is loaded, storing the compiled PCRE objects in an array. A function to match a string or pseudo-variable against any of these groups is provided. The text file can be modified and reloaded at any time via a MI command. The module also offers a function to perform a PCRE matching operation against a regular expression provided as function parameter.

I hope it is useful.

----------------------------------------------------------------------

>Comment By: Daniel-Constantin Mierla (miconda)
Date: 2009-01-14 12:46

Message:
Here are couple remarks:
- it is recommended use pkg_malloc()/pkg_free() instead of standard
malloc/free for temporary operation.
- in regex_mod.c line 616, it might be needed to check if
(*pcres_addr)[num_pcre] is NULL, maybe a reload removed a group or
parameter is not a valid group. Just something I could spot.

Once you double-check the second, you can go ahead and check in the code.
Your account has write access. Let me know if you have troubles.


----------------------------------------------------------------------

Comment By: Daniel-Constantin Mierla (miconda)
Date: 2009-01-14 11:33

Message:
Thanks. I will review it and then grant you SVN access to commit.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=743022&aid=2505748&group_id=139143



More information about the Devel mailing list