[sr-dev] git:5.4:aed88c64: pv: use the corresponding functions for parsing names of xavu and xavi

[Daniel-Constantin Mierla]

Module: kamailio
Branch: 5.4
Commit: aed88c64710493452ee3ea8468c80c3c01287111
URL: https://github.com/kamailio/kamailio/commit/aed88c64710493452ee3ea8468c80c3c01287111

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date: 2020-09-29T09:20:49+02:00

pv: use the corresponding functions for parsing names of xavu and xavi

(cherry picked from commit aa8589ba66f984e93438a91a7fe8b26bd9d5b62e)

---

Modified: src/modules/pv/pv.c
Modified: src/modules/pv/pv_xavp.c

---

Diff:  https://github.com/kamailio/kamailio/commit/aed88c64710493452ee3ea8468c80c3c01287111.diff
Patch: https://github.com/kamailio/kamailio/commit/aed88c64710493452ee3ea8468c80c3c01287111.patch

---

diff --git a/src/modules/pv/pv.c b/src/modules/pv/pv.c
index 4d3d8b343c..ab48a54647 100644
--- a/src/modules/pv/pv.c
+++ b/src/modules/pv/pv.c
@@ -96,10 +96,10 @@ static pv_export_t mod_pvs[] = {
 		pv_parse_xavp_name, 0, 0, 0 },
 	{ {"xavu", sizeof("xavu")-1}, /* xavu */
 		PVT_XAVU, pv_get_xavu, pv_set_xavu,
-		pv_parse_xavp_name, 0, 0, 0 },
+		pv_parse_xavu_name, 0, 0, 0 },
 	{ {"xavi", sizeof("xavi")-1}, /* xavi */
 		PVT_XAVI, pv_get_xavi, pv_set_xavi,
-		pv_parse_xavp_name, 0, 0, 0 },
+		pv_parse_xavi_name, 0, 0, 0 },
 	{{"avp", (sizeof("avp")-1)}, PVT_AVP, pv_get_avp, pv_set_avp,
 		pv_parse_avp_name, pv_parse_index, 0, 0},
 	{{"hdr", (sizeof("hdr")-1)}, PVT_HDR, pv_get_hdr, 0, pv_parse_hdr_name,
@@ -1221,7 +1221,7 @@ static int w_xav_child_seti(sip_msg_t *msg, char *prname, char *pcname,
 		return -1;
 	}
 
- 	return ki_xav_child_seti(msg, &rname, &cname, ival, _case);
+	return ki_xav_child_seti(msg, &rname, &cname, ival, _case);
 }
 
 static int w_xavp_child_seti(sip_msg_t *msg, char *prname, char *pcname,
diff --git a/src/modules/pv/pv_xavp.c b/src/modules/pv/pv_xavp.c
index 9510c405ba..36127a2484 100644
--- a/src/modules/pv/pv_xavp.c
+++ b/src/modules/pv/pv_xavp.c
@@ -234,7 +234,7 @@ int pv_set_xavp(struct sip_msg* msg, pv_param_t *param,
 			xavp_rm_by_index(&xname->name, idx, NULL);
 			return 0;
 		}
-		
+
 		if(xname->next->index.type==PVT_EXTRA)
 		{
 			/* get the index */
@@ -352,7 +352,7 @@ int pv_set_xavp(struct sip_msg* msg, pv_param_t *param,
 			return -1;
 		return 0;
 	}
-		
+
 	/* xavp with xavp list value */
 	if(xname->next->index.type==PVT_EXTRA)
 	{
@@ -389,7 +389,7 @@ int pv_set_xavp(struct sip_msg* msg, pv_param_t *param,
 
 		if(avp->val.type!=SR_XTYPE_XAVP)
 			return -1;
-			
+
 		if(xname->next->index.type==PVT_EXTRA) {
 			if(idxf1==PV_IDX_ALL) {
 				/* ignore: should iterate and set same value to all xavps
@@ -416,7 +416,7 @@ int pv_set_xavp(struct sip_msg* msg, pv_param_t *param,
 	/* add new xavp with xavp list */
 	if(xavp_add_value(&xname->next->name, &xval, &list)==NULL)
 		return -1;
-	
+
 	/* build xavp value */
 	memset(&xval, 0, sizeof(sr_xval_t));
 	xval.type = SR_XTYPE_XAVP;
@@ -895,7 +895,7 @@ int pv_set_xavu(struct sip_msg* msg, pv_param_t *param,
 			xavu_rm_by_name(&xname->name, NULL);
 			return 0;
 		}
-		
+
 		avu = xavu_lookup(&xname->name, NULL);
 		if(avu!=NULL && avu->val.type==SR_XTYPE_XAVP) {
 			xavu_rm_by_name(&xname->next->name, &avu->val.v.xavp);