osaf/services/saf/amf/amfd/ckpt_dec.cc  |  134 ++++++++++++++++++++-----------
 osaf/services/saf/amf/amfd/ckpt_enc.cc  |  121 +++++++++++++---------------
 osaf/services/saf/amf/amfd/ckpt_updt.cc |   94 +++++++++++-----------
 3 files changed, 191 insertions(+), 158 deletions(-)


diff --git a/osaf/services/saf/amf/amfd/ckpt_dec.cc 
b/osaf/services/saf/amf/amfd/ckpt_dec.cc
--- a/osaf/services/saf/amf/amfd/ckpt_dec.cc
+++ b/osaf/services/saf/amf/amfd/ckpt_dec.cc
@@ -22,6 +22,7 @@
 
 ******************************************************************************/
 
+#include <osaf_extended_name.h>
 #include <logtrace.h>
 #include <amfd.h>
 #include <cluster.h>
@@ -208,7 +209,6 @@
        osaf_decode_uint32(ub, &cb->nodes_exit_cnt);
 }
 
-
 /****************************************************************************\
  * Function: dec_cb_config
  *
@@ -263,9 +263,12 @@
        AVD_AVND *avnd,
        const uint16_t peer_version)
 {
+       TRACE_ENTER();
+       SaNameT node_name;
        osaf_decode_uint32(ub, &avnd->node_info.nodeId);
        osaf_decode_saclmnodeaddresst(ub, &avnd->node_info.nodeAddress);
-       osaf_decode_sanamet(ub, &avnd->name);
+       osaf_decode_sanamet(ub, &node_name);
+       avnd->name = Amf::to_string(&node_name);
        osaf_decode_bool(ub, reinterpret_cast<bool*>(&avnd->node_info.member));
        osaf_decode_satimet(ub, &avnd->node_info.bootTimestamp);
        osaf_decode_uint64(ub, 
reinterpret_cast<uint64_t*>(&avnd->node_info.initialViewNumber));
@@ -275,7 +278,8 @@
        osaf_decode_uint32(ub, reinterpret_cast<uint32_t*>(&avnd->node_state));
        osaf_decode_uint32(ub, reinterpret_cast<uint32_t*>(&avnd->type));
        osaf_decode_uint32(ub, &avnd->rcv_msg_id);
-       osaf_decode_uint32(ub, &avnd->snd_msg_id);      
+       osaf_decode_uint32(ub, &avnd->snd_msg_id);
+       TRACE_LEAVE();
 }
 
 /****************************************************************************\
@@ -314,7 +318,9 @@
 
        case NCS_MBCSV_ACT_RMV:
                /* Send only key information */
-               osaf_decode_sanamet(&dec->i_uba, &avnd.name);
+               SaNameT node_name;
+               osaf_decode_sanamet(&dec->i_uba, &node_name);
+               avnd.name = Amf::to_string(&node_name);
                break;
 
        default:
@@ -333,7 +339,10 @@
 
 void decode_app(NCS_UBAID *ub, AVD_APP *app)
 {
-       osaf_decode_sanamet(ub, &app->name);
+       SaNameT app_name;
+
+       osaf_decode_sanamet(ub, &app_name);
+       app->name = Amf::to_string(&app_name);
        osaf_decode_uint32(ub, (uint32_t*)&app->saAmfApplicationAdminState);
        osaf_decode_uint32(ub, &app->saAmfApplicationCurrNumSGs);
 }
