Module: sip-router Branch: master Commit: af69a3075ac4a7a46d72b0d60e6bc1740a56b7a7 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=af69a307...
Author: Juha Heinanen jh@tutpro.com Committer: Juha Heinanen jh@tutpro.com Date: Fri Sep 5 17:11:47 2014 +0300
modules/mtree: fixed return value of mt_match when mode=2 - initialize pv_values avp by mt_match when mode=2 - documented mt_match return value
---
modules/mtree/README | 26 ++++++++++++++------------ modules/mtree/doc/mtree_admin.xml | 7 +++++-- modules/mtree/mtree.c | 12 +++++++++--- 3 files changed, 28 insertions(+), 17 deletions(-)
diff --git a/modules/mtree/README b/modules/mtree/README index b0a6543..28dd6d8 100644 --- a/modules/mtree/README +++ b/modules/mtree/README @@ -20,9 +20,9 @@ Juha Heinanen
- Copyright � 2010 Daniel-Constantin Mierla (asipto.com) + Copyright (c) 2010 Daniel-Constantin Mierla (asipto.com)
- Copyright � 2011 Juha Heinanen + Copyright (c) 2011 Juha Heinanen __________________________________________________________________
Table of Contents @@ -275,8 +275,8 @@ modparam("mtree", "pv_value", "$var(mtval)")
3.10. pv_values (string)
- The PV spec where to store the matched values when mtree is of type 0 - or 2 and mode of mt_match() call has value 2. It can be any AVP. + The AVP where to store the matched values when mtree is of type 0 or 2 + and mode of mt_match() call has value 2.
Default value is "$avp(s:tvalues)".
@@ -323,7 +323,7 @@ modparam("mtree", "mt_allow_duplicates", 1)
4.1. mt_match(mtree, pv, mode)
-4.1. mt_match(mtree, pv, mode) +4.1. mt_match(mtree, pv, mode)
Match 'pv' value against 'mtree'. If 'mtree' type is 0 or 2 and value of 'mode' is NOT 2, sets a value of the longest matching prefix to @@ -333,6 +333,8 @@ modparam("mtree", "mt_allow_duplicates", 1) matching prefix is in avp index 0. Parameter 'mode' can be an integer constant or a pseudo variable with integer value.
+ Returns 1 if match succeeded and -1 otherwise. + Example 1.14. mt_match usage ... mt_match("mytree", "$rU", "0"); @@ -345,7 +347,7 @@ mt_match("mytree", "$rU", "0"); 5.3. mt_summary 5.4. mt_match
-5.1. mt_list +5.1. mt_list
List content of a tree.
@@ -359,7 +361,7 @@ mt_match("mytree", "$rU", "0"); _mtname_ _empty_line_
-5.2. mt_reload +5.2. mt_reload
Reload mtree from database.
@@ -374,7 +376,7 @@ mt_match("mytree", "$rU", "0"); _mtname_ _empty_line_
-5.3. mt_summary +5.3. mt_summary
List usage summary for all trees.
@@ -386,7 +388,7 @@ mt_match("mytree", "$rU", "0"); :mt_summary:_reply_fifo_file_ _empty_line_
-5.4. mt_match +5.4. mt_match
Match prefix value against mtree.
@@ -410,7 +412,7 @@ mt_match("mytree", "$rU", "0"); 6.2. mtree.reload 6.3. mtree.match
-6.1. mtree.summary +6.1. mtree.summary
List usage summary for all trees or for the tree whose name is given as parameter. @@ -418,7 +420,7 @@ mt_match("mytree", "$rU", "0"); Parameters: * _mtree_ - (optional) the name of the tree.
-6.2. mtree.reload +6.2. mtree.reload
Reload mtree from database to memory.
@@ -426,7 +428,7 @@ mt_match("mytree", "$rU", "0"); * _mtree_ - name of mtree or empty string meaning all mtrees
-6.3. mtree.match +6.3. mtree.match
Match prefix value against mtree
diff --git a/modules/mtree/doc/mtree_admin.xml b/modules/mtree/doc/mtree_admin.xml index cf91bbc..17c2c8f 100644 --- a/modules/mtree/doc/mtree_admin.xml +++ b/modules/mtree/doc/mtree_admin.xml @@ -248,8 +248,8 @@ modparam("mtree", "pv_value", "$var(mtval)") <section> <title><varname>pv_values</varname> (string)</title> <para> - The PV spec where to store the matched values when mtree is of type - 0 or 2 and mode of mt_match() call has value 2. It can be any AVP. + The AVP where to store the matched values when mtree is of type + 0 or 2 and mode of mt_match() call has value 2. </para> <para> <emphasis> @@ -344,6 +344,9 @@ modparam("mtree", "mt_allow_duplicates", 1) matching prefix is in avp index 0. Parameter 'mode' can be an integer constant or a pseudo variable with integer value. </para> + <para> + Returns 1 if match succeeded and -1 otherwise. + </para> <example> <title><function>mt_match</function> usage</title> <programlisting format="linespecific"> diff --git a/modules/mtree/mtree.c b/modules/mtree/mtree.c index 63521f5..6b6283d 100644 --- a/modules/mtree/mtree.c +++ b/modules/mtree/mtree.c @@ -336,7 +336,7 @@ is_t* mt_get_tvalue(m_tree_t *pt, str *tomatch, int *len)
int mt_add_tvalues(struct sip_msg *msg, m_tree_t *pt, str *tomatch) { - int l; + int l, n; mt_node_t *itn; int_str val, values_avp_name; unsigned short values_name_type; @@ -353,7 +353,9 @@ int mt_add_tvalues(struct sip_msg *msg, m_tree_t *pt, str *tomatch) return -1; }
- l = 0; + destroy_avps(values_name_type, values_avp_name, 1); + + l = n = 0; itn = pt->head;
while (itn != NULL && l < tomatch->len && l < MT_MAX_DEPTH) { @@ -377,6 +379,7 @@ int mt_add_tvalues(struct sip_msg *msg, m_tree_t *pt, str *tomatch) val.s.s); add_avp(values_name_type|AVP_VAL_STR, values_avp_name, val); } + n++; tvalues = tvalues->next; }
@@ -384,7 +387,10 @@ int mt_add_tvalues(struct sip_msg *msg, m_tree_t *pt, str *tomatch) l++; }
- return 0; + if (n > 0) + return 0; + else + return -1; }
int mt_match_prefix(struct sip_msg *msg, m_tree_t *it,
-
Juha Heinanen