[sr-dev] git:master:01ec5207: modules/ims_registrar_scscf: reset t pointer if tx not found - avoid crash

jaybeepee jason.penton at gmail.com
Thu Apr 14 12:31:21 CEST 2016


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

Author: jaybeepee <jason.penton at gmail.com>
Committer: jaybeepee <jason.penton at gmail.com>
Date: 2016-04-14T12:31:07+02:00

modules/ims_registrar_scscf: reset t pointer if tx not found - avoid crash

---

Modified: modules/ims_registrar_scscf/cxdx_sar.c

---

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

---

diff --git a/modules/ims_registrar_scscf/cxdx_sar.c b/modules/ims_registrar_scscf/cxdx_sar.c
index c2bb291..eac67fc 100644
--- a/modules/ims_registrar_scscf/cxdx_sar.c
+++ b/modules/ims_registrar_scscf/cxdx_sar.c
@@ -141,7 +141,8 @@ void async_cdp_callback(int is_timeout, void *param, AAAMessage *saa, long elaps
         LM_DBG("There is transaction data this must have been called from save or assign server unreg");
         data = (saved_transaction_t*) param;
         if (tmb.t_lookup_ident(&t, data->tindex, data->tlabel) < 0) {
-            LM_ERR("t_continue: transaction not found\n");
+            LM_ERR("t_continue: transaction not found and t is now pointing to %p and will be set to NULL\n", t);
+            t = NULL;
             rerrno = R_SAR_FAILED;
             goto error_no_send;
         }




More information about the sr-dev mailing list