@@ -371,7 +380,10 @@
 
 static void decode_sg(NCS_UBAID *ub, AVD_SG *sg)
 {
-       osaf_decode_sanamet(ub, &sg->name);
+       SaNameT sg_name;
+
+       osaf_decode_sanamet(ub, &sg_name);
+       sg->name = Amf::to_string(&sg_name);
        osaf_decode_uint32(ub, (uint32_t*)&sg->saAmfSGAdminState);
        osaf_decode_uint32(ub, &sg->saAmfSGNumCurrAssignedSUs);
        osaf_decode_uint32(ub, &sg->saAmfSGNumCurrInstantiatedSpareSUs);
@@ -412,13 +424,18 @@
 
 static void decode_su(NCS_UBAID *ub, AVD_SU *su, uint16_t peer_version)
 {
-       osaf_decode_sanamet(ub, &su->name);
+       SaNameT su_name;
+       SaNameT node_name;
+
+       osaf_decode_sanamet(ub, &su_name);
+       su->name = Amf::to_string(&su_name);
        osaf_decode_bool(ub, (bool*)&su->saAmfSUPreInstantiable);
        osaf_decode_uint32(ub, (uint32_t*)&su->saAmfSUOperState);
        osaf_decode_uint32(ub, (uint32_t*)&su->saAmfSUAdminState);
        osaf_decode_uint32(ub, (uint32_t*)&su->saAmfSuReadinessState);
        osaf_decode_uint32(ub, (uint32_t*)&su->saAmfSUPresenceState);
-       osaf_decode_sanamet(ub, &su->saAmfSUHostedByNode);
+       osaf_decode_sanamet(ub, &node_name);
+       su->saAmfSUHostedByNode = Amf::to_string(&node_name);
        osaf_decode_uint32(ub, &su->saAmfSUNumCurrActiveSIs);
        osaf_decode_uint32(ub, &su->saAmfSUNumCurrStandbySIs);
        osaf_decode_uint32(ub, &su->saAmfSURestartCount);
@@ -473,13 +490,18 @@
 #endif
        TRACE_ENTER2("my_version: %u, to_version: %u", ver_compare, 
peer_version);
 
-       osaf_decode_sanamet(ub, &si->name);
+       SaNameT si_name;
+       SaNameT si_protected_by_sg;
+
+       osaf_decode_sanamet(ub, &si_name);
+       si->name = Amf::to_string(&si_name);
        osaf_decode_uint32(ub, (uint32_t*)&si->saAmfSIAdminState);
        osaf_decode_uint32(ub, (uint32_t*)&si->saAmfSIAssignmentState);
        osaf_decode_uint32(ub, (uint32_t*)&si->saAmfSINumCurrActiveAssignments);
        osaf_decode_uint32(ub, 
(uint32_t*)&si->saAmfSINumCurrStandbyAssignments);
        osaf_decode_uint32(ub, (uint32_t*)&si->si_switch);
-       osaf_decode_sanamet(ub, &si->saAmfSIProtectedbySG);
+       osaf_decode_sanamet(ub, &si_protected_by_sg);
+       si->saAmfSIProtectedbySG = Amf::to_string(&si_protected_by_sg);
        osaf_decode_bool(ub, &si->alarm_sent);
 
        if (peer_version >= ver_compare) {
@@ -670,12 +692,14 @@
  *
 \**************************************************************************/
 void decode_comp(NCS_UBAID *ub, AVD_COMP *comp) {
-  osaf_decode_sanamet(ub, &comp->comp_info.name);
-  osaf_decode_uint32(ub, 
reinterpret_cast<uint32_t*>(&comp->saAmfCompOperState));
-  osaf_decode_uint32(ub, 
reinterpret_cast<uint32_t*>(&comp->saAmfCompReadinessState));
-  osaf_decode_uint32(ub, 
reinterpret_cast<uint32_t*>(&comp->saAmfCompPresenceState));
-  osaf_decode_uint32(ub, &comp->saAmfCompRestartCount);
-  osaf_decode_sanamet(ub, &comp->saAmfCompCurrProxyName);
+       SaNameT proxy_name;
+       osaf_decode_sanamet(ub, &comp->comp_info.name);
+       osaf_decode_uint32(ub, 
reinterpret_cast<uint32_t*>(&comp->saAmfCompOperState));
+       osaf_decode_uint32(ub, 
reinterpret_cast<uint32_t*>(&comp->saAmfCompReadinessState));
+       osaf_decode_uint32(ub, 
reinterpret_cast<uint32_t*>(&comp->saAmfCompPresenceState));
+       osaf_decode_uint32(ub, &comp->saAmfCompRestartCount);
+       osaf_decode_sanamet(ub, &proxy_name);
+       comp->saAmfCompCurrProxyName = Amf::to_string(&proxy_name);
 }
 
 /****************************************************************************\
@@ -838,11 +862,13 @@
        /* 
         * Action in this case is just to update.
         */
-       osaf_decode_sanamet(&dec->i_uba, &avnd.name);
+       SaNameT name;
+       osaf_decode_sanamet(&dec->i_uba, &name);
+       avnd.name = Amf::to_string(&name);
        osaf_decode_uint32(&dec->i_uba, 
reinterpret_cast<uint32_t*>(&avnd.saAmfNodeAdminState));
 
-       if (nullptr == (avnd_struct = avd_node_get(&avnd.name))) {
-               LOG_ER("%s: node not found, nodeid=%s", __FUNCTION__, 
avnd.name.value);
+       if (nullptr == (avnd_struct = avd_node_get(avnd.name))) {
+               LOG_ER("%s: node not found, nodeid=%s", __FUNCTION__, 
avnd.name.c_str());
                return NCSCC_RC_FAILURE;
        }
 
@@ -879,11 +905,13 @@
        /* 
         * Action in this case is just to update.
         */
-       osaf_decode_sanamet(&dec->i_uba, &avnd.name);
+       SaNameT name;
+       osaf_decode_sanamet(&dec->i_uba, &name);
+       avnd.name = Amf::to_string(&name);
        osaf_decode_uint32(&dec->i_uba, 
reinterpret_cast<uint32_t*>(&avnd.saAmfNodeOperState));
 
-       if (nullptr == (avnd_struct = avd_node_get(&avnd.name))) {
-               LOG_ER("%s: node not found, nodeid=%s", __FUNCTION__, 
avnd.name.value);
+       if (nullptr == (avnd_struct = avd_node_get(avnd.name))) {
+               LOG_ER("%s: node not found, nodeid=%s", __FUNCTION__, 
avnd.name.c_str());
                return NCSCC_RC_FAILURE;
        }
 
@@ -920,11 +948,13 @@
        /* 
         * Action in this case is just to update.
         */
-       osaf_decode_sanamet(&dec->i_uba, &avnd.name);
+       SaNameT name;
+       osaf_decode_sanamet(&dec->i_uba, &name);
+       avnd.name = Amf::to_string(&name);
        osaf_decode_uint32(&dec->i_uba, 
reinterpret_cast<uint32_t*>(&avnd.node_state));
        
-       if (nullptr == (avnd_struct = avd_node_get(&avnd.name))) {
-               LOG_ER("%s: node not found, nodeid=%s", __FUNCTION__, 
avnd.name.value);
+       if (nullptr == (avnd_struct = avd_node_get(avnd.name))) {
+               LOG_ER("%s: node not found, nodeid=%s", __FUNCTION__, 
avnd.name.c_str());
                return NCSCC_RC_FAILURE;
        }
 
@@ -1067,7 +1097,7 @@
        TRACE("ng_using_saAmfSGAdminState:%u",sg->ng_using_saAmfSGAdminState);
 
        TRACE_LEAVE2("'%s', saAmfSGAdminState=%u, sg_updt:%d",
-               sg->name.value, sg->saAmfSGAdminState, 
cb->async_updt_cnt.sg_updt);
+               sg->name.c_str(), sg->saAmfSGAdminState, 
cb->async_updt_cnt.sg_updt);
        return NCSCC_RC_SUCCESS;
 }
 
@@ -1098,7 +1128,7 @@
        cb->async_updt_cnt.sg_updt++;
 
        TRACE_LEAVE2("'%s', saAmfSGNumCurrAssignedSUs=%u, sg_updt:%d",
-               sg->name.value, sg->saAmfSGNumCurrAssignedSUs,
+               sg->name.c_str(), sg->saAmfSGNumCurrAssignedSUs,
                cb->async_updt_cnt.sg_updt);
        return NCSCC_RC_SUCCESS;
 }
@@ -1130,7 +1160,7 @@
        cb->async_updt_cnt.sg_updt++;
 
        TRACE_LEAVE2("'%s', saAmfSGNumCurrInstantiatedSpareSUs=%u, sg_updt:%d",
-               sg->name.value, sg->saAmfSGNumCurrInstantiatedSpareSUs,
+               sg->name.c_str(), sg->saAmfSGNumCurrInstantiatedSpareSUs,
                cb->async_updt_cnt.sg_updt);
        return NCSCC_RC_SUCCESS;
 }
@@ -1162,7 +1192,7 @@
        cb->async_updt_cnt.sg_updt++;
 
        TRACE_LEAVE2("'%s', saAmfSGNumCurrNonInstantiatedSpareSUs=%u, 
sg_updt:%d",
-               sg->name.value, sg->saAmfSGNumCurrNonInstantiatedSpareSUs,
+               sg->name.c_str(), sg->saAmfSGNumCurrNonInstantiatedSpareSUs,
                cb->async_updt_cnt.sg_updt);
        return NCSCC_RC_SUCCESS;
 }
@@ -1194,7 +1224,7 @@
        cb->async_updt_cnt.sg_updt++;
 
        TRACE_LEAVE2("'%s', adjust_state=%u, sg_updt:%d",
-               sg->name.value, sg->adjust_state, cb->async_updt_cnt.sg_updt);
+               sg->name.c_str(), sg->adjust_state, cb->async_updt_cnt.sg_updt);
        return NCSCC_RC_SUCCESS;
 }
 
@@ -1225,7 +1255,7 @@
        cb->async_updt_cnt.sg_updt++;
 
        TRACE_LEAVE2("'%s', sg_fsm_state=%u, sg_updt:%d",
-               sg->name.value, sg->sg_fsm_state, cb->async_updt_cnt.sg_updt);
+               sg->name.c_str(), sg->sg_fsm_state, cb->async_updt_cnt.sg_updt);
        return NCSCC_RC_SUCCESS;
 }
 
