[sr-dev] git:master:8e5cf546: modules/ims_auth: remove process id check that could fail on messages processed into shm

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


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

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

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

---

Modified: modules/ims_auth/pvt_message.c

---

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

---

diff --git a/modules/ims_auth/pvt_message.c b/modules/ims_auth/pvt_message.c
index 3b8274e..17f852e 100644
--- a/modules/ims_auth/pvt_message.c
+++ b/modules/ims_auth/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