Ack from me.

Best regards,
Zoran

-----Original Message-----
From: Anders Bjornerstedt [mailto:[email protected]] 
Sent: den 16 september 2014 09:23
To: [email protected]
Cc: [email protected]
Subject: [devel] [PATCH 1 of 1] imm: Compare err instead of assign in 
immnd_evt_proc_rt_object_delete [#1087]

 osaf/services/saf/immsv/immnd/immnd_evt.c |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


Error detected by coverity.

diff --git a/osaf/services/saf/immsv/immnd/immnd_evt.c 
b/osaf/services/saf/immsv/immnd/immnd_evt.c
--- a/osaf/services/saf/immsv/immnd/immnd_evt.c
+++ b/osaf/services/saf/immsv/immnd/immnd_evt.c
@@ -7091,7 +7091,7 @@ static void immnd_evt_proc_rt_object_del
                }
        } 
 
-        if(spApplConn && (err = SA_AIS_OK) && !delayedReply) {
+        if(spApplConn && (err == SA_AIS_OK) && !delayedReply) {
                int ix = 0;
                /* Indicates object is marked with SA_IMM_ATTR_NOTIFY and
                   special applier is present at this node and we dont need to

------------------------------------------------------------------------------
Want excitement?
Manually upgrade your production database.
When you want reliability, choose Perforce.
Perforce version control. Predictably reliable.
http://pubads.g.doubleclick.net/gampad/clk?id=157508191&iu=/4140/ostg.clktrk
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

------------------------------------------------------------------------------
Want excitement?
Manually upgrade your production database.
When you want reliability, choose Perforce.
Perforce version control. Predictably reliable.
http://pubads.g.doubleclick.net/gampad/clk?id=157508191&iu=/4140/ostg.clktrk
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to