@@ -1285,15 +1315,15 @@
        AVD_SU *su = su_db->find(Amf::to_string(&name));
        if (su == nullptr) {
                TRACE("'%s' does not exist, creating it", name.value);
-               su = new AVD_SU(&name);
-               unsigned int rc = su_db->insert(Amf::to_string(&su->name), su);
+               su = new AVD_SU(Amf::to_string(&name));
+               unsigned int rc = su_db->insert(su->name, su);
                osafassert(rc == NCSCC_RC_SUCCESS);
        }
        
        osaf_decode_uint32(&dec->i_uba, (uint32_t*)&su->saAmfSUOperState);
 
        cb->async_updt_cnt.su_updt++;
-       avd_saImmOiRtObjectUpdate(&su->name, "saAmfSUOperState",
+       avd_saImmOiRtObjectUpdate(su->name, "saAmfSUOperState",
                SA_IMM_ATTR_SAUINT32T, &su->saAmfSUOperState);
 
        TRACE_LEAVE2("'%s', saAmfSUOperState=%u, su_updt:%d",
@@ -1327,7 +1357,7 @@
 
        cb->async_updt_cnt.su_updt++;
 
-       avd_saImmOiRtObjectUpdate(&su->name, "saAmfSUAdminState",
+       avd_saImmOiRtObjectUpdate(su->name, "saAmfSUAdminState",
                SA_IMM_ATTR_SAUINT32T, &su->saAmfSUAdminState);
        TRACE_LEAVE2("'%s', saAmfSUAdminState=%u, su_updt:%d",
                name.value, su->saAmfSUAdminState, cb->async_updt_cnt.su_updt);
@@ -1360,7 +1390,7 @@
 
        cb->async_updt_cnt.su_updt++;
 
-       avd_saImmOiRtObjectUpdate(&su->name, "saAmfSUReadinessState",
+       avd_saImmOiRtObjectUpdate(su->name, "saAmfSUReadinessState",
                SA_IMM_ATTR_SAUINT32T, &su->saAmfSuReadinessState);
        TRACE_LEAVE2("'%s', saAmfSuReadinessState=%u, su_updt:%d",
                name.value, su->saAmfSuReadinessState, 
cb->async_updt_cnt.su_updt);
@@ -1393,7 +1423,7 @@
 
        cb->async_updt_cnt.su_updt++;
 
-       avd_saImmOiRtObjectUpdate(&su->name, "saAmfSUPresenceState",
+       avd_saImmOiRtObjectUpdate(su->name, "saAmfSUPresenceState",
                SA_IMM_ATTR_SAUINT32T, &su->saAmfSUPresenceState);
        TRACE_LEAVE2("'%s', saAmfSUPresenceState=%u, su_updt:%d",
                name.value, su->saAmfSUPresenceState, 
cb->async_updt_cnt.su_updt);
@@ -1657,7 +1687,7 @@
        osaf_decode_sanamet(&dec->i_uba, &name);
        AVD_SI *si = si_db->find(Amf::to_string(&name));
        if (si == nullptr) {
-               si = avd_si_new(&name);
+               si = avd_si_new(Amf::to_string(&name));
                osafassert(si != nullptr);
                avd_si_db_add(si);              
        }
@@ -1818,6 +1848,7 @@
 static uint32_t dec_comp_proxy_comp_name(AVD_CL_CB *cb, NCS_MBCSV_CB_DEC *dec) 
{
   AVD_COMP comp;
   AVD_COMP *comp_struct {};
+  SaNameT proxy_name;
 
   TRACE_ENTER();
 
@@ -1825,7 +1856,8 @@
    * Action in this case is just to update.
    */
   osaf_decode_sanamet(&dec->i_uba, &comp.comp_info.name);
-  osaf_decode_sanamet(&dec->i_uba, &comp.saAmfCompCurrProxyName);
+  osaf_decode_sanamet(&dec->i_uba, &proxy_name);
+  comp.saAmfCompCurrProxyName = Amf::to_string(&proxy_name);
 
   if (nullptr == (comp_struct = 
comp_db->find(Amf::to_string(&comp.comp_info.name))))
     osafassert(0);
@@ -1877,12 +1909,14 @@
    */
   osaf_decode_sanamet(&dec->i_uba, &comp.comp_info.name);
   osaf_decode_uint32(&dec->i_uba, reinterpret_cast<uint32_t*> 
(&comp.saAmfCompOperState));
-
-  comp_struct = avd_comp_get_or_create(&comp.comp_info.name);
+  
+  const std::string comp_name = Amf::to_string(&comp.comp_info.name);
+
+  comp_struct = avd_comp_get_or_create(comp_name);
 
   /* Update the fields received in this checkpoint message */
   comp_struct->saAmfCompOperState = comp.saAmfCompOperState;
-  avd_saImmOiRtObjectUpdate(&comp_struct->comp_info.name, "saAmfCompOperState",
+  avd_saImmOiRtObjectUpdate(comp_name, "saAmfCompOperState",
                             SA_IMM_ATTR_SAUINT32T, 
&comp_struct->saAmfCompOperState);
 
   cb->async_updt_cnt.comp_updt++;
@@ -1917,8 +1951,10 @@
   osaf_decode_sanamet(&dec->i_uba, &comp.comp_info.name);
   osaf_decode_uint32(&dec->i_uba, reinterpret_cast<uint32_t*> 
(&comp.saAmfCompReadinessState));
 
-  if (nullptr == (comp_struct = 
comp_db->find(Amf::to_string(&comp.comp_info.name)))) {
-    LOG_ER("%s: comp not found, %s", __FUNCTION__, comp.comp_info.name.value);
+  const std::string comp_name(Amf::to_string(&comp.comp_info.name));
+
+  if (nullptr == (comp_struct = comp_db->find(comp_name))) {
+    LOG_ER("%s: comp not found, %s", __FUNCTION__, comp_name.c_str());
     return NCSCC_RC_FAILURE;
   }
 
@@ -1927,7 +1963,7 @@
 
   cb->async_updt_cnt.comp_updt++;
 
-  avd_saImmOiRtObjectUpdate(&comp_struct->comp_info.name, 
"saAmfCompReadinessState",
+  avd_saImmOiRtObjectUpdate(comp_name, "saAmfCompReadinessState",
                             SA_IMM_ATTR_SAUINT32T, 
&comp_struct->saAmfCompReadinessState);
   TRACE_LEAVE();
   return NCSCC_RC_SUCCESS;
@@ -1968,7 +2004,7 @@
   comp_struct->saAmfCompPresenceState = comp.saAmfCompPresenceState;
 
   cb->async_updt_cnt.comp_updt++;
-  avd_saImmOiRtObjectUpdate(&comp_struct->comp_info.name, 
"saAmfCompPresenceState",
+  avd_saImmOiRtObjectUpdate(Amf::to_string(&comp_struct->comp_info.name), 
"saAmfCompPresenceState",
                             SA_IMM_ATTR_SAUINT32T, 
&comp_struct->saAmfCompPresenceState);
 
   TRACE_LEAVE();
@@ -2693,7 +2729,9 @@
 
 void decode_comp_cs_type_config(NCS_UBAID *ub, AVD_COMPCS_TYPE* comp_cs_type)
 {
-       osaf_decode_sanamet(ub, &comp_cs_type->name);
+       SaNameT name;
+       osaf_decode_sanamet(ub, &name);
+       comp_cs_type->name = Amf::to_string(&name);
        osaf_decode_uint32(ub, &comp_cs_type->saAmfCompNumCurrActiveCSIs);
        osaf_decode_uint32(ub, &comp_cs_type->saAmfCompNumCurrStandbyCSIs);
 }
@@ -2796,7 +2834,7 @@
        osaf_decode_uint32(&dec->i_uba, (uint32_t*)&ng->saAmfNGAdminState);
        cb->async_updt_cnt.ng_updt++;
 
-       TRACE("'%s',saAmfNGAdminState:%d",ng->name.value, 
ng->saAmfNGAdminState);
+       TRACE("'%s',saAmfNGAdminState:%d",ng->name.c_str(), 
ng->saAmfNGAdminState);
        ng->admin_ng_pend_cbk.invocation = 0;
        ng->admin_ng_pend_cbk.admin_oper = 
static_cast<SaAmfAdminOperationIdT>(0);
        for (std::set<std::string>::const_iterator iter = 
ng->saAmfNGNodeList.begin();
diff --git a/osaf/services/saf/amf/amfd/ckpt_enc.cc 
b/osaf/services/saf/amf/amfd/ckpt_enc.cc
--- a/osaf/services/saf/amf/amfd/ckpt_enc.cc
+++ b/osaf/services/saf/amf/amfd/ckpt_enc.cc
@@ -281,7 +281,7 @@
 {
        osaf_encode_uint32(ub, avnd->node_info.nodeId);
        osaf_encode_saclmnodeaddresst(ub, &avnd->node_info.nodeAddress);
-       osaf_encode_sanamet(ub, &avnd->name);
+       osaf_encode_sanamet_o2(ub, avnd->name.c_str());
        osaf_encode_bool(ub, avnd->node_info.member);
        osaf_encode_satimet(ub, avnd->node_info.bootTimestamp);
        osaf_encode_uint64(ub, avnd->node_info.initialViewNumber);
@@ -326,7 +326,7 @@
                break;
        case NCS_MBCSV_ACT_RMV:
                /* Send only key information */
-               osaf_encode_sanamet(&enc->io_uba, &avnd->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, avnd->name.c_str());
                break;
        default:
                osafassert(0);
@@ -338,7 +338,7 @@
 
 void encode_app(NCS_UBAID *ub, const AVD_APP *app)
 {
-       osaf_encode_sanamet(ub, &app->name);
+       osaf_encode_sanamet_o2(ub, app->name.c_str());
        osaf_encode_uint32(ub, app->saAmfApplicationAdminState);
        osaf_encode_uint32(ub, app->saAmfApplicationCurrNumSGs);
 }
@@ -374,7 +374,7 @@
                encode_app(&enc->io_uba, app);  
                break;
        case NCS_MBCSV_ACT_RMV:
-               osaf_encode_sanamet(&enc->io_uba, &app->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, app->name.c_str());
                break;
        default:
                osafassert(0);
@@ -386,7 +386,7 @@
 
 static void encode_sg(NCS_UBAID *ub, const AVD_SG *sg)
 {
-       osaf_encode_sanamet(ub, &sg->name);
+       osaf_encode_sanamet_o2(ub, sg->name.c_str());
        osaf_encode_uint32(ub, sg->saAmfSGAdminState);
        osaf_encode_uint32(ub, sg->saAmfSGNumCurrAssignedSUs);
        osaf_encode_uint32(ub, sg->saAmfSGNumCurrInstantiatedSpareSUs);
@@ -420,7 +420,7 @@
                encode_sg(&enc->io_uba, sg);
                break;
        case NCS_MBCSV_ACT_RMV:
-               osaf_encode_sanamet(&enc->io_uba, &sg->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, sg->name.c_str());
                break;
        default:
                osafassert(0);
@@ -432,13 +432,13 @@
 
 static void encode_su(NCS_UBAID *ub, AVD_SU *su, uint16_t peer_version)
 {
-       osaf_encode_sanamet(ub, &su->name);
+       osaf_encode_sanamet_o2(ub, su->name.c_str());
        osaf_encode_bool(ub, (bool)su->saAmfSUPreInstantiable); // TODO(hafe) 
change to bool
        osaf_encode_uint32(ub, su->saAmfSUOperState);
        osaf_encode_uint32(ub, su->saAmfSUAdminState);
        osaf_encode_uint32(ub, su->saAmfSuReadinessState);
        osaf_encode_uint32(ub, su->saAmfSUPresenceState);
-       osaf_encode_sanamet(ub, &su->saAmfSUHostedByNode);
+       osaf_encode_sanamet_o2(ub, su->saAmfSUHostedByNode.c_str());
        osaf_encode_uint32(ub, su->saAmfSUNumCurrActiveSIs);
        osaf_encode_uint32(ub, su->saAmfSUNumCurrStandbySIs);
        osaf_encode_uint32(ub, su->saAmfSURestartCount);
@@ -474,7 +474,7 @@
                break;
        case NCS_MBCSV_ACT_RMV: {
                const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-               osaf_encode_sanamet(&enc->io_uba, &su->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
                break;
        }
        default:
@@ -499,13 +499,13 @@
 #endif
        TRACE_ENTER2("my_version: %u, to_version: %u", ver_compare, 
peer_version);
 
-       osaf_encode_sanamet(ub, &si->name);
+       osaf_encode_sanamet_o2(ub, si->name.c_str());
        osaf_encode_uint32(ub, si->saAmfSIAdminState);
        osaf_encode_uint32(ub, si->saAmfSIAssignmentState);
        osaf_encode_uint32(ub, si->saAmfSINumCurrActiveAssignments);
        osaf_encode_uint32(ub, si->saAmfSINumCurrStandbyAssignments);
        osaf_encode_uint32(ub, si->si_switch);
-       osaf_encode_sanamet(ub, &si->saAmfSIProtectedbySG);
+       osaf_encode_sanamet_o2(ub, si->saAmfSIProtectedbySG.c_str());
        osaf_encode_bool(ub, si->alarm_sent);
        
        if (peer_version >= ver_compare) {
@@ -548,7 +548,7 @@
                break;
        case NCS_MBCSV_ACT_RMV:
                /* Send only key information */
-               osaf_encode_sanamet(&enc->io_uba, &si->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
                break;
        default:
                osafassert(0);
@@ -590,7 +590,7 @@
                /*
                 * Send SI key.
                 */
-               osaf_encode_sanamet(&enc->io_uba, &si->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
                break;
 
        case NCS_MBCSV_ACT_UPDATE:
@@ -610,12 +610,13 @@
        const AVD_SG *sg,
        const uint16_t peer_version)
 {
-       osaf_encode_sanamet(ub, &sg->name);
-       osaf_encode_sanamet(ub, &sg->si_tobe_redistributed->name);
-       osaf_encode_sanamet(ub, &sg->min_assigned_su->name);
-       osaf_encode_sanamet(ub, &sg->max_assigned_su->name);
+       osaf_encode_sanamet_o2(ub, sg->name.c_str());
+       osaf_encode_sanamet_o2(ub, sg->si_tobe_redistributed->name.c_str());
+       osaf_encode_sanamet_o2(ub, sg->min_assigned_su->name.c_str());
+       osaf_encode_sanamet_o2(ub, sg->max_assigned_su->name.c_str());
 }
 
+
 /*********************************************************************
  * @brief encodes si transfer parameters
  * @param[in] cb
@@ -634,7 +635,7 @@
 
        case NCS_MBCSV_ACT_RMV:
                /* Send only key information */
-               osaf_encode_sanamet(&enc->io_uba, &sg->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, sg->name.c_str());
                break;
 
        default:
@@ -649,14 +650,14 @@
        const AVD_SU_SI_REL *susi,
        const uint16_t peer_version)
 {
-       osaf_encode_sanamet(ub, &susi->su->name);
-       osaf_encode_sanamet(ub, &susi->si->name);
+       osaf_encode_sanamet_o2(ub, susi->su->name.c_str());
+       osaf_encode_sanamet_o2(ub, susi->si->name.c_str());
        osaf_encode_uint32(ub, susi->state);
        osaf_encode_uint32(ub, susi->fsm);
        if (peer_version >= AVD_MBCSV_SUB_PART_VERSION_3) {
                osaf_encode_bool(ub, static_cast<bool>(susi->csi_add_rem));
-               osaf_encode_sanamet(ub, &susi->comp_name);
-               osaf_encode_sanamet(ub, &susi->csi_name);
+               osaf_encode_sanamet_o2(ub, susi->comp_name.c_str());
+               osaf_encode_sanamet_o2(ub, susi->csi_name.c_str());
        };
 }
 
@@ -725,7 +726,7 @@
   osaf_encode_uint32(ub, comp->saAmfCompReadinessState);
   osaf_encode_uint32(ub, comp->saAmfCompPresenceState);
   osaf_encode_uint32(ub, comp->saAmfCompRestartCount);
-  osaf_encode_sanamet(ub, &comp->saAmfCompCurrProxyName);
+  osaf_encode_sanamet_o2(ub, comp->saAmfCompCurrProxyName.c_str());
 }
 
 /****************************************************************************\
@@ -790,7 +791,7 @@
        switch (enc->io_action) {
        case NCS_MBCSV_ACT_ADD:
        case NCS_MBCSV_ACT_RMV:
-               osaf_encode_sanamet(&enc->io_uba, &su->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
                break;
        case NCS_MBCSV_ACT_UPDATE:
        default:
@@ -864,7 +865,7 @@
         * error. Call EDU encode to encode this field.
         */
        if (NCS_MBCSV_ACT_UPDATE == enc->io_action) {
-               osaf_encode_sanamet(&enc->io_uba, &avnd->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, avnd->name.c_str());
                osaf_encode_uint32(&enc->io_uba, avnd->saAmfNodeAdminState);
        } else
                osafassert(0);
@@ -898,7 +899,7 @@
         * error. Call EDU encode to encode this field.
         */
        if (NCS_MBCSV_ACT_UPDATE == enc->io_action) {
-               osaf_encode_sanamet(&enc->io_uba, &avnd->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, avnd->name.c_str());
                osaf_encode_uint32(&enc->io_uba, avnd->saAmfNodeOperState);
        } else
                osafassert(0);
@@ -932,7 +933,7 @@
         * error. Call EDU encode to encode this field.
         */
        if (NCS_MBCSV_ACT_UPDATE == enc->io_action) {
-               osaf_encode_sanamet(&enc->io_uba, &avnd->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, avnd->name.c_str());
                osaf_encode_uint32(&enc->io_uba, avnd->node_state);
        } else
                osafassert(0);
@@ -1028,7 +1029,7 @@
 
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SG *sg = (AVD_SG *)(NCS_INT64_TO_PTR_CAST(enc->io_reo_hdl));
-       osaf_encode_sanamet(&enc->io_uba, &sg->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, sg->name.c_str());
        osaf_encode_uint32(&enc->io_uba, sg->saAmfSGAdminState);
 
        TRACE_LEAVE();
@@ -1054,7 +1055,7 @@
 
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SG *sg = (AVD_SG *)(NCS_INT64_TO_PTR_CAST(enc->io_reo_hdl));
-       osaf_encode_sanamet(&enc->io_uba, &sg->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, sg->name.c_str());
        osaf_encode_uint32(&enc->io_uba, sg->saAmfSGNumCurrAssignedSUs);
 
        TRACE_LEAVE();
@@ -1080,7 +1081,7 @@
 
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SG *sg = (AVD_SG *)(NCS_INT64_TO_PTR_CAST(enc->io_reo_hdl));
-       osaf_encode_sanamet(&enc->io_uba, &sg->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, sg->name.c_str());
        osaf_encode_uint32(&enc->io_uba, 
sg->saAmfSGNumCurrInstantiatedSpareSUs);
 
        TRACE_LEAVE();
@@ -1106,7 +1107,7 @@
 
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SG *sg = (AVD_SG *)(NCS_INT64_TO_PTR_CAST(enc->io_reo_hdl));
-       osaf_encode_sanamet(&enc->io_uba, &sg->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, sg->name.c_str());
        osaf_encode_uint32(&enc->io_uba, 
sg->saAmfSGNumCurrNonInstantiatedSpareSUs);
 
        TRACE_LEAVE();
@@ -1132,7 +1133,7 @@
 
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SG *sg = (AVD_SG *)(NCS_INT64_TO_PTR_CAST(enc->io_reo_hdl));
-       osaf_encode_sanamet(&enc->io_uba, &sg->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, sg->name.c_str());
        osaf_encode_uint32(&enc->io_uba, sg->adjust_state);
 
        TRACE_LEAVE();
@@ -1158,7 +1159,7 @@
 
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SG *sg = (AVD_SG *)(NCS_INT64_TO_PTR_CAST(enc->io_reo_hdl));
-       osaf_encode_sanamet(&enc->io_uba, &sg->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, sg->name.c_str());
        osaf_encode_uint32(&enc->io_uba, sg->sg_fsm_state);
 
        TRACE_LEAVE();
@@ -1183,7 +1184,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_bool(&enc->io_uba, su->saAmfSUPreInstantiable);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1207,7 +1208,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->saAmfSUOperState);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1231,7 +1232,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->saAmfSUAdminState);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1255,7 +1256,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->saAmfSuReadinessState);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1279,7 +1280,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->saAmfSUPresenceState);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1303,7 +1304,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->saAmfSUNumCurrActiveSIs);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1327,7 +1328,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->saAmfSUNumCurrStandbySIs);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1351,7 +1352,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_bool(&enc->io_uba, su->term_state);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1375,7 +1376,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->su_switch);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1399,7 +1400,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->su_act_state);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1423,7 +1424,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SU *su = (AVD_SU *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &su->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str());
        osaf_encode_uint32(&enc->io_uba, su->saAmfSURestartCount);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1448,7 +1449,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SI *si = (AVD_SI *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &si->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
        osaf_encode_uint32(&enc->io_uba, si->saAmfSIAdminState);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1473,7 +1474,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SI *si = (AVD_SI *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &si->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
        osaf_encode_uint32(&enc->io_uba, si->saAmfSIAssignmentState);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1492,7 +1493,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SI *si = (AVD_SI *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &si->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
        osaf_encode_uint32(&enc->io_uba, si->si_dep_state);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1517,7 +1518,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SI *si = (AVD_SI *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &si->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
        osaf_encode_uint32(&enc->io_uba, si->saAmfSINumCurrActiveAssignments);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1542,7 +1543,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SI *si = (AVD_SI *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &si->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
        osaf_encode_uint32(&enc->io_uba, si->saAmfSINumCurrStandbyAssignments);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1567,7 +1568,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SI *si = (AVD_SI *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &si->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
        osaf_encode_uint32(&enc->io_uba, si->si_switch);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1592,7 +1593,7 @@
        TRACE_ENTER();
        osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
        const AVD_SI *si = (AVD_SI *)enc->io_reo_hdl;
-       osaf_encode_sanamet(&enc->io_uba, &si->name);
+       osaf_encode_sanamet_o2(&enc->io_uba, si->name.c_str());
        osaf_encode_bool(&enc->io_uba, si->alarm_sent);
        TRACE_LEAVE();
        return NCSCC_RC_SUCCESS;
@@ -1622,7 +1623,7 @@
    */
   if (NCS_MBCSV_ACT_UPDATE == enc->io_action) {
     osaf_encode_sanamet(&enc->io_uba, &comp->comp_info.name);
-    osaf_encode_sanamet(&enc->io_uba, &comp->saAmfCompCurrProxyName);
+    osaf_encode_sanamet_o2(&enc->io_uba, comp->saAmfCompCurrProxyName.c_str());
   } else {
     osafassert(0);
   }
@@ -1771,9 +1772,7 @@
    * error. Call EDU encode to encode this field.
    */
   if (NCS_MBCSV_ACT_UPDATE == enc->io_action) {
-
     osaf_encode_sanamet(&enc->io_uba, &comp->comp_info.name);
-
     osaf_encode_uint32(&enc->io_uba, comp->saAmfCompRestartCount);
   } else {
     osafassert(0);
@@ -2065,7 +2064,6 @@
 static uint32_t enc_cs_si_config(AVD_CL_CB *cb, NCS_MBCSV_CB_ENC *enc, 
uint32_t *num_of_obj)
 {
        uint32_t status = NCSCC_RC_SUCCESS;
-       SaNameT si_name;
        TRACE_ENTER();
 
        /* 
@@ -2076,7 +2074,6 @@
                const AVD_SI *si = it->second;
                encode_si(cb, &enc->io_uba, si, enc->i_peer_version);
 
-               si_name = si->name;
                (*num_of_obj)++;
        }
 
@@ -2113,7 +2110,7 @@
                status = enc_su_oper_list(cb, sg, enc);
 
                if (status != NCSCC_RC_SUCCESS) {
-                       LOG_ER("%s: encode failed, %s", __FUNCTION__, 
sg->name.value);
+                       LOG_ER("%s: encode failed, %s", __FUNCTION__, 
sg->name.c_str());
                        return NCSCC_RC_FAILURE;
                }
 
@@ -2153,7 +2150,7 @@
                if (nullptr == sg->admin_si) 
                        continue;
 
-               osaf_encode_sanamet(&enc->io_uba, &sg->admin_si->name);
+               osaf_encode_sanamet_o2(&enc->io_uba, 
sg->admin_si->name.c_str());
 
                (*num_of_obj)++;
        }
@@ -2379,7 +2376,7 @@
         * Now walk through the entire SU operation list and encode it.
         */
        for (const auto& su : sg->su_oper_list) {
-               osaf_encode_sanamet(&enc->io_uba, &(su->name)); 
+               osaf_encode_sanamet_o2(&enc->io_uba, su->name.c_str()); 
                num_of_opr_su++;
        }
 
@@ -2398,7 +2395,7 @@
        const AVD_COMPCS_TYPE *comp_cs_type,
        const uint16_t peer_version)
 {
-       osaf_encode_sanamet(ub, &comp_cs_type->name);
+       osaf_encode_sanamet_o2(ub, comp_cs_type->name.c_str());
        osaf_encode_uint32(ub, comp_cs_type->saAmfCompNumCurrActiveCSIs);
        osaf_encode_uint32(ub, comp_cs_type->saAmfCompNumCurrStandbyCSIs);
 }
@@ -2461,7 +2458,6 @@
 \**************************************************************************/
 static uint32_t enc_cs_comp_cs_type_config(AVD_CL_CB *cb, NCS_MBCSV_CB_ENC 
*enc, uint32_t *num_of_obj)
 {
-       SaNameT dn = {0};
        TRACE_ENTER();
 
        /*
@@ -2472,7 +2468,6 @@
                AVD_COMPCS_TYPE *compcstype = it->second;
                encode_comp_cs_type_config(&enc->io_uba, compcstype, 
enc->i_peer_version);
 
-               dn = compcstype->name;
                (*num_of_obj)++;
        }
 
@@ -2492,7 +2487,7 @@
         TRACE_ENTER();
         osafassert(NCS_MBCSV_ACT_UPDATE == enc->io_action);
         const AVD_AMF_NG *ng = (AVD_AMF_NG *)enc->io_reo_hdl;
-        osaf_encode_sanamet(&enc->io_uba, &ng->name);
+        osaf_encode_sanamet_o2(&enc->io_uba, ng->name.c_str());
         osaf_encode_uint32(&enc->io_uba, ng->saAmfNGAdminState);
         TRACE_LEAVE();
         return NCSCC_RC_SUCCESS;
diff --git a/osaf/services/saf/amf/amfd/ckpt_updt.cc 
b/osaf/services/saf/amf/amfd/ckpt_updt.cc
--- a/osaf/services/saf/amf/amfd/ckpt_updt.cc
+++ b/osaf/services/saf/amf/amfd/ckpt_updt.cc
@@ -49,12 +49,12 @@
        uint32_t rc = NCSCC_RC_SUCCESS;
        AVD_AVND *node;
 
-       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_node->name.value);
+       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_node->name.c_str());
 
        osafassert (action == NCS_MBCSV_ACT_UPDATE);
 
-       if (nullptr == (node = avd_node_get(&ckpt_node->name))) {
-               LOG_ER("avd_node_get FAILED for '%s'", ckpt_node->name.value);
+       if (nullptr == (node = avd_node_get(ckpt_node->name))) {
+               LOG_ER("avd_node_get FAILED for '%s'", ckpt_node->name.c_str());
                rc = NCSCC_RC_FAILURE;
                goto done;
        }
@@ -98,15 +98,15 @@
 {
        uint32_t rc = NCSCC_RC_SUCCESS;
 
-       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_app->name.value);
+       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_app->name.c_str());
 
        osafassert (action == NCS_MBCSV_ACT_UPDATE);
 
-       AVD_APP *app = app_db->find(Amf::to_string(&ckpt_app->name));
+       AVD_APP *app = app_db->find(ckpt_app->name);
        if (app == nullptr) {
-               TRACE("'%s' does not exist, creating it", ckpt_app->name.value);
-               app = new AVD_APP(&ckpt_app->name);
-               app_db->insert(Amf::to_string(&app->name), app);
+               TRACE("'%s' does not exist, creating it", 
ckpt_app->name.c_str());
+               app = new AVD_APP(ckpt_app->name);
+               app_db->insert(app->name, app);
                osafassert(rc == NCSCC_RC_SUCCESS);
                goto done;
        }
@@ -140,12 +140,12 @@
        uint32_t rc = NCSCC_RC_SUCCESS;
        AVD_SG *sg;
 
-       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_sg->name.value);
+       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_sg->name.c_str());
 
        osafassert (action == NCS_MBCSV_ACT_UPDATE);
 
-       if (nullptr == (sg = sg_db->find(Amf::to_string(&ckpt_sg->name)))) {
-               LOG_ER("sg_db->find() FAILED for '%s'", ckpt_sg->name.value);
+       if (nullptr == (sg = sg_db->find(ckpt_sg->name))) {
+               LOG_ER("sg_db->find() FAILED for '%s'", ckpt_sg->name.c_str());
                rc = NCSCC_RC_FAILURE;
                goto done;
        }
@@ -183,12 +183,12 @@
        uint32_t rc = NCSCC_RC_SUCCESS;
        AVD_SU *su;
 
-       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_su->name.value);
+       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_su->name.c_str());
 
        osafassert (action == NCS_MBCSV_ACT_UPDATE);
 
-       if (nullptr == (su = su_db->find(Amf::to_string(&ckpt_su->name)))) {
-               LOG_ER("su_db->find FAILED for '%s'", ckpt_su->name.value);
+       if (nullptr == (su = su_db->find(ckpt_su->name))) {
+               LOG_ER("su_db->find FAILED for '%s'", ckpt_su->name.c_str());
                rc = NCSCC_RC_FAILURE;
                goto done;
        }
@@ -200,18 +200,18 @@
        su->saAmfSUPresenceState = ckpt_su->saAmfSUPresenceState;
        su->saAmfSUNumCurrActiveSIs = ckpt_su->saAmfSUNumCurrActiveSIs;
        su->saAmfSUNumCurrStandbySIs = ckpt_su->saAmfSUNumCurrStandbySIs;
-       memcpy(&su->saAmfSUHostedByNode, &ckpt_su->saAmfSUHostedByNode, 
sizeof(SaNameT));
+       su->saAmfSUHostedByNode = ckpt_su->saAmfSUHostedByNode;
        su->term_state = ckpt_su->term_state;
        su->su_switch = ckpt_su->su_switch;
        su->saAmfSURestartCount = ckpt_su->saAmfSURestartCount;
 
-       avd_saImmOiRtObjectUpdate(&su->name, "saAmfSUOperState",
+       avd_saImmOiRtObjectUpdate(su->name, "saAmfSUOperState",
                        SA_IMM_ATTR_SAUINT32T, &su->saAmfSUOperState);
-       avd_saImmOiRtObjectUpdate(&su->name, "saAmfSUAdminState",
+       avd_saImmOiRtObjectUpdate(su->name, "saAmfSUAdminState",
                        SA_IMM_ATTR_SAUINT32T, &su->saAmfSUAdminState);
-       avd_saImmOiRtObjectUpdate(&su->name, "saAmfSUReadinessState",
+       avd_saImmOiRtObjectUpdate(su->name, "saAmfSUReadinessState",
                        SA_IMM_ATTR_SAUINT32T, &su->saAmfSuReadinessState);
-       avd_saImmOiRtObjectUpdate(&su->name, "saAmfSUPresenceState",
+       avd_saImmOiRtObjectUpdate(su->name, "saAmfSUPresenceState",
                        SA_IMM_ATTR_SAUINT32T, &su->saAmfSUPresenceState);
 done:
        TRACE_LEAVE2("%u", rc);
@@ -238,11 +238,11 @@
        uint32_t rc = NCSCC_RC_FAILURE;
        AVD_SI *si;
 
-       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_si->name.value);
+       TRACE_ENTER2("%s - '%s'", action_name[action], ckpt_si->name.c_str());
 
        osafassert (action == NCS_MBCSV_ACT_UPDATE);
 
-       if (nullptr == (si = avd_si_get(&ckpt_si->name))) {
+       if (nullptr == (si = avd_si_get(ckpt_si->name))) {
                LOG_WA("%s: avd_si_get FAILED", __FUNCTION__);
                goto done;
        }
@@ -254,7 +254,7 @@
        si->saAmfSIAssignmentState = ckpt_si->saAmfSIAssignmentState;
        si->saAmfSIProtectedbySG = ckpt_si->saAmfSIProtectedbySG;
        si->alarm_sent = ckpt_si->alarm_sent;
-       si->sg_of_si = sg_db->find(Amf::to_string(&si->saAmfSIProtectedbySG));
+       si->sg_of_si = sg_db->find(si->saAmfSIProtectedbySG);
 
        rc = NCSCC_RC_SUCCESS;
 done:
@@ -351,7 +351,7 @@
 
        switch (action) {
        case NCS_MBCSV_ACT_ADD:
-               sg_ptr->si_tobe_redistributed = 
avd_si_get(&si_trans_ckpt->si_name); 
+               sg_ptr->si_tobe_redistributed = 
avd_si_get(Amf::to_string(&si_trans_ckpt->si_name)); 
                sg_ptr->min_assigned_su = 
su_db->find(Amf::to_string(&si_trans_ckpt->min_su_name)); 
                sg_ptr->max_assigned_su = 
su_db->find(Amf::to_string(&si_trans_ckpt->max_su_name)); 
                break;
@@ -399,11 +399,11 @@
 
        TRACE_ENTER2("'%s' '%s'", su_si_ckpt->si_name.value, 
su_si_ckpt->su_name.value);
 
-       su_si_rel_ptr = avd_susi_find(cb, &su_si_ckpt->su_name, 
&su_si_ckpt->si_name);
+       su_si_rel_ptr = avd_susi_find(cb, Amf::to_string(&su_si_ckpt->su_name), 
Amf::to_string(&su_si_ckpt->si_name));
 
        su_ptr = su_db->find(Amf::to_string(&su_si_ckpt->su_name));
        osafassert(su_ptr);
-       si_ptr_up = avd_si_get(&su_si_ckpt->si_name);
+       si_ptr_up = avd_si_get(Amf::to_string(&su_si_ckpt->si_name));
        osafassert(si_ptr_up);
 
        /* Since csi_add_rem flag is not memset in older versions, make sure it 
is set here. */
@@ -431,29 +431,29 @@
                        su_si_rel_ptr->state = su_si_ckpt->state;
                        su_si_rel_ptr->csi_add_rem = su_si_ckpt->csi_add_rem;
                        if (su_si_rel_ptr->csi_add_rem) {
-                               su_si_rel_ptr->comp_name = 
su_si_ckpt->comp_name;
-                               su_si_rel_ptr->csi_name = su_si_ckpt->csi_name;
-                               TRACE("compcsi create for '%s' '%s'", 
su_si_rel_ptr->comp_name.value, su_si_rel_ptr->csi_name.value);
-                               if ((comp_ptr = 
comp_db->find(Amf::to_string(&(su_si_rel_ptr->comp_name)))) == nullptr) {
-                                       LOG_ER("comp_db->find() FAILED for 
'%s'", su_si_rel_ptr->comp_name.value);
+                               su_si_rel_ptr->comp_name = 
Amf::to_string(&su_si_ckpt->comp_name);
+                               su_si_rel_ptr->csi_name = 
Amf::to_string(&su_si_ckpt->csi_name);
+                               TRACE("compcsi create for '%s' '%s'", 
su_si_rel_ptr->comp_name.c_str(), su_si_rel_ptr->csi_name.c_str());
+                               if ((comp_ptr = 
comp_db->find(su_si_rel_ptr->comp_name)) == nullptr) {
+                                       LOG_ER("comp_db->find() FAILED for 
'%s'", su_si_rel_ptr->comp_name.c_str());
                                        return NCSCC_RC_FAILURE;
                                }
-                               if ((csi_ptr = 
csi_db->find(Amf::to_string(&(su_si_rel_ptr->csi_name)))) == nullptr) {
+                               if ((csi_ptr = 
csi_db->find(su_si_rel_ptr->csi_name)) == nullptr) {
                                        /* This condition will arise if there 
is some delay in the ccb apply callback
                                         * So create csi and add it to the 
csi_db here, later in the ccb apply callback
                                         * attributes will be updated and csi 
will be added to the model
                                         */
-                                       csi_ptr = 
csi_create(&su_si_rel_ptr->csi_name);
+                                       csi_ptr = 
csi_create(su_si_rel_ptr->csi_name);
                                        osafassert(csi_ptr);
                                }
                                if ((avd_compcsi_create(su_si_rel_ptr, csi_ptr, 
comp_ptr, false)) == nullptr) {
                                        LOG_ER("avd_compcsi_create FAILED for 
csi '%s' comp '%s'",
-                                               
su_si_rel_ptr->csi_name.value,su_si_rel_ptr->comp_name.value);
+                                               
su_si_rel_ptr->csi_name.c_str(),su_si_rel_ptr->comp_name.c_str());
                                        return NCSCC_RC_FAILURE;
                                }       
                        } else {
-                               
memset(&(su_si_rel_ptr->comp_name),0,sizeof(SaNameT));
-                               
memset(&(su_si_rel_ptr->csi_name),0,sizeof(SaNameT));
+                               su_si_rel_ptr->comp_name = "";
+                               su_si_rel_ptr->csi_name = "";
                        }
                } else {
                        LOG_ER("%s:%u", __FUNCTION__, __LINE__);
@@ -532,14 +532,14 @@
 {
        uint32_t rc = NCSCC_RC_FAILURE;
        AVD_COMP *comp;
-       const SaNameT *dn = &ckpt_comp->comp_info.name;
+       const std::string& dn = Amf::to_string(&ckpt_comp->comp_info.name);
 
-       TRACE_ENTER2("%s - '%s'", action_name[action], dn->value);
+       TRACE_ENTER2("%s - '%s'", action_name[action], dn.c_str());
 
-       osafassert (action == NCS_MBCSV_ACT_UPDATE);
+       osafassert(action == NCS_MBCSV_ACT_UPDATE);
 
-       if (nullptr == (comp = comp_db->find(Amf::to_string(dn)))) {
-               LOG_ER("comp_db->find() FAILED for '%s'", dn->value);
+       if (nullptr == (comp = comp_db->find(dn))) {
+               LOG_ER("comp_db->find() FAILED for '%s'", dn.c_str());
                goto done;
        }
        comp->saAmfCompOperState = ckpt_comp->saAmfCompOperState;
@@ -548,11 +548,11 @@
        comp->saAmfCompReadinessState = ckpt_comp->saAmfCompReadinessState;
        /* SaNameT struct copy */
        comp->saAmfCompCurrProxyName = ckpt_comp->saAmfCompCurrProxyName;
-       avd_saImmOiRtObjectUpdate(&comp->comp_info.name, "saAmfCompOperState",
+       avd_saImmOiRtObjectUpdate(dn, "saAmfCompOperState",
                        SA_IMM_ATTR_SAUINT32T, &comp->saAmfCompOperState);
-       avd_saImmOiRtObjectUpdate(&comp->comp_info.name, 
"saAmfCompReadinessState",
+       avd_saImmOiRtObjectUpdate(dn, "saAmfCompReadinessState",
                        SA_IMM_ATTR_SAUINT32T, &comp->saAmfCompReadinessState);
-       avd_saImmOiRtObjectUpdate(&comp->comp_info.name, 
"saAmfCompPresenceState",
+       avd_saImmOiRtObjectUpdate(dn, "saAmfCompPresenceState",
                        SA_IMM_ATTR_SAUINT32T, &comp->saAmfCompPresenceState);
 
        rc = NCSCC_RC_SUCCESS;
@@ -581,14 +581,14 @@
 {
        uint32_t rc = NCSCC_RC_FAILURE;
        AVD_COMPCS_TYPE *ccst;
-       const SaNameT *dn = &ckpt_compcstype->name;
+       const std::string& dn = ckpt_compcstype->name;
 
-       TRACE_ENTER2("%s - '%s'", action_name[action], dn->value);
+       TRACE_ENTER2("%s - '%s'", action_name[action], dn.c_str());
 
        osafassert (action == NCS_MBCSV_ACT_UPDATE);
 
-       if (nullptr == (ccst = compcstype_db->find(Amf::to_string(dn)))) {
-               LOG_ER("compcstype_db->find()FAILED for '%s'", dn->value);
+       if (nullptr == (ccst = compcstype_db->find(dn))) {
+               LOG_ER("compcstype_db->find()FAILED for '%s'", dn.c_str());
                goto done;
        }
        ccst->saAmfCompNumCurrActiveCSIs = 
ckpt_compcstype->saAmfCompNumCurrActiveCSIs;


------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=267308311&iu=/4140
_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to