[sr-dev] git:master:caeda232: registrar: reg_xavp_cfg should not be reset when sock_flag is not set

Ian Yu-Hsun Lin ianyuhsunlin at gmail.com
Thu Jan 14 09:29:11 CET 2016


Module: kamailio
Branch: master
Commit: caeda232d6e2d29a5da202c9db244b407c94ede2
URL: https://github.com/kamailio/kamailio/commit/caeda232d6e2d29a5da202c9db244b407c94ede2

Author: Ian Yu-Hsun Lin <ianyuhsunlin at gmail.com>
Committer: Ian Yu-Hsun Lin <ianyuhsunlin at gmail.com>
Date: 2016-01-14T16:02:25+08:00

registrar: reg_xavp_cfg should not be reset when sock_flag is not set

- since there are functions like `registered()` would use it
- reported by Jayesh Nambiar (@jayesh1017) in the comment of GH#470

---

Modified: modules/registrar/reg_mod.c

---

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

---

diff --git a/modules/registrar/reg_mod.c b/modules/registrar/reg_mod.c
index 0d7c5ac..f859db4 100644
--- a/modules/registrar/reg_mod.c
+++ b/modules/registrar/reg_mod.c
@@ -365,7 +365,6 @@ static int mod_init(void)
 	} else if (reg_xavp_cfg.s) {
 		if (reg_xavp_cfg.len == 0 || sock_flag == -1) {
 			LM_WARN("empty reg_xavp_cfg or sock_flag no set -> resetting\n");
-			reg_xavp_cfg.len = 0;
 			sock_flag = -1;
 		}
 	} else if (sock_flag!=-1) {




More information about the sr-dev mailing list