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

Module: kamailio Branch: master Commit: 62a669f9f4125f88e60b2a1f621442a8ca923c82 URL: https://github.com/kamailio/kamailio/commit/62a669f9f4125f88e60b2a1f621442a… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)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/62a669f9f4125f88e60b2a1f621442a… Patch: https://github.com/kamailio/kamailio/commit/62a669f9f4125f88e60b2a1f621442a… --- 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);