modules/ims_qos,modules/ims_registrar_scscf,modules/ims_auth: Warning logical-op...
authorjaybeepee <jason.penton@gmail.com>
Tue, 7 Jun 2016 06:40:29 +0000 (08:40 +0200)
committerjaybeepee <jason.penton@gmail.com>
Tue, 7 Jun 2016 06:42:15 +0000 (08:42 +0200)
modules/ims_auth/pvt_message.c
modules/ims_qos/mod.c
modules/ims_registrar_scscf/pvt_message.c

index ba8a53b..3b8274e 100644 (file)
@@ -84,7 +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
+        if ((_pv_treq.T != t || t->uas.request != _pv_treq.tmsgp)
                 && t->uas.request->id != _pv_treq.id) {
 
             /* make a copy */
index af9ed91..b0ec900 100644 (file)
@@ -683,7 +683,7 @@ static int w_rx_aar(struct sip_msg *msg, char *route, char* dir, char *c_id, int
         (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
+    if ((_pv_treq.T != t || t->uas.request != _pv_treq.tmsgp)
             && t->uas.request->id != _pv_treq.id) {
 
         /* make a copy */
index ba8a53b..3b8274e 100644 (file)
@@ -84,7 +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
+        if ((_pv_treq.T != t || t->uas.request != _pv_treq.tmsgp)
                 && t->uas.request->id != _pv_treq.id) {
 
             /* make a copy */