[sr-dev] git:master:09fc2d25: modules/ims_regstrar_scscf: remove process id check that could fail on messages processed into shm

jaybeepee jason.penton at gmail.com
Mon Sep 12 16:58:34 CEST 2016


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

Author: jaybeepee <jason.penton at gmail.com>
Committer: jaybeepee <jason.penton at gmail.com>
Date: 2016-09-12T16:57:21+02:00

modules/ims_regstrar_scscf: remove process id check that could fail on messages processed into shm

---

Modified: modules/ims_registrar_scscf/pvt_message.c

---

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

---

diff --git a/modules/ims_registrar_scscf/pvt_message.c b/modules/ims_registrar_scscf/pvt_message.c
index 3b8274e..22d54aa 100644
--- a/modules/ims_registrar_scscf/pvt_message.c
+++ b/modules/ims_registrar_scscf/pvt_message.c
@@ -84,9 +84,7 @@ struct sip_msg* get_request_from_tx(struct cell *t) {
         (which we cannot assume) then we would pollute the shm_msg t->uas.request if we did any parsing on it. Instead, we need to 
         make a private copy of the message and free it when we are done 
          */
-        if ((_pv_treq.T != t || t->uas.request != _pv_treq.tmsgp)
-                && t->uas.request->id != _pv_treq.id) {
-
+        if ((_pv_treq.T != t || t->uas.request != _pv_treq.tmsgp)) { 
             /* make a copy */
             if (_pv_treq.buf == NULL || _pv_treq.buf_size < t->uas.request->len + 1) {
                 if (_pv_treq.buf != NULL)




More information about the sr-dev mailing list