[sr-dev] git:master:62a669f9: Merge pull request #1527 from tdimitrov/pcscf_crash

GitHub noreply at github.com
Thu May 24 08:36:34 CEST 2018


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

Author: Daniel-Constantin Mierla <miconda at gmail.com>
Committer: GitHub <noreply at github.com>
Date: 2018-05-24T08:36:28+02:00

Merge pull request #1527 from tdimitrov/pcscf_crash

ims_registrar_pcscf: Update tmp security only if there are sec-agree …

---

Modified: src/modules/ims_registrar_pcscf/save.c

---

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

---

diff --git a/src/modules/ims_registrar_pcscf/save.c b/src/modules/ims_registrar_pcscf/save.c
index 0edf7bd808..39dfe4d589 100644
--- a/src/modules/ims_registrar_pcscf/save.c
+++ b/src/modules/ims_registrar_pcscf/save.c
@@ -354,9 +354,11 @@ int save_pending(struct sip_msg* _m, udomain_t* _d) {
 	}
 
     // Update security parameters
-    if(ul.update_temp_security(_d, sec_params->type, sec_params, pcontact) != 0)
-    {
-        LM_ERR("Error updating temp security\n");
+    if(sec_params) {
+        if(ul.update_temp_security(_d, sec_params->type, sec_params, pcontact) != 0)
+        {
+            LM_ERR("Error updating temp security\n");
+        }
     }
 
 	ul.unlock_udomain(_d, &ci.via_host, ci.via_port, ci.via_prot);




More information about the sr-dev mailing list