Ack /Hans > -----Original Message----- > From: [email protected] [mailto:[email protected]] > Sent: den 22 maj 2014 10:31 > To: Hans Feldt; [email protected]; Hans Nordebäck > Cc: [email protected] > Subject: [PATCH 1 of 1] amfd: initialize current assignment attributes in SU > constructor [713] > > osaf/services/saf/amf/amfd/su.cc | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > > SIs are not assigned to SU when SU was unlocked. > > AMFD checks su->saAmfSUNumCurrActiveSIs and assigns SIs if its value is less > than saAmfSGMaxActiveSIsperSU then only SIs are assigned to the SU. Since > saAmfSUNumCurrActiveSIs is not initialized to 0, the garbage value can > be very high leading to unassignment. > > Initialize saAmfSUNumCurrActiveSIs and saAmfSUNumCurrStandbySIs to 0 in > SU contructor. > > diff --git a/osaf/services/saf/amf/amfd/su.cc > b/osaf/services/saf/amf/amfd/su.cc > --- a/osaf/services/saf/amf/amfd/su.cc > +++ b/osaf/services/saf/amf/amfd/su.cc > @@ -42,6 +42,8 @@ AVD_SU::AVD_SU(const SaNameT *dn) { > saAmfSUOperState = SA_AMF_OPERATIONAL_DISABLED; > saAmfSUPresenceState = SA_AMF_PRESENCE_UNINSTANTIATED; > saAmfSuReadinessState = SA_AMF_READINESS_OUT_OF_SERVICE; > + saAmfSUNumCurrActiveSIs = 0; > + saAmfSUNumCurrStandbySIs = 0; > su_is_external = false; > sg_of_su = NULL; > su_on_node = NULL;
------------------------------------------------------------------------------ "Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE Instantly run your Selenium tests across 300+ browser/OS combos. Get unparalleled scalability from the best Selenium testing platform available Simple to use. Nothing to install. Get started now for free." http://p.sf.net/sfu/SauceLabs _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
