- **status**: review --> fixed
- **Comment**:

[staging:931f2e]
[staging:f22e0f]
[staging:f72077]
[staging:b1f859]

changeset:   5872:b1f8591c9b06
tag:         tip
parent:      5868:d0b1dccbf675
user:        Neelakanta Reddy <reddy.neelaka...@oracle.com>
date:        Fri Sep 19 19:13:18 2014 +0530
summary:     imm:corrected missing cl_node NULL check for imma resurrect [#1094]

changeset:   5871:f72077302d66
branch:      opensaf-4.5.x
parent:      5867:d473834e29e6
user:        Neelakanta Reddy <reddy.neelaka...@oracle.com>
date:        Fri Sep 19 19:13:18 2014 +0530
summary:     imm:corrected missing cl_node NULL check for imma resurrect [#1094]

changeset:   5870:f22e0f187c56
branch:      opensaf-4.4.x
parent:      5866:cb97bdd822a7
user:        Neelakanta Reddy <reddy.neelaka...@oracle.com>
date:        Fri Sep 19 19:13:18 2014 +0530
summary:     imm:corrected missing cl_node NULL check for imma resurrect [#1094]

changeset:   5869:931f2e7d2178
branch:      opensaf-4.3.x
parent:      5865:675468762129
user:        Neelakanta Reddy <reddy.neelaka...@oracle.com>
date:        Fri Sep 19 19:13:18 2014 +0530
summary:     imm:corrected missing cl_node NULL check for imma resurrect [#1094]





---

** [tickets:#1094] imm: missing NULL check in imma_xx_resurrect**

**Status:** fixed
**Milestone:** 4.3.3
**Created:** Tue Sep 16, 2014 01:02 PM UTC by Zoran Milinkovic
**Last Updated:** Fri Sep 19, 2014 12:26 PM UTC
**Owner:** Neelakanta Reddy

In imma_om_resurrect and imma_oi_resurrect, imma_client_node_get may set 
cl_node to NULL, and later usage of cl_node may cause uncertain behavior.

imma_om_resurrect:

         imma_client_node_get(&cb->client_tree, &immHandle, &cl_node);
         if (cl_node && cl_node->isOm)
         {
                 cl_node->stale = true;
                 cl_node->exposed = true;
         } else {
                 TRACE_3("client_node_get failed");
         }
         timeout = cl_node->syncr_timeout;

imma_oi_resurrect:

         imma_client_node_get(&cb->client_tree, &immOiHandle, &cl_node);
         if (cl_node && !cl_node->isOm) {
                 cl_node->stale = true;
                 cl_node->exposed = true;
         } else {
                 TRACE_3("client_node_get failed");
         }
 
         timeout = cl_node->syncr_timeout;


---

Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is 
subscribed to https://sourceforge.net/p/opensaf/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/opensaf/admin/tickets/options.  Or, if this is a 
mailing list, you can unsubscribe from the mailing list.
------------------------------------------------------------------------------
Slashdot TV.  Video for Nerds.  Stuff that Matters.
http://pubads.g.doubleclick.net/gampad/clk?id=160591471&iu=/4140/ostg.clktrk
_______________________________________________
Opensaf-tickets mailing list
Opensaf-tickets@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets

Reply via email to