Module: sip-router Branch: master Commit: cbeb136c60d4890599d39665a2d27d865ae29b76 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=cbeb136… Author: Elena-Ramona Modroiu <ramona(a)asipto.com> Committer: Elena-Ramona Modroiu <ramona(a)asipto.com> Date: Wed Sep 26 12:11:36 2012 +0200 userblacklist: updates for new pv_elem_t structure - safety checks to detect dynamic vs static string parameters --- modules_k/userblacklist/userblacklist.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/modules_k/userblacklist/userblacklist.c b/modules_k/userblacklist/userblacklist.c index 4433064..134db25 100644 --- a/modules_k/userblacklist/userblacklist.c +++ b/modules_k/userblacklist/userblacklist.c @@ -190,9 +190,9 @@ static int check_user_blacklist_fixup(void** param, int param_no) return E_UNSPEC; } - if(!model->spec.getf) { + if(model->spec==NULL || model->spec->getf==NULL) { if(param_no == 1) { - if(str2int(&s, (unsigned int*)&model->spec.pvp.pvn.u.isname.name.n) != 0) { + if(str2int(&s, (unsigned int*)&model->spec->pvp.pvn.u.isname.name.n) != 0) { LM_ERR("wrong value [%.*s] for parameter %d\n", s.len, s.s, param_no); return E_UNSPEC; } @@ -227,21 +227,21 @@ static int check_user_list(struct sip_msg *msg, char* str1, char* str2, char* st char req_number[MAXNUMBERLEN+1]; /* user */ - if(((pv_elem_p)str1)->spec.getf) { + if(((pv_elem_p)str1)->spec!=NULL && ((pv_elem_p)str1)->spec->getf!=NULL) { if(pv_printf_s(msg, (pv_elem_p)str1, &user) != 0) { LM_ERR("cannot print user pseudo-variable\n"); return -1; } } /* domain */ - if(((pv_elem_p)str2)->spec.getf) { + if(((pv_elem_p)str2)->spec!=NULL && ((pv_elem_p)str2)->spec->getf) { if(pv_printf_s(msg, (pv_elem_p)str2, &domain) != 0) { LM_ERR("cannot print domain pseudo-variable\n"); return -1; } } /* source number */ - if(str3 != NULL && ((pv_elem_p)str3)->spec.getf) { + if(str3 != NULL && ((pv_elem_p)str3)->spec!=NULL && ((pv_elem_p)str3)->spec->getf!=NULL) { if(pv_printf_s(msg, (pv_elem_p)str3, &number) != 0) { LM_ERR("cannot print number pseudo-variable\n"); return -1;