[sr-dev] git:4.3:9c6eef91: avpops: remove unecessary test for deleting avps by name rules

[Daniel-Constantin Mierla]

Module: kamailio
Branch: 4.3
Commit: 9c6eef916be7926ae2c7f926da670a3e01226232
URL: https://github.com/kamailio/kamailio/commit/9c6eef916be7926ae2c7f926da670a3e01226232

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: Daniel-Constantin Mierla <miconda at gmail.com>
Date: 2015-07-14T15:22:18+02:00

avpops: remove unecessary test for deleting avps by name rules

- causes crash, following the change to use cached pv structures
- reported by Alexandr Dubovikov, GH #239

(cherry picked from commit b120b47c39034d4bc384ff733ce62b845f2b4062)

---

Modified: modules/avpops/avpops_impl.c

---

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

---

diff --git a/modules/avpops/avpops_impl.c b/modules/avpops/avpops_impl.c
index 5d1351a..4a81eb9 100644
--- a/modules/avpops/avpops_impl.c
+++ b/modules/avpops/avpops_impl.c
@@ -791,10 +791,6 @@ int ops_delete_avp(struct sip_msg* msg, struct fis_param *ap)
 			((ap->opd&AVPOPS_VAL_INT)&&((avp->flags&AVP_NAME_STR))==0) ||
 			((ap->opd&AVPOPS_VAL_STR)&&(avp->flags&AVP_NAME_STR)) )  )
 				continue;
-			if((ap->u.sval->pvp.pvn.u.isname.type&AVP_SCRIPT_MASK)!=0
-					&& ((ap->u.sval->pvp.pvn.u.isname.type&AVP_SCRIPT_MASK)
-								&avp->flags)==0)
-				continue;
 			/* remove avp */
 			destroy_avp( avp );
 			n++;