[sr-dev] git:master: carrierroute: updates for usage of pv cache in pv_elem_t

Module: sip-router Branch: master Commit: 111bb96a0f18a3a1366e60deed54a4bd54ba865b URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=111bb96… Author: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt; Committer: Daniel-Constantin Mierla &lt;miconda(a)gmail.com&gt; Date: Fri Sep 28 23:31:50 2012 +0200 carrierroute: updates for usage of pv cache in pv_elem_t --- modules/carrierroute/cr_fixup.c | 6 +++--- modules/carrierroute/cr_func.c | 38 +++++++++++++++++++------------------- 2 files changed, 22 insertions(+), 22 deletions(-) diff --git a/modules/carrierroute/cr_fixup.c b/modules/carrierroute/cr_fixup.c index 0659907..0477e28 100644 --- a/modules/carrierroute/cr_fixup.c +++ b/modules/carrierroute/cr_fixup.c @@ -186,9 +186,9 @@ static int avp_name_fixup(void ** param) { LM_ERR("could not fixup parameter"); return -1; } - if (((gparam_p)(*param))->v.pve->spec.type == PVT_AVP && - ((gparam_p)(*param))->v.pve->spec.pvp.pvn.u.isname.name.s.len == 0 && - ((gparam_p)(*param))->v.pve->spec.pvp.pvn.u.isname.name.s.s == 0) { + if (((gparam_p)(*param))->v.pve->spec->type == PVT_AVP && + ((gparam_p)(*param))->v.pve->spec->pvp.pvn.u.isname.name.s.len == 0 && + ((gparam_p)(*param))->v.pve->spec->pvp.pvn.u.isname.name.s.s == 0) { LM_ERR("malformed or non AVP type definition\n"); return -1; } diff --git a/modules/carrierroute/cr_func.c b/modules/carrierroute/cr_func.c index 2b9a85f..3cda514 100644 --- a/modules/carrierroute/cr_func.c +++ b/modules/carrierroute/cr_func.c @@ -85,16 +85,16 @@ static inline int cr_gp2id(struct sip_msg *_msg, gparam_t *gp, struct name_map_t break; case GPARAM_TYPE_PVE: /* does this PV hold an AVP? */ - if (gp->v.pve->spec.type==PVT_AVP) { - avp = search_first_avp(gp->v.pve->spec.pvp.pvn.u.isname.type, - gp->v.pve->spec.pvp.pvn.u.isname.name, &avp_val, 0); + if (gp->v.pve->spec->type==PVT_AVP) { + avp = search_first_avp(gp->v.pve->spec->pvp.pvn.u.isname.type, + gp->v.pve->spec->pvp.pvn.u.isname.name, &avp_val, 0); if (!avp) { - if(gp->v.pve->spec.pvp.pvn.u.isname.type & AVP_NAME_STR) - LM_ERR("cannot find AVP '%.*s'\n", gp->v.pve->spec.pvp.pvn.u.isname.name.s.len, - gp->v.pve->spec.pvp.pvn.u.isname.name.s.s); - else if(gp->v.pve->spec.pvp.pvn.u.isname.type & AVP_NAME_RE) + if(gp->v.pve->spec->pvp.pvn.u.isname.type & AVP_NAME_STR) + LM_ERR("cannot find AVP '%.*s'\n", gp->v.pve->spec->pvp.pvn.u.isname.name.s.len, + gp->v.pve->spec->pvp.pvn.u.isname.name.s.s); + else if(gp->v.pve->spec->pvp.pvn.u.isname.type & AVP_NAME_RE) LM_ERR("cannot find AVP regex\n"); - else LM_ERR("cannot find AVP '%d'\n", gp->v.pve->spec.pvp.pvn.u.isname.name.n); + else LM_ERR("cannot find AVP '%d'\n", gp->v.pve->spec->pvp.pvn.u.isname.name.n); return -1; } if ((avp->flags&AVP_VAL_STR)==0) { @@ -102,12 +102,12 @@ static inline int cr_gp2id(struct sip_msg *_msg, gparam_t *gp, struct name_map_t } else { id = map_name2id(map, size, &avp_val.s); if (id < 0) { - if(gp->v.pve->spec.pvp.pvn.u.isname.type & AVP_NAME_STR) - LM_ERR("cannot map carrier with id %.*s from AVP '%.*s'\n", avp_val.s.len, avp_val.s.s, gp->v.pve->spec.pvp.pvn.u.isname.name.s.len, - gp->v.pve->spec.pvp.pvn.u.isname.name.s.s); - else if(gp->v.pve->spec.pvp.pvn.u.isname.type & AVP_NAME_RE) + if(gp->v.pve->spec->pvp.pvn.u.isname.type & AVP_NAME_STR) + LM_ERR("cannot map carrier with id %.*s from AVP '%.*s'\n", avp_val.s.len, avp_val.s.s, gp->v.pve->spec->pvp.pvn.u.isname.name.s.len, + gp->v.pve->spec->pvp.pvn.u.isname.name.s.s); + else if(gp->v.pve->spec->pvp.pvn.u.isname.type & AVP_NAME_RE) LM_ERR("cannot map carrier with id %.*s from AVP regex\n", avp_val.s.len, avp_val.s.s); - else LM_ERR("cannot map carrier with id %.*s from AVP '%d'\n", avp_val.s.len, avp_val.s.s, gp->v.pve->spec.pvp.pvn.u.isname.name.n); + else LM_ERR("cannot map carrier with id %.*s from AVP '%d'\n", avp_val.s.len, avp_val.s.s, gp->v.pve->spec->pvp.pvn.u.isname.name.n); return -1; } return id; @@ -187,8 +187,8 @@ static int set_next_domain_on_rule(struct failure_route_rule *frr_head, ((rr->host.len == 0) || (str_strcmp(host, &rr->host)==0)) && (reply_code_matcher(&(rr->reply_code), reply_code)==0)) { avp_val.n = rr->next_domain; - if (add_avp(dstavp->v.pve->spec.pvp.pvn.u.isname.type, - dstavp->v.pve->spec.pvp.pvn.u.isname.name, avp_val)<0) { + if (add_avp(dstavp->v.pve->spec->pvp.pvn.u.isname.type, + dstavp->v.pve->spec->pvp.pvn.u.isname.name, avp_val)<0) { LM_ERR("set AVP failed\n"); return -1; } @@ -354,8 +354,8 @@ static int actually_rewrite(const struct route_rule *rs, str *dest, if (descavp) { avp_val.s = rs->comment; - if (add_avp(AVP_VAL_STR | descavp->v.pve->spec.pvp.pvn.u.isname.type, - descavp->v.pve->spec.pvp.pvn.u.isname.name, avp_val)<0) { + if (add_avp(AVP_VAL_STR | descavp->v.pve->spec->pvp.pvn.u.isname.type, + descavp->v.pve->spec->pvp.pvn.u.isname.name, avp_val)<0) { LM_ERR("set AVP failed\n"); return -1; } @@ -639,8 +639,8 @@ int cr_load_user_carrier(struct sip_msg * _msg, gparam_t *_user, gparam_t *_doma return -1; } else { /* set avp */ - if (add_avp(_dstavp->v.pve->spec.pvp.pvn.u.isname.type, - _dstavp->v.pve->spec.pvp.pvn.u.isname.name, avp_val)<0) { + if (add_avp(_dstavp->v.pve->spec->pvp.pvn.u.isname.type, + _dstavp->v.pve->spec->pvp.pvn.u.isname.name, avp_val)<0) { LM_ERR("add AVP failed\n"); return -1; }