[SR-Users] tag column on address table

Kelvin Chua kelchy at gmail.com
Mon Jan 6 13:09:59 CET 2014


sorry my bad, it is available on the other address functions, just not on
the allow_source_address_group.
anyway, here is the patch to master

diff --git a/modules/permissions/hash.c b/modules/permissions/hash.c
index 3a16581..c80a3f2 100644
--- a/modules/permissions/hash.c
+++ b/modules/permissions/hash.c
@@ -486,13 +486,23 @@ int find_group_in_addr_hash_table(struct addr_list**
table,
 {
        struct addr_list *np;
        str addr_str;
+       int_str avp_val;

        addr_str.s = (char*)addr->u.addr;
        addr_str.len = 4;

+       get_tag_avp(&tag_avp, &tag_avp_type);
        for (np = table[perm_hash(addr_str)]; np != NULL; np = np->next) {
                if (((np->port == 0) || (np->port == port))
                                && ip_addr_cmp(&np->addr, addr)) {
+                       if ( np->tag.s != NULL ) {
+                               avp_val.s.s = (char *)np->tag.s;
+                               avp_val.s.len = strlen(avp_val.s.s);
+                               if (add_avp(tag_avp_type|AVP_VAL_STR,
tag_avp, avp_val) != 0) {
+                                       LM_ERR("failed to set of tag_avp
failed\n");
+                                       return -1;
+                               }
+                       }
                        return np->grp;
                }
        }


Kelvin Chua


On Mon, Jan 6, 2014 at 7:59 PM, Kelvin Chua <kelchy at gmail.com> wrote:

> checking the source code, looks like, it's not being set at all on any of
> those functions
>
> Kelvin Chua
>
>
> On Mon, Jan 6, 2014 at 7:37 PM, Daniel-Constantin Mierla <
> miconda at gmail.com> wrote:
>
>> Hello,
>>
>>
>> On 04/01/14 21:47, Juha Heinanen wrote:
>>
>>> Kelvin Chua writes:
>>>
>>>  is the peer_tag_avp used on allow_source_address_group?
>>>>
>>> based on fast scan of permissions module source, it appears that
>>> peer_tag_avp is set only by allow_trusted function.
>>>
>> the peer tag avp should be set also for allow_address() and
>> allow_source_address(). I don't think it is set for
>> allow_source_address_group().
>>
>> Cheers,
>> Daniel
>>
>> --
>> Daniel-Constantin Mierla - http://www.asipto.com
>> http://twitter.com/#!/miconda - http://www.linkedin.com/in/miconda
>>
>>
>>
>> _______________________________________________
>> SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
>> sr-users at lists.sip-router.org
>> http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-users/attachments/20140106/8ae9316a/attachment.html>


More information about the sr-users mailing list