Ack, The change in avnd_su_pres_inst_suterm_hdler is straightforward. However, I think the change in avnd_su_pres_terming_compuninst() got to be tested in broader flows than this ticket.
Thanks, Mathi. > -----Original Message----- > From: Nagendra Kumar > Sent: Monday, July 08, 2013 4:42 PM > To: Praveen Malviya; Mathivanan Naickan Palanivelu > Cc: [email protected] > Subject: [PATCH 1 of 1] amfnd: Don't execute terminate command for > uninstantiated component [#485] > > osaf/services/saf/avsv/avnd/avnd_susm.c | 8 ++++++++ > 1 files changed, 8 insertions(+), 0 deletions(-) > > > diff --git a/osaf/services/saf/avsv/avnd/avnd_susm.c > b/osaf/services/saf/avsv/avnd/avnd_susm.c > --- a/osaf/services/saf/avsv/avnd/avnd_susm.c > +++ b/osaf/services/saf/avsv/avnd/avnd_susm.c > @@ -1923,6 +1923,10 @@ uint32_t avnd_su_pres_inst_suterm_hdler( > for (curr_comp = > m_AVND_COMP_FROM_SU_DLL_NODE_GET(m_NCS_DBLIST_FIND_LAST(& > su->comp_list)); > curr_comp; > curr_comp = > m_AVND_COMP_FROM_SU_DLL_NODE_GET(m_NCS_DBLIST_FIND_PREV( > &curr_comp->su_dll_node))) { > + > + if (curr_comp->pres == > SA_AMF_PRESENCE_UNINSTANTIATED) > + continue; > + > /* terminate the pi comp */ > if > (m_AVND_COMP_TYPE_IS_PREINSTANTIABLE(curr_comp)) { > TRACE("Running the component clc FSM, > terminate the component"); > @@ -2283,6 +2287,10 @@ uint32_t avnd_su_pres_terming_compuninst > for (curr_comp = > m_AVND_COMP_FROM_SU_DLL_NODE_GET(m_NCS_DBLIST_FIND_PREV( > &comp->su_dll_node)); > curr_comp; > curr_comp = > m_AVND_COMP_FROM_SU_DLL_NODE_GET(m_NCS_DBLIST_FIND_PREV( > &curr_comp->su_dll_node))) { > + > + if (curr_comp->pres == > SA_AMF_PRESENCE_UNINSTANTIATED) > + continue; > + > /* terminate the pi comp */ > if > (m_AVND_COMP_TYPE_IS_PREINSTANTIABLE(curr_comp)) { > TRACE("Running the component clc FSM"); ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
