Hi Hung, Ack from me. Reviewed and tested the patch.
Best regards, Zoran -----Original Message----- From: Hung Nguyen [mailto:[email protected]] Sent: Wednesday, September 09, 2015 7:02 PM To: Anders Björnerstedt; Zoran Milinkovic; [email protected] Cc: [email protected] Subject: [PATCH 1 of 1] imm: Don't assign default values to empty valued attributes when sync [#1472] osaf/services/saf/immsv/immnd/ImmModel.cc | 12 ++---------- 1 files changed, 2 insertions(+), 10 deletions(-) Don't assign default values to empty valued attributes when sync. This fix is for both CONFIG and RUNTIME objects. diff --git a/osaf/services/saf/immsv/immnd/ImmModel.cc b/osaf/services/saf/immsv/immnd/ImmModel.cc --- a/osaf/services/saf/immsv/immnd/ImmModel.cc +++ b/osaf/services/saf/immsv/immnd/ImmModel.cc @@ -16180,17 +16180,9 @@ ImmModel::objectSync(const ImmsvOmObject ImmAttrValue* attrValue = NULL; if(attr->mFlags & SA_IMM_ATTR_MULTI_VALUE) { - if(attr->mDefaultValue.empty()) { - attrValue = new ImmAttrMultiValue(); - } else { - attrValue = new ImmAttrMultiValue(attr->mDefaultValue); - } + attrValue = new ImmAttrMultiValue(); } else { - if(attr->mDefaultValue.empty()) { - attrValue = new ImmAttrValue(); - } else { - attrValue = new ImmAttrValue(attr->mDefaultValue); - } + attrValue = new ImmAttrValue(); } //Set admin owner as a regular attribute and then also a pointer ------------------------------------------------------------------------------ Monitor Your Dynamic Infrastructure at Any Scale With Datadog! Get real-time metrics from all of your servers, apps and tools in one place. SourceForge users - Click here to start your Free Trial of Datadog now! http://pubads.g.doubleclick.net/gampad/clk?id=241902991&iu=/4140 _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
