Ack (Not tested). Thanks -Nagu
> -----Original Message----- > From: Nguyen TK Luu [mailto:[email protected]] > Sent: 19 January 2017 13:37 > To: [email protected]; [email protected]; > [email protected]; Nagendra Kumar; Praveen Malviya > Cc: [email protected] > Subject: [PATCH 1 of 1] amf: change saAmfCompCmdEnv attribute to > writable [#2255] > > src/amf/amfd/comp.cc | 15 +++++++++++++++ > src/amf/amfnd/compdb.cc | 1 + > src/amf/common/amf_defs.h | 1 + > src/amf/config/amf_classes.xml | 1 + > 4 files changed, 18 insertions(+), 0 deletions(-) > > > diff --git a/src/amf/amfd/comp.cc b/src/amf/amfd/comp.cc > --- a/src/amf/amfd/comp.cc > +++ b/src/amf/amfd/comp.cc > @@ -963,6 +963,15 @@ > } > } > } > + } else if (!strcmp(attribute->attrName, > "saAmfCompCmdEnv")) { > + if (value_is_deleted == true) > + continue; > + char *param_val = *((char **)value); > + if (nullptr == param_val) { > + report_ccb_validation_error(opdata, > + "Modification of > saAmfCompCmdEnv Fail, nullptr arg"); > + goto done; > + } > } else if (!strcmp(attribute->attrName, > "saAmfCompInstantiateCmdArgv")) { > if (value_is_deleted == true) > continue; > @@ -1314,6 +1323,12 @@ > param.attr_id = saAmfCompType_ID; > param.name_sec = *dn; > > + } else if (!strcmp(attribute->attrName, > "saAmfCompCmdEnv")) { > + > + /* Node director will reread configuration from IMM > */ > + param.attr_id = saAmfCompCmdEnv_ID; > + TRACE("saAmfCompCmdEnv modified."); > + > } else if (!strcmp(attribute->attrName, > "saAmfCompInstantiateCmdArgv")) { > > /* Node director will reread configuration from IMM > */ diff --git a/src/amf/amfnd/compdb.cc b/src/amf/amfnd/compdb.cc > --- a/src/amf/amfnd/compdb.cc > +++ b/src/amf/amfnd/compdb.cc > @@ -366,6 +366,7 @@ > case saAmfCompCleanupCmd_ID: > case saAmfCompAmStartCmd_ID: > case saAmfCompAmStopCmd_ID: > + case saAmfCompCmdEnv_ID: > comp->config_is_valid = 0; > break; > case saAmfCompInstantiateTimeout_ID: > diff --git a/src/amf/common/amf_defs.h b/src/amf/common/amf_defs.h > --- a/src/amf/common/amf_defs.h > +++ b/src/amf/common/amf_defs.h > @@ -247,6 +247,7 @@ > saAmfCompCurrProxyName_ID = 37, > saAmfCompAMEnable_ID = 38, > saAmfCompProxyStatus_ID = 39, > + saAmfCompCmdEnv_ID = 40, > saAmfCompType_ID, > } AVSV_AMF_COMP_ATTR_ID; > > diff --git a/src/amf/config/amf_classes.xml > b/src/amf/config/amf_classes.xml > --- a/src/amf/config/amf_classes.xml > +++ b/src/amf/config/amf_classes.xml > @@ -1056,6 +1056,7 @@ > <name>saAmfCompCmdEnv</name> > <type>SA_STRING_T</type> > <category>SA_CONFIG</category> > + <flag>SA_WRITABLE</flag> > <flag>SA_MULTI_VALUE</flag> > </attr> > <attr> ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
