osaf/libs/agents/saf/imma/imma_db.cc | 7 +- osaf/libs/agents/saf/imma/imma_init.cc | 4 +- osaf/libs/agents/saf/imma/imma_mds.cc | 4 +- osaf/libs/agents/saf/imma/imma_oi_api.cc | 34 +++--- osaf/libs/agents/saf/imma/imma_om_api.cc | 144 +++++++++++++++--------------- osaf/libs/agents/saf/imma/imma_proc.cc | 96 ++++++++++---------- osaf/libs/agents/saf/imma/imma_proc.h | 8 +- 7 files changed, 147 insertions(+), 150 deletions(-)
Fix "invalid conversion" errors. diff --git a/osaf/libs/agents/saf/imma/imma_db.cc b/osaf/libs/agents/saf/imma/imma_db.cc --- a/osaf/libs/agents/saf/imma/imma_db.cc +++ b/osaf/libs/agents/saf/imma/imma_db.cc @@ -227,7 +227,7 @@ void imma_oi_ccb_record_add(IMMA_CLIENT_ return; } - new_ccb = calloc(1, sizeof(struct imma_oi_ccb_record)); + new_ccb = (imma_oi_ccb_record *) calloc(1, sizeof(imma_oi_ccb_record)); new_ccb->ccbId = ccbId; if(!inv) {/* zero inv =>PBE or Applier => count ops. */ @@ -1206,7 +1206,7 @@ uint32_t imma_search_node_delete(IMMA_CB } if (search_node->mLastAttributes) { - imma_freeSearchAttrs(search_node->mLastAttributes); + imma_freeSearchAttrs((SaImmAttrValuesT_2 **) search_node->mLastAttributes); search_node->mLastAttributes = NULL; } @@ -1354,7 +1354,7 @@ SaStringT* imma_getErrorStrings(IMMSV_SA errStrs = errStrs->next; } - errStringArr = calloc(listSize + 1, sizeof(SaStringT)); + errStringArr = (SaStringT *) calloc(listSize + 1, sizeof(SaStringT)); errStrs = errRsp->errStrings; listSize = 0; @@ -1393,7 +1393,6 @@ void imma_freeSearchAttrs(SaImmAttrValue free(att->attrValues); /*free-4 */ att->attrValues = NULL; att->attrValuesNumber = 0; - att->attrValueType = 0; free(att); /*free-2 */ attr[ix] = NULL; diff --git a/osaf/libs/agents/saf/imma/imma_init.cc b/osaf/libs/agents/saf/imma/imma_init.cc --- a/osaf/libs/agents/saf/imma/imma_init.cc +++ b/osaf/libs/agents/saf/imma/imma_init.cc @@ -184,7 +184,6 @@ static uint32_t imma_create(NCSMDS_SVC_I imma_mds_unregister(cb); mds_reg_fail: - cb->sv_id = 0; imma_db_destroy(cb); db_init_fail: @@ -226,7 +225,6 @@ static uint32_t imma_destroy(NCSMDS_SVC_ /* Destroy the IMMA database */ osafassert(cb->sv_id == sv_id); - cb->sv_id = 0; imma_db_destroy(cb); /* destroy the lock */ @@ -455,7 +453,7 @@ void imma_copyAttrValue(IMMSV_EDU_ATTR_V } p->val.x.size = valueSize; - p->val.x.buf = (valueSize) ? malloc(valueSize) : NULL; + p->val.x.buf = (valueSize) ? (char *) malloc(valueSize) : NULL; if (attrValue && valueSize) { switch (attrValueType) { diff --git a/osaf/libs/agents/saf/imma/imma_mds.cc b/osaf/libs/agents/saf/imma/imma_mds.cc --- a/osaf/libs/agents/saf/imma/imma_mds.cc +++ b/osaf/libs/agents/saf/imma/imma_mds.cc @@ -542,7 +542,7 @@ static uint32_t imma_mds_dec(IMMA_CB *cb } #endif /*ABT DOES NOT WORK */ - evt = calloc(1, sizeof(IMMSV_EVT)); /* calloc zeroes memory */ + evt = (IMMSV_EVT *) calloc(1, sizeof(IMMSV_EVT)); /* calloc zeroes memory */ if (!evt) return NCSCC_RC_FAILURE; @@ -604,7 +604,7 @@ uint32_t imma_mds_msg_sync_send(uint32_t rc = ncsmds_api(&mds_info); if (rc == NCSCC_RC_SUCCESS) - *o_evt = mds_info.info.svc_send.info.sndrsp.o_rsp; + *o_evt = (IMMSV_EVT *) mds_info.info.svc_send.info.sndrsp.o_rsp; return rc; } diff --git a/osaf/libs/agents/saf/imma/imma_oi_api.cc b/osaf/libs/agents/saf/imma/imma_oi_api.cc --- a/osaf/libs/agents/saf/imma/imma_oi_api.cc +++ b/osaf/libs/agents/saf/imma/imma_oi_api.cc @@ -1028,7 +1028,7 @@ static SaAisErrorT admin_op_result_commo for (i = 0; returnParams[i]; ++i) { param = returnParams[i]; /*alloc-a */ - IMMSV_ADMIN_OPERATION_PARAM *p = malloc(sizeof(IMMSV_ADMIN_OPERATION_PARAM)); + IMMSV_ADMIN_OPERATION_PARAM *p = (IMMSV_ADMIN_OPERATION_PARAM *) malloc(sizeof(IMMSV_ADMIN_OPERATION_PARAM)); memset(p, 0, sizeof(IMMSV_ADMIN_OPERATION_PARAM)); TRACE("PARAM:%s \n", param->paramName); @@ -1066,7 +1066,7 @@ static SaAisErrorT admin_op_result_commo goto mds_send_fail; } /*alloc-b */ - p->paramName.buf = malloc(p->paramName.size); + p->paramName.buf = (char *) malloc(p->paramName.size); strncpy(p->paramName.buf, param->paramName, p->paramName.size); p->paramType = param->paramType; @@ -1123,7 +1123,7 @@ static SaAisErrorT admin_op_result_commo free(p->paramName.buf); p->paramName.buf = NULL; } - immsv_evt_free_att_val(&(p->paramBuffer), p->paramType); /*free-c */ + immsv_evt_free_att_val(&(p->paramBuffer), (SaImmValueTypeT) p->paramType); /*free-c */ p->next = NULL; free(p); /*free-a */ } @@ -1418,7 +1418,7 @@ SaAisErrorT saImmOiImplementerSet(SaImmO rc = out_evt->info.imma.info.implSetRsp.error; if (rc == SA_AIS_OK) { cl_node->mImplementerId = out_evt->info.imma.info.implSetRsp.implId; - cl_node->mImplementerName = calloc(1, nameLen); + cl_node->mImplementerName = (char *) calloc(1, nameLen); strncpy(cl_node->mImplementerName, implementerName, nameLen); } } @@ -2563,12 +2563,12 @@ static SaAisErrorT rt_object_update_comm } /*alloc-2 */ - p = calloc(1, sizeof(IMMSV_ATTR_MODS_LIST)); + p = (IMMSV_ATTR_MODS_LIST *) calloc(1, sizeof(IMMSV_ATTR_MODS_LIST)); p->attrModType = attrMod->modType; p->attrValue.attrName.size = strlen(attrMod->modAttr.attrName) + 1; /* alloc 3 */ - p->attrValue.attrName.buf = malloc(p->attrValue.attrName.size); + p->attrValue.attrName.buf = (char *) malloc(p->attrValue.attrName.size); strncpy(p->attrValue.attrName.buf, attrMod->modAttr.attrName, p->attrValue.attrName.size); p->attrValue.attrValuesNumber = attrMod->modAttr.attrValuesNumber; @@ -2584,7 +2584,7 @@ static SaAisErrorT rt_object_update_comm unsigned int i; for (i = 1; i <= numAdded; ++i) { /*alloc-5 */ - IMMSV_EDU_ATTR_VAL_LIST *al = calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); + IMMSV_EDU_ATTR_VAL_LIST *al = (IMMSV_EDU_ATTR_VAL_LIST *) calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); /*alloc-6 */ imma_copyAttrValue(&(al->n), attrMod->modAttr.attrValueType, avarr[i]); al->next = p->attrValue.attrMoreValues; /*NULL initially */ @@ -2692,13 +2692,13 @@ static SaAisErrorT rt_object_update_comm if (p->attrValue.attrValuesNumber) { immsv_evt_free_att_val(&(p->attrValue.attrValue), /*free-4 */ - p->attrValue.attrValueType); + (SaImmValueTypeT) p->attrValue.attrValueType); while (p->attrValue.attrMoreValues) { IMMSV_EDU_ATTR_VAL_LIST *al = p->attrValue.attrMoreValues; p->attrValue.attrMoreValues = al->next; al->next = NULL; - immsv_evt_free_att_val(&(al->n), p->attrValue.attrValueType); /*free-6 */ + immsv_evt_free_att_val(&(al->n), (SaImmValueTypeT) p->attrValue.attrValueType); /*free-6 */ free(al); /*free-5 */ } } @@ -2853,7 +2853,7 @@ static SaAisErrorT rt_object_create_comm evt.info.immnd.info.objCreate.className.size = strlen(className) + 1; /*alloc-1 */ - evt.info.immnd.info.objCreate.className.buf = malloc(evt.info.immnd.info.objCreate.className.size); + evt.info.immnd.info.objCreate.className.buf = (char *) malloc(evt.info.immnd.info.objCreate.className.size); strncpy(evt.info.immnd.info.objCreate.className.buf, className, evt.info.immnd.info.objCreate.className.size); if(objectName) { @@ -2922,7 +2922,7 @@ static SaAisErrorT rt_object_create_comm } else if(attr->attrValueType == SA_IMM_ATTR_SANAMET) { int n; for(n=0; n<attr->attrValuesNumber; n++) { - if(!osaf_is_extended_name_valid(attr->attrValues[n])) { + if(!osaf_is_extended_name_valid((SaNameT*) attr->attrValues[n])) { rc = SA_AIS_ERR_INVALID_PARAM; TRACE_2("ERR_INVALID_PARAM: Attribute '%s' value is not valid", attr->attrName); goto mds_send_fail; @@ -2931,7 +2931,7 @@ static SaAisErrorT rt_object_create_comm } /*alloc-3 */ - p = calloc(1, sizeof(IMMSV_ATTR_VALUES_LIST)); + p = (IMMSV_ATTR_VALUES_LIST *) calloc(1, sizeof(IMMSV_ATTR_VALUES_LIST)); p->n.attrName.size = strlen(attr->attrName) + 1; if (p->n.attrName.size >= IMMSV_MAX_ATTR_NAME_LENGTH) { @@ -2942,7 +2942,7 @@ static SaAisErrorT rt_object_create_comm } /*alloc-4 */ - p->n.attrName.buf = malloc(p->n.attrName.size); + p->n.attrName.buf = (char *) malloc(p->n.attrName.size); strncpy(p->n.attrName.buf, attr->attrName, p->n.attrName.size); p->n.attrValuesNumber = attr->attrValuesNumber; @@ -2957,7 +2957,7 @@ static SaAisErrorT rt_object_create_comm unsigned int i; for (i = 1; i <= numAdded; ++i) { /*alloc-6 */ - IMMSV_EDU_ATTR_VAL_LIST *al = calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); + IMMSV_EDU_ATTR_VAL_LIST *al = (IMMSV_EDU_ATTR_VAL_LIST *) calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); /*alloc-7 */ imma_copyAttrValue(&(al->n), attr->attrValueType, avarr[i]); @@ -3036,13 +3036,13 @@ static SaAisErrorT rt_object_create_comm p->n.attrName.buf = NULL; } - immsv_evt_free_att_val(&(p->n.attrValue), p->n.attrValueType); /*free-5 */ + immsv_evt_free_att_val(&(p->n.attrValue), (SaImmValueTypeT) p->n.attrValueType); /*free-5 */ while (p->n.attrMoreValues) { IMMSV_EDU_ATTR_VAL_LIST *al = p->n.attrMoreValues; p->n.attrMoreValues = al->next; al->next = NULL; - immsv_evt_free_att_val(&(al->n), p->n.attrValueType); /*free-7 */ + immsv_evt_free_att_val(&(al->n), (SaImmValueTypeT) p->n.attrValueType); /*free-7 */ free(al); /*free-6 */ } @@ -3567,7 +3567,7 @@ static SaAisErrorT getAdmoName(SaImmHandleT privateOmHandle, IMMA_CALLBACK_INFO * cbi, SaNameT* admoNameOut) { SaAisErrorT rc = SA_AIS_OK; - const SaImmAttrNameT admoNameAttr = SA_IMM_ATTR_ADMIN_OWNER_NAME; + const SaImmAttrNameT admoNameAttr = (char *) SA_IMM_ATTR_ADMIN_OWNER_NAME; SaImmAttrValuesT_2 **attributes = NULL; SaImmAttrValuesT_2 *attrVal = NULL; TRACE_ENTER(); diff --git a/osaf/libs/agents/saf/imma/imma_om_api.cc b/osaf/libs/agents/saf/imma/imma_om_api.cc --- a/osaf/libs/agents/saf/imma/imma_om_api.cc +++ b/osaf/libs/agents/saf/imma/imma_om_api.cc @@ -49,7 +49,7 @@ static const char *immLoaderName = IMMSV /* TODO: ABT move these to cb ? OR stop using the cb*/ static SaInt32T immInvocations = 0; -static SaBoolT immOmIsLoader = SA_FALSE; +static bool immOmIsLoader = false; static const char *sysaClName = SA_IMM_ATTR_CLASS_NAME; static const char *sysaAdmName = SA_IMM_ATTR_ADMIN_OWNER_NAME; @@ -1020,7 +1020,7 @@ SaAisErrorT saImmOmAdminOwnerInitialize( osaf_extended_name_alloc(adminOwnerName, &evt.info.immnd.info.adminitReq.i.adminOwnerName); if (releaseOwnershipOnFinalize) { - evt.info.immnd.info.adminitReq.i.releaseOwnershipOnFinalize = true; + evt.info.immnd.info.adminitReq.i.releaseOwnershipOnFinalize = SA_TRUE; /* Release on finalize can not be undone in case of IMMND crash. The om-handle can then not be resurrected unless this admin-owner has been finalized before the IMMND crash. @@ -1102,7 +1102,7 @@ SaAisErrorT saImmOmAdminOwnerInitialize( rc = out_evt->info.imma.info.admInitRsp.error; if (rc == SA_AIS_OK) { ao_node->mAdminOwnerId = out_evt->info.imma.info.admInitRsp.ownerId; - ao_node->mAdminOwnerName = calloc(1, nameLen+1); + ao_node->mAdminOwnerName = (char *) calloc(1, nameLen+1); strncpy(ao_node->mAdminOwnerName, adminOwnerName, nameLen); } else {goto admin_owner_node_free;} } else { @@ -1834,7 +1834,7 @@ static SaAisErrorT ccb_object_create_com evt.info.immnd.info.objCreate.className.size = strlen(className) + 1; /*alloc-1 */ - evt.info.immnd.info.objCreate.className.buf = malloc(evt.info.immnd.info.objCreate.className.size); + evt.info.immnd.info.objCreate.className.buf = (char *) malloc(evt.info.immnd.info.objCreate.className.size); if (evt.info.immnd.info.objCreate.className.buf == NULL) { rc = SA_AIS_ERR_NO_MEMORY; goto mds_send_fail; @@ -1933,7 +1933,7 @@ static SaAisErrorT ccb_object_create_com } /*alloc-3 */ - p = calloc(1, sizeof(IMMSV_ATTR_VALUES_LIST)); + p = (IMMSV_ATTR_VALUES_LIST *) calloc(1, sizeof(IMMSV_ATTR_VALUES_LIST)); p->n.attrName.size = strlen(attr->attrName) + 1; if (p->n.attrName.size >= IMMSV_MAX_ATTR_NAME_LENGTH) { @@ -1945,7 +1945,7 @@ static SaAisErrorT ccb_object_create_com } /*alloc-4 */ - p->n.attrName.buf = malloc(p->n.attrName.size); + p->n.attrName.buf = (char *) malloc(p->n.attrName.size); strncpy(p->n.attrName.buf, attr->attrName, p->n.attrName.size); @@ -1961,7 +1961,7 @@ static SaAisErrorT ccb_object_create_com unsigned int i; for (i = 1; i <= numAdded; ++i) { /*alloc-6 */ - IMMSV_EDU_ATTR_VAL_LIST *al = calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); + IMMSV_EDU_ATTR_VAL_LIST *al = (IMMSV_EDU_ATTR_VAL_LIST *) calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); /*alloc-7 */ imma_copyAttrValue(&(al->n), attr->attrValueType, avarr[i]); @@ -2022,13 +2022,13 @@ static SaAisErrorT ccb_object_create_com p->n.attrName.buf = NULL; } - immsv_evt_free_att_val(&(p->n.attrValue), p->n.attrValueType); /*free-5 */ + immsv_evt_free_att_val(&(p->n.attrValue), (SaImmValueTypeT) p->n.attrValueType); /*free-5 */ while (p->n.attrMoreValues) { IMMSV_EDU_ATTR_VAL_LIST *al = p->n.attrMoreValues; p->n.attrMoreValues = al->next; al->next = NULL; - immsv_evt_free_att_val(&(al->n), p->n.attrValueType); /*free-7 */ + immsv_evt_free_att_val(&(al->n), (SaImmValueTypeT) p->n.attrValueType); /*free-7 */ free(al); /*free-6 */ } @@ -2449,12 +2449,12 @@ static SaAisErrorT ccb_object_modify_com /*alloc-2 */ - p = calloc(1, sizeof(IMMSV_ATTR_MODS_LIST)); + p = (IMMSV_ATTR_MODS_LIST *) calloc(1, sizeof(IMMSV_ATTR_MODS_LIST)); p->attrModType = attrMod->modType; p->attrValue.attrName.size = strlen(attrMod->modAttr.attrName) + 1; /* alloc 3 */ - p->attrValue.attrName.buf = malloc(p->attrValue.attrName.size); + p->attrValue.attrName.buf = (char *) malloc(p->attrValue.attrName.size); strncpy(p->attrValue.attrName.buf, attrMod->modAttr.attrName, p->attrValue.attrName.size); p->attrValue.attrValuesNumber = attrMod->modAttr.attrValuesNumber; @@ -2470,7 +2470,7 @@ static SaAisErrorT ccb_object_modify_com unsigned int i; for (i = 1; i <= numAdded; ++i) { /*alloc-5 */ - IMMSV_EDU_ATTR_VAL_LIST *al = calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); + IMMSV_EDU_ATTR_VAL_LIST *al = (IMMSV_EDU_ATTR_VAL_LIST *) calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); /*alloc-6 */ imma_copyAttrValue(&(al->n), attrMod->modAttr.attrValueType, avarr[i]); al->next = p->attrValue.attrMoreValues; /*NULL initially */ @@ -2521,13 +2521,13 @@ static SaAisErrorT ccb_object_modify_com if (p->attrValue.attrValuesNumber) { immsv_evt_free_att_val(&(p->attrValue.attrValue), /*free-4 */ - p->attrValue.attrValueType); + (SaImmValueTypeT) p->attrValue.attrValueType); while (p->attrValue.attrMoreValues) { IMMSV_EDU_ATTR_VAL_LIST *al = p->attrValue.attrMoreValues; p->attrValue.attrMoreValues = al->next; al->next = NULL; - immsv_evt_free_att_val(&(al->n), p->attrValue.attrValueType); /*free-6 */ + immsv_evt_free_att_val(&(al->n), (SaImmValueTypeT) p->attrValue.attrValueType); /*free-6 */ free(al); /*free-5 */ } } @@ -3831,7 +3831,7 @@ static SaAisErrorT admin_op_invoke_commo for (i = 0; params[i]; ++i) { const SaImmAdminOperationParamsT_2 *param = params[i]; /*alloc-2 */ - IMMSV_ADMIN_OPERATION_PARAM *p = malloc(sizeof(IMMSV_ADMIN_OPERATION_PARAM)); + IMMSV_ADMIN_OPERATION_PARAM *p = (IMMSV_ADMIN_OPERATION_PARAM *) malloc(sizeof(IMMSV_ADMIN_OPERATION_PARAM)); memset(p, 0, sizeof(IMMSV_ADMIN_OPERATION_PARAM)); TRACE("PARAM:%s \n", param->paramName); @@ -3856,7 +3856,7 @@ static SaAisErrorT admin_op_invoke_commo goto mds_send_fail; } /*alloc-3 */ - p->paramName.buf = malloc(p->paramName.size); + p->paramName.buf = (char *) malloc(p->paramName.size); strncpy(p->paramName.buf, param->paramName, p->paramName.size); p->paramType = param->paramType; @@ -3895,7 +3895,7 @@ static SaAisErrorT admin_op_invoke_commo rc = out_evt->info.imma.info.errRsp.error; TRACE("ERROR returned:%u", rc); osafassert(rc != SA_AIS_OK); - *operationReturnValue = IMMSV_IMPOSSIBLE_ERROR; //Bogus result since error is set. + *operationReturnValue = (SaAisErrorT) IMMSV_IMPOSSIBLE_ERROR; //Bogus result since error is set. } else { TRACE("Normal return"); osafassert((out_evt->info.imma.type == IMMA_EVT_ND2A_ADMOP_RSP) || @@ -3957,7 +3957,7 @@ static SaAisErrorT admin_op_invoke_commo free(p->paramName.buf); p->paramName.buf = NULL; } - immsv_evt_free_att_val(&(p->paramBuffer), p->paramType); /*free-4 */ + immsv_evt_free_att_val(&(p->paramBuffer), (SaImmValueTypeT) p->paramType); /*free-4 */ p->next = NULL; free(p); /*free-2 */ } @@ -4373,7 +4373,7 @@ static SaAisErrorT admin_op_invoke_async for (i = 0; params[i]; ++i) { param = params[i]; /*alloc-2 */ - IMMSV_ADMIN_OPERATION_PARAM *p = malloc(sizeof(IMMSV_ADMIN_OPERATION_PARAM)); + IMMSV_ADMIN_OPERATION_PARAM *p = (IMMSV_ADMIN_OPERATION_PARAM *) malloc(sizeof(IMMSV_ADMIN_OPERATION_PARAM)); memset(p, 0, sizeof(IMMSV_ADMIN_OPERATION_PARAM)); TRACE("PARAM:%s ", param->paramName); @@ -4399,7 +4399,7 @@ static SaAisErrorT admin_op_invoke_async } /*alloc-3 */ - p->paramName.buf = malloc(p->paramName.size); + p->paramName.buf = (char *) malloc(p->paramName.size); strncpy(p->paramName.buf, param->paramName, p->paramName.size); p->paramType = param->paramType; @@ -4477,7 +4477,7 @@ static SaAisErrorT admin_op_invoke_async free(p->paramName.buf); p->paramName.buf = NULL; } - immsv_evt_free_att_val(&(p->paramBuffer), p->paramType); /*free-4 */ + immsv_evt_free_att_val(&(p->paramBuffer), (SaImmValueTypeT) p->paramType); /*free-4 */ p->next = NULL; free(p); /*free-2 */ p=NULL; @@ -4748,7 +4748,7 @@ SaAisErrorT saImmOmClassCreate_2(SaImmHa TRACE("Zero length class name, not allowed."); goto mds_send_fail; } - evt.info.immnd.info.classDescr.className.buf = malloc(evt.info.immnd.info.classDescr.className.size); /*alloc-1 */ + evt.info.immnd.info.classDescr.className.buf = (char *) malloc(evt.info.immnd.info.classDescr.className.size); /*alloc-1 */ strncpy(evt.info.immnd.info.classDescr.className.buf, className, (size_t)evt.info.immnd.info.classDescr.className.size); @@ -4794,7 +4794,7 @@ SaAisErrorT saImmOmClassCreate_2(SaImmHa } IMMSV_ATTR_DEF_LIST *p = /*alloc-2 */ - malloc(sizeof(IMMSV_ATTR_DEF_LIST)); + (IMMSV_ATTR_DEF_LIST *) malloc(sizeof(IMMSV_ATTR_DEF_LIST)); memset(p, 0, sizeof(IMMSV_ATTR_DEF_LIST)); p->d.attrName.size = strlen(attr->attrName) + 1; @@ -4806,11 +4806,11 @@ SaAisErrorT saImmOmClassCreate_2(SaImmHa p=NULL; goto mds_send_fail; } - p->d.attrName.buf = malloc(p->d.attrName.size); /* alloc-3 */ + p->d.attrName.buf = (char *) malloc(p->d.attrName.size); /* alloc-3 */ strncpy(p->d.attrName.buf, attr->attrName, p->d.attrName.size); p->d.attrValueType = attr->attrValueType; - if (!imma_proc_is_valid_type(p->d.attrValueType)) { + if (!imma_proc_is_valid_type((SaImmValueTypeT) p->d.attrValueType)) { rc = SA_AIS_ERR_INVALID_PARAM; TRACE("Unknown type not allowed for attr:%s class%s", p->d.attrName.buf, evt.info.immnd.info.classDescr.className.buf); @@ -4825,7 +4825,7 @@ SaAisErrorT saImmOmClassCreate_2(SaImmHa if (attr->attrDefaultValue) { p->d.attrDefaultValue = /*alloc-4.1 */ - malloc(sizeof(IMMSV_EDU_ATTR_VAL)); + (IMMSV_EDU_ATTR_VAL *) malloc(sizeof(IMMSV_EDU_ATTR_VAL)); memset(p->d.attrDefaultValue, 0, sizeof(IMMSV_EDU_ATTR_VAL)); /*alloc-4.2 */ imma_copyAttrValue(p->d.attrDefaultValue, attr->attrValueType, attr->attrDefaultValue); @@ -4839,11 +4839,11 @@ SaAisErrorT saImmOmClassCreate_2(SaImmHa if (!attrClNameExist) { /*TRACE("Creating class name attribute def"); */ sysattr = /*alloc-2 */ - malloc(sizeof(IMMSV_ATTR_DEF_LIST)); + (IMMSV_ATTR_DEF_LIST *) malloc(sizeof(IMMSV_ATTR_DEF_LIST)); memset(sysattr, 0, sizeof(IMMSV_ATTR_DEF_LIST)); sysattr->d.attrName.size = strlen(sysaClName) + 1; - sysattr->d.attrName.buf = malloc(sysattr->d.attrName.size); /*alloc-3 */ + sysattr->d.attrName.buf = (char *) malloc(sysattr->d.attrName.size); /*alloc-3 */ strncpy(sysattr->d.attrName.buf, sysaClName, sysattr->d.attrName.size); sysattr->d.attrValueType = SA_IMM_ATTR_SASTRINGT; if (classCategory == SA_IMM_CLASS_CONFIG) { @@ -4854,7 +4854,7 @@ SaAisErrorT saImmOmClassCreate_2(SaImmHa SA_IMM_ATTR_PERSISTENT) : (SA_IMM_ATTR_RUNTIME | SA_IMM_ATTR_CACHED); } sysattr->d.attrNtfId = 0; /*alloc-4.1 */ - sysattr->d.attrDefaultValue = malloc(sizeof(IMMSV_EDU_ATTR_VAL)); + sysattr->d.attrDefaultValue = (IMMSV_EDU_ATTR_VAL *) malloc(sizeof(IMMSV_EDU_ATTR_VAL)); memset(sysattr->d.attrDefaultValue, 0, sizeof(IMMSV_EDU_ATTR_VAL)); /*alloc-4.2 */ imma_copyAttrValue(sysattr->d.attrDefaultValue, SA_IMM_ATTR_SASTRINGT, (SaImmAttrValueT)&className); @@ -4866,11 +4866,11 @@ SaAisErrorT saImmOmClassCreate_2(SaImmHa if (!attrAdmNameExist) { /*TRACE("Creating admin-owner name attribute def"); */ sysattr = /*alloc-2 */ - malloc(sizeof(IMMSV_ATTR_DEF_LIST)); + (IMMSV_ATTR_DEF_LIST *) malloc(sizeof(IMMSV_ATTR_DEF_LIST)); memset(sysattr, 0, sizeof(IMMSV_ATTR_DEF_LIST)); sysattr->d.attrName.size = strlen(sysaAdmName) + 1; - sysattr->d.attrName.buf = malloc(sysattr->d.attrName.size); /*alloc-3 */ + sysattr->d.attrName.buf = (char *) malloc(sysattr->d.attrName.size); /*alloc-3 */ strncpy(sysattr->d.attrName.buf, sysaAdmName, sysattr->d.attrName.size); sysattr->d.attrValueType = SA_IMM_ATTR_SASTRINGT; /* Should this attribute really be a config attribute ? @@ -4892,11 +4892,11 @@ SaAisErrorT saImmOmClassCreate_2(SaImmHa if (!attrImplNameExist) { /*TRACE("Creating implementer name attribute def"); */ sysattr = /*alloc-2 */ - malloc(sizeof(IMMSV_ATTR_DEF_LIST)); + (IMMSV_ATTR_DEF_LIST *) malloc(sizeof(IMMSV_ATTR_DEF_LIST)); memset(sysattr, 0, sizeof(IMMSV_ATTR_DEF_LIST)); sysattr->d.attrName.size = strlen(sysaImplName) + 1; - sysattr->d.attrName.buf = malloc(sysattr->d.attrName.size); /*alloc-3 */ + sysattr->d.attrName.buf = (char *) malloc(sysattr->d.attrName.size); /*alloc-3 */ strncpy(sysattr->d.attrName.buf, sysaImplName, sysattr->d.attrName.size); sysattr->d.attrValueType = SA_IMM_ATTR_SASTRINGT; /* Should this attribute really be a config attribute ? @@ -5056,7 +5056,7 @@ SaAisErrorT saImmOmClassDescriptionGet_2 evt.info.immnd.type = IMMND_EVT_A2ND_CLASS_DESCR_GET; evt.info.immnd.info.classDescr.className.size = strlen(className) + 1; - evt.info.immnd.info.classDescr.className.buf = malloc(evt.info.immnd.info.classDescr.className.size); /*alloc-0 */ + evt.info.immnd.info.classDescr.className.buf = (char *) malloc(evt.info.immnd.info.classDescr.className.size); /*alloc-0 */ strncpy(evt.info.immnd.info.classDescr.className.buf, className, (size_t)evt.info.immnd.info.classDescr.className.size); @@ -5105,7 +5105,7 @@ SaAisErrorT saImmOmClassDescriptionGet_2 SaImmAttrDefinitionT_2 **attr = NULL; size_t attrDataSize = 0; - *classCategory = out_evt->info.imma.info.classDescr.classCategory; + *classCategory = (SaImmClassCategoryT) out_evt->info.imma.info.classDescr.classCategory; IMMSV_ATTR_DEF_LIST *p = out_evt->info.imma.info.classDescr.attrDefinitions; while (p) { @@ -5114,18 +5114,18 @@ SaAisErrorT saImmOmClassDescriptionGet_2 } attrDataSize = sizeof(SaImmAttrDefinitionT_2 *) * (noOfAttributes + 1); - attr = malloc(attrDataSize); /*alloc-1 */ + attr = (SaImmAttrDefinitionT_2**) malloc(attrDataSize); /*alloc-1 */ TRACE("Alloc attrdefs array:%p", attr); memset(attr, 0, attrDataSize); p = out_evt->info.imma.info.classDescr.attrDefinitions; for (; i < noOfAttributes; i++) { IMMSV_ATTR_DEF_LIST *prev = p; IMMSV_ATTR_DEFINITION *q = &(p->d); - attr[i] = malloc(sizeof(SaImmAttrDefinitionT_2)); /*alloc-2 */ - attr[i]->attrName = malloc(q->attrName.size + 1); /*alloc-3 */ + attr[i] = (SaImmAttrDefinitionT_2 *) malloc(sizeof(SaImmAttrDefinitionT_2)); /*alloc-2 */ + attr[i]->attrName = (char *) malloc(q->attrName.size + 1); /*alloc-3 */ strncpy(attr[i]->attrName, (const char *)q->attrName.buf, q->attrName.size + 1); attr[i]->attrName[q->attrName.size] = 0; - attr[i]->attrValueType = q->attrValueType; + attr[i]->attrValueType = (SaImmValueTypeT) q->attrValueType; attr[i]->attrFlags = q->attrFlags; /* attr[i]->attrNtfId = q->attrNtfId; */ @@ -5212,7 +5212,7 @@ SaAisErrorT saImmOmClassDescriptionGet_2 case SA_IMM_ATTR_SASTRINGT: strp = (SaStringT *)copyv; *strp = /*alloc-5 */ - malloc(q->attrDefaultValue->val.x.size); + (char *) malloc(q->attrDefaultValue->val.x.size); memcpy(*strp, q->attrDefaultValue->val.x.buf, q->attrDefaultValue->val.x.size); break; @@ -5237,7 +5237,7 @@ SaAisErrorT saImmOmClassDescriptionGet_2 attr[i]->attrDefaultValue = copyv; /*Delete source attr-value */ - immsv_evt_free_att_val(q->attrDefaultValue, q->attrValueType); + immsv_evt_free_att_val(q->attrDefaultValue, (SaImmValueTypeT) q->attrValueType); free(q->attrDefaultValue); q->attrDefaultValue = NULL; @@ -5453,7 +5453,7 @@ SaAisErrorT saImmOmClassDelete(SaImmHand evt.info.immnd.type = IMMND_EVT_A2ND_CLASS_DELETE; evt.info.immnd.info.classDescr.className.size = strlen(className) + 1; - evt.info.immnd.info.classDescr.className.buf = malloc(evt.info.immnd.info.classDescr.className.size); /*alloc-1 */ + evt.info.immnd.info.classDescr.className.buf = (char *) malloc(evt.info.immnd.info.classDescr.className.size); /*alloc-1 */ strncpy(evt.info.immnd.info.classDescr.className.buf, className, (size_t)evt.info.immnd.info.classDescr.className.size); @@ -5923,7 +5923,7 @@ static SaAisErrorT accessor_get_common(S IMMSV_ATTR_NAME_LIST *p = (IMMSV_ATTR_NAME_LIST *) malloc(sizeof(IMMSV_ATTR_NAME_LIST)); /* alloc-2 */ p->name.size = strlen(*namev) + 1; - p->name.buf = malloc(p->name.size); /* alloc-3 */ + p->name.buf = (char *) malloc(p->name.size); /* alloc-3 */ strncpy(p->name.buf, *namev, p->name.size); p->name.buf[p->name.size - 1] = 0; @@ -6006,27 +6006,27 @@ static SaAisErrorT accessor_get_common(S } p = attrValueList; - attr = calloc(noOfAttributes + 1, sizeof(SaImmAttrValuesT_2 *)); /* alloc-1 */ + attr = (SaImmAttrValuesT_2 **) calloc(noOfAttributes + 1, sizeof(SaImmAttrValuesT_2 *)); /* alloc-1 */ for(i=0; i<noOfAttributes; i++, p = p->next) { IMMSV_ATTR_VALUES *q = &(p->n); - attr[i] = calloc(1, sizeof(SaImmAttrValuesT_2)); /* alloc-2 */ - attr[i]->attrName = malloc(q->attrName.size + 1); /* alloc-3 */ + attr[i] = (SaImmAttrValuesT_2 *) calloc(1, sizeof(SaImmAttrValuesT_2)); /* alloc-2 */ + attr[i]->attrName = (char *) malloc(q->attrName.size + 1); /* alloc-3 */ strncpy(attr[i]->attrName, (const char *)q->attrName.buf, q->attrName.size + 1); attr[i]->attrName[q->attrName.size] = 0; /*redundant. */ attr[i]->attrValuesNumber = q->attrValuesNumber; attr[i]->attrValueType = (SaImmValueTypeT)q->attrValueType; if (q->attrValuesNumber) { - attr[i]->attrValues = calloc(1, q->attrValuesNumber * sizeof(SaImmAttrValueT)); /*alloc-4 */ + attr[i]->attrValues = (SaImmAttrValueT *) calloc(1, q->attrValuesNumber * sizeof(SaImmAttrValueT)); /*alloc-4 */ /*alloc-5 */ - attr[i]->attrValues[0] = imma_copyAttrValue3(q->attrValueType, &(q->attrValue)); + attr[i]->attrValues[0] = imma_copyAttrValue3((SaImmValueTypeT) q->attrValueType, &(q->attrValue)); if (q->attrValuesNumber > 1) { int ix; IMMSV_EDU_ATTR_VAL_LIST *r = q->attrMoreValues; for (ix = 1; ix < q->attrValuesNumber; ++ix) { osafassert(r); - attr[i]->attrValues[ix] = imma_copyAttrValue3(q->attrValueType, &(r->n)); /*alloc-5 */ + attr[i]->attrValues[ix] = imma_copyAttrValue3((SaImmValueTypeT) q->attrValueType, &(r->n)); /*alloc-5 */ r = r->next; } } @@ -6155,10 +6155,10 @@ static unsigned int get_obj_size(const I obj_size += (avl->n.attrName.size + 4 + 1); /* string + size + next-marker */ if(avl->n.attrValuesNumber) { - obj_size += get_att_val_size(&(avl->n.attrValue), avl->n.attrValueType); + obj_size += get_att_val_size(&(avl->n.attrValue), (SaImmValueTypeT) avl->n.attrValueType); if (avl->n.attrValuesNumber > 1) { obj_size += get_attr_list_size(avl->n.attrMoreValues, - avl->n.attrValueType); + (SaImmValueTypeT) avl->n.attrValueType); } } avl=avl->next; @@ -6631,13 +6631,13 @@ SaAisErrorT immsv_sync(SaImmHandleT immH evt.info.immnd.info.obj_sync.className.size = strlen(className) + 1; /*alloc-1 */ - evt.info.immnd.info.obj_sync.className.buf = malloc(evt.info.immnd.info.obj_sync.className.size); + evt.info.immnd.info.obj_sync.className.buf = (char *) malloc(evt.info.immnd.info.obj_sync.className.size); strncpy(evt.info.immnd.info.obj_sync.className.buf, className, evt.info.immnd.info.obj_sync.className.size); evt.info.immnd.info.obj_sync.objectName.size = osaf_extended_name_length(objectName) + 1; /*alloc-2 */ - evt.info.immnd.info.obj_sync.objectName.buf = malloc(evt.info.immnd.info.obj_sync.objectName.size); + evt.info.immnd.info.obj_sync.objectName.buf = (char *) malloc(evt.info.immnd.info.obj_sync.objectName.size); memcpy(evt.info.immnd.info.obj_sync.objectName.buf, osaf_extended_name_borrow(objectName), evt.info.immnd.info.obj_sync.objectName.size - 1); @@ -6656,7 +6656,7 @@ SaAisErrorT immsv_sync(SaImmHandleT immH } /*alloc-3 */ - IMMSV_ATTR_VALUES_LIST *p = calloc(1, sizeof(IMMSV_ATTR_VALUES_LIST)); + IMMSV_ATTR_VALUES_LIST *p = (IMMSV_ATTR_VALUES_LIST *) calloc(1, sizeof(IMMSV_ATTR_VALUES_LIST)); p->n.attrName.size = strlen(attr->attrName) + 1; if (p->n.attrName.size >= IMMSV_MAX_ATTR_NAME_LENGTH) { @@ -6668,7 +6668,7 @@ SaAisErrorT immsv_sync(SaImmHandleT immH } /*alloc-4 */ - p->n.attrName.buf = malloc(p->n.attrName.size); + p->n.attrName.buf = (char *) malloc(p->n.attrName.size); strncpy(p->n.attrName.buf, attr->attrName, p->n.attrName.size); p->n.attrValuesNumber = attr->attrValuesNumber; @@ -6683,7 +6683,7 @@ SaAisErrorT immsv_sync(SaImmHandleT immH unsigned int i; for (i = 1; i <= numAdded; ++i) { /*alloc-6 */ - IMMSV_EDU_ATTR_VAL_LIST *al = calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); + IMMSV_EDU_ATTR_VAL_LIST *al = (IMMSV_EDU_ATTR_VAL_LIST *) calloc(1, sizeof(IMMSV_EDU_ATTR_VAL_LIST)); /*alloc-7 */ imma_copyAttrValue(&(al->n), attr->attrValueType, avarr[i]); @@ -6713,7 +6713,7 @@ SaAisErrorT immsv_sync(SaImmHandleT immH { /* Case C only. */ /* There is more space. Push EVT onto batch & get ready for more objects */ - IMMSV_OM_OBJECT_SYNC* tmp = calloc(1, sizeof(IMMSV_OM_OBJECT_SYNC)); + IMMSV_OM_OBJECT_SYNC* tmp = (IMMSV_OM_OBJECT_SYNC*) calloc(1, sizeof(IMMSV_OM_OBJECT_SYNC)); tmp->next = batch; (*batchp) = tmp; @@ -6803,13 +6803,13 @@ SaAisErrorT immsv_sync(SaImmHandleT immH p->n.attrName.buf = NULL; } - immsv_evt_free_att_val(&(p->n.attrValue), p->n.attrValueType); /*free-5 */ + immsv_evt_free_att_val(&(p->n.attrValue), (SaImmValueTypeT) p->n.attrValueType); /*free-5 */ while (p->n.attrMoreValues) { IMMSV_EDU_ATTR_VAL_LIST *al = p->n.attrMoreValues; p->n.attrMoreValues = al->next; al->next = NULL; - immsv_evt_free_att_val(&(al->n), p->n.attrValueType); /*free-7 */ + immsv_evt_free_att_val(&(al->n), (SaImmValueTypeT) p->n.attrValueType); /*free-7 */ free(al); /*free-6 */ } @@ -7239,7 +7239,7 @@ static SaAisErrorT search_init_common(Sa if(searchParam->searchOneAttr.attrName) { req->searchParam.choice.oneAttrParam.attrName.size = strlen(searchParam->searchOneAttr.attrName) + 1; req->searchParam.choice.oneAttrParam.attrName.buf = /*alloc-2 */ - malloc(req->searchParam.choice.oneAttrParam.attrName.size); + (char *) malloc(req->searchParam.choice.oneAttrParam.attrName.size); strncpy(req->searchParam.choice.oneAttrParam.attrName.buf, (char *)searchParam->searchOneAttr.attrName, (size_t)req->searchParam.choice.oneAttrParam.attrName.size); @@ -7269,7 +7269,7 @@ static SaAisErrorT search_init_common(Sa IMMSV_ATTR_NAME_LIST *p = (IMMSV_ATTR_NAME_LIST *) malloc(sizeof(IMMSV_ATTR_NAME_LIST)); /*alloc-4 */ p->name.size = strlen(*namev) + 1; - p->name.buf = malloc(p->name.size); /*alloc-5 */ + p->name.buf = (char *) malloc(p->name.size); /*alloc-5 */ strncpy(p->name.buf, *namev, p->name.size); p->name.buf[p->name.size - 1] = 0; @@ -7700,12 +7700,12 @@ searchresult: } attrDataSize = sizeof(SaImmAttrValuesT_2 *) * (noOfAttributes + 1); - attr = calloc(1, attrDataSize); /*alloc-1 */ + attr = (SaImmAttrValuesT_2 **) calloc(1, attrDataSize); /*alloc-1 */ p = res_body->attrValuesList; for (; i < noOfAttributes; i++, p = p->next) { IMMSV_ATTR_VALUES *q = &(p->n); - attr[i] = calloc(1, sizeof(SaImmAttrValuesT_2)); /*alloc-2 */ - attr[i]->attrName = malloc(q->attrName.size + 1); /*alloc-3 */ + attr[i] = (SaImmAttrValuesT_2 *) calloc(1, sizeof(SaImmAttrValuesT_2)); /*alloc-2 */ + attr[i]->attrName = (char *) malloc(q->attrName.size + 1); /*alloc-3 */ strncpy(attr[i]->attrName, (const char *)q->attrName.buf, q->attrName.size + 1); attr[i]->attrName[q->attrName.size] = 0; /*redundant. */ attr[i]->attrValuesNumber = q->attrValuesNumber; @@ -7717,16 +7717,16 @@ searchresult: SaStringT* strp; Allocate the array of pointers, even if only one value */ - attr[i]->attrValues = calloc(1, q->attrValuesNumber * sizeof(SaImmAttrValueT)); /*alloc-4 */ + attr[i]->attrValues = (SaImmAttrValueT *) calloc(1, q->attrValuesNumber * sizeof(SaImmAttrValueT)); /*alloc-4 */ /*alloc-5 */ - attr[i]->attrValues[0] = imma_copyAttrValue3(q->attrValueType, &(q->attrValue)); + attr[i]->attrValues[0] = imma_copyAttrValue3((SaImmValueTypeT) q->attrValueType, &(q->attrValue)); if (q->attrValuesNumber > 1) { int ix; IMMSV_EDU_ATTR_VAL_LIST *r = q->attrMoreValues; for (ix = 1; ix < q->attrValuesNumber; ++ix) { osafassert(r); - attr[i]->attrValues[ix] = imma_copyAttrValue3(q->attrValueType, &(r->n));/*alloc-5 */ + attr[i]->attrValues[ix] = imma_copyAttrValue3((SaImmValueTypeT) q->attrValueType, &(r->n));/*alloc-5 */ r = r->next; } } @@ -8162,7 +8162,7 @@ static SaAisErrorT admin_owner_set_commo int i; for (i = 0; objectNames[i]; ++i) { - IMMSV_OBJ_NAME_LIST *ol = calloc(1, sizeof(IMMSV_OBJ_NAME_LIST)); /*a */ + IMMSV_OBJ_NAME_LIST *ol = (IMMSV_OBJ_NAME_LIST *) calloc(1, sizeof(IMMSV_OBJ_NAME_LIST)); /*a */ ol->name.size = strlen(objectNames[i]) + 1; ol->name.buf = (char *)objectNames[i]; ol->next = admo_set_evt.info.immnd.info.admReq.objectNames; /*null initially */ @@ -8403,7 +8403,7 @@ static SaAisErrorT admin_owner_release_c int i; for (i = 0; objectNames[i]; ++i) { - IMMSV_OBJ_NAME_LIST *ol = calloc(1, sizeof(IMMSV_OBJ_NAME_LIST)); /*a */ + IMMSV_OBJ_NAME_LIST *ol = (IMMSV_OBJ_NAME_LIST *) calloc(1, sizeof(IMMSV_OBJ_NAME_LIST)); /*a */ ol->name.size = strlen(objectNames[i]) + 1; ol->name.buf = (char *)objectNames[i]; ol->next = admo_set_evt.info.immnd.info.admReq.objectNames; /*null initially */ @@ -8614,7 +8614,7 @@ static SaAisErrorT admin_owner_clear_com int i; for (i = 0; objectNames[i]; ++i) { - IMMSV_OBJ_NAME_LIST *ol = calloc(1, sizeof(IMMSV_OBJ_NAME_LIST)); /*a */ + IMMSV_OBJ_NAME_LIST *ol = (IMMSV_OBJ_NAME_LIST *) calloc(1, sizeof(IMMSV_OBJ_NAME_LIST)); /*a */ ol->name.size = strlen(objectNames[i]) + 1; ol->name.buf = (char *)objectNames[i]; ol->next = admo_set_evt.info.immnd.info.admReq.objectNames; /*null initially */ @@ -9153,7 +9153,7 @@ static SaBoolT imma_re_initialize_admin_ evt.info.immnd.info.adminitReq.client_hdl = immHandle; osaf_extended_name_alloc(adm_found_node->mAdminOwnerName, &evt.info.immnd.info.adminitReq.i.adminOwnerName); - evt.info.immnd.info.adminitReq.i.releaseOwnershipOnFinalize = false; + evt.info.immnd.info.adminitReq.i.releaseOwnershipOnFinalize = SA_FALSE; temp_hdl = adm_found_node->admin_owner_hdl; timeout = cl_node->syncr_timeout; @@ -9697,7 +9697,7 @@ SaAisErrorT immsv_om_augment_ccb_get_adm { SaAisErrorT rc = SA_AIS_OK; - const SaImmAttrNameT admoNameAttr = SA_IMM_ATTR_ADMIN_OWNER_NAME; + const SaImmAttrNameT admoNameAttr = (char *) SA_IMM_ATTR_ADMIN_OWNER_NAME; SaImmAttrNameT attributeNames[2] = {admoNameAttr, NULL}; SaImmAttrValuesT_2 **attributes = NULL; SaImmAttrValuesT_2 *attrVal = NULL; diff --git a/osaf/libs/agents/saf/imma/imma_proc.cc b/osaf/libs/agents/saf/imma/imma_proc.cc --- a/osaf/libs/agents/saf/imma/imma_proc.cc +++ b/osaf/libs/agents/saf/imma/imma_proc.cc @@ -56,7 +56,7 @@ static void imma_proc_free_callback(IMMA Notes : None ******************************************************************************/ -uint32_t imma_version_validate(SaVersionT *version) +SaAisErrorT imma_version_validate(SaVersionT *version) { SaAisErrorT retCode = SA_AIS_OK; @@ -253,7 +253,7 @@ static void imma_proc_admin_op_async_rsp imma_proc_decrement_pending_reply(cl_node, false); /* Allocate the Callback info */ - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { /* Fill the Call Back Info */ callback->type = IMMA_CALLBACK_OM_ADMIN_OP_RSP; @@ -306,16 +306,16 @@ SaImmAdminOperationParamsT_2 **imma_proc IMMSV_ADMIN_OPERATION_PARAM *prev = p; out_params[i] = (SaImmAdminOperationParamsT_2 *) malloc(sizeof(SaImmAdminOperationParamsT_2)); /*alloc-2 */ - out_params[i]->paramName = malloc(p->paramName.size + 1); /*alloc-3 */ + out_params[i]->paramName = (char *) malloc(p->paramName.size + 1); /*alloc-3 */ strncpy(out_params[i]->paramName, p->paramName.buf, p->paramName.size + 1); out_params[i]->paramName[p->paramName.size] = 0; /*string too long=>truncate */ free(p->paramName.buf); p->paramName.buf = NULL; p->paramName.size = 0; out_params[i]->paramType = (SaImmValueTypeT)p->paramType; - out_params[i]->paramBuffer = imma_copyAttrValue3(p->paramType, /*alloc-4 */ + out_params[i]->paramBuffer = imma_copyAttrValue3((SaImmValueTypeT) p->paramType, /*alloc-4 */ &(p->paramBuffer)); - immsv_evt_free_att_val(&(p->paramBuffer), p->paramType); + immsv_evt_free_att_val(&(p->paramBuffer), (SaImmValueTypeT) p->paramType); p = p->next; prev->next = NULL; free(prev); @@ -336,7 +336,7 @@ static void imma_proc_admop(IMMA_CB *cb, { IMMA_CALLBACK_INFO *callback; IMMA_CLIENT_NODE *cl_node = NULL; - SaBoolT isPbeAdmOp=(evt->type == IMMA_EVT_ND2A_IMM_PBE_ADMOP); + bool isPbeAdmOp = (evt->type == IMMA_EVT_ND2A_IMM_PBE_ADMOP); /*TODO: correct this, ugly use of continuationId */ SaImmOiHandleT implHandle = evt->info.admOpReq.continuationId; @@ -366,7 +366,7 @@ static void imma_proc_admop(IMMA_CB *cb, } /* Allocate the Callback info */ - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { /* Fill the Call Back Info */ if(isPbeAdmOp) { @@ -552,7 +552,7 @@ void imma_proc_terminate_oi_ccbs(IMMA_CB err = SA_AIS_ERR_FAILED_OPERATION; } - IMMA_CALLBACK_INFO *callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + IMMA_CALLBACK_INFO *callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); osafassert(callback); if (err == SA_AIS_OK) { callback->type = IMMA_CALLBACK_OI_CCB_APPLY; @@ -604,7 +604,7 @@ void imma_proc_stale_dispatch(IMMA_CB *c struct imma_oi_ccb_record *oiCcb = cl_node->activeOiCcbs; /* Send the stale handle triggering ipc-message */ - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); osafassert(callback); callback->type = IMMA_CALLBACK_STALE_HANDLE; callback->lcl_imm_hdl = 0LL; @@ -639,7 +639,7 @@ void imma_proc_stale_dispatch(IMMA_CB *c /* Non critical & stale CCB must have been aborted by server side. Generate abort upcall immediately, i.e. no need to wait for resurrect. */ - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); osafassert(callback); callback->type = IMMA_CALLBACK_OI_CCB_ABORT; callback->lcl_imm_hdl = cl_node->handle; @@ -768,7 +768,7 @@ static void imma_proc_rt_attr_update(IMM } /* Allocate the Callback info */ - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { /* Fill the Call Back Info */ callback->type = IMMA_CALLBACK_OI_RT_ATTR_UPDATE; @@ -811,7 +811,7 @@ static void imma_proc_ccb_completed(IMMA { IMMA_CALLBACK_INFO *callback; IMMA_CLIENT_NODE *cl_node = NULL; - SaBoolT isPrtObj=(evt->info.ccbCompl.ccbId == 0); + bool isPrtObj = (evt->info.ccbCompl.ccbId == 0); SaImmOiHandleT implHandle = evt->info.ccbCompl.immHandle; @@ -854,7 +854,7 @@ static void imma_proc_ccb_completed(IMMA } /* Allocate the Callback info */ - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { /* Fill the Call Back Info */ if(isPrtObj) { @@ -903,7 +903,7 @@ static void imma_proc_ccb_apply(IMMA_CB return; } - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { callback->type = IMMA_CALLBACK_OI_CCB_APPLY; callback->lcl_imm_hdl = implHandle; @@ -946,7 +946,7 @@ static void imma_proc_ccb_abort(IMMA_CB return; } - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { callback->type = IMMA_CALLBACK_OI_CCB_ABORT; callback->lcl_imm_hdl = implHandle; @@ -979,8 +979,8 @@ static void imma_proc_obj_delete(IMMA_CB { IMMA_CALLBACK_INFO *callback; IMMA_CLIENT_NODE *cl_node = NULL; - SaBoolT isPrtObj=((evt->info.objDelete.ccbId == 0) && (evt->info.objDelete.adminOwnerId != 0)); - SaBoolT isSpApplRto=((evt->info.objDelete.ccbId == 0) && (evt->info.objDelete.adminOwnerId == 0)); + bool isPrtObj = ((evt->info.objDelete.ccbId == 0) && (evt->info.objDelete.adminOwnerId != 0)); + bool isSpApplRto = ((evt->info.objDelete.ccbId == 0) && (evt->info.objDelete.adminOwnerId == 0)); if(isSpApplRto) {TRACE_3("imma_proc_obj_delete CCBID==0 admoId=0 SPECIAL applier RTO delete");} SaImmOiHandleT implHandle = evt->info.objDelete.immHandle; @@ -1007,7 +1007,7 @@ static void imma_proc_obj_delete(IMMA_CB } } - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { if(isPrtObj) { callback->type = IMMA_CALLBACK_PBE_PRT_OBJ_DELETE; @@ -1113,7 +1113,7 @@ static void imma_proc_obj_create(IMMA_CB } /* Allocate the Callback info */ - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { /* Fill the Call Back Info */ if(isPrtObj) { @@ -1174,8 +1174,8 @@ static void imma_proc_obj_modify(IMMA_CB { IMMA_CALLBACK_INFO *callback; IMMA_CLIENT_NODE *cl_node = NULL; - SaBoolT isPrtAttrs=((evt->info.objModify.ccbId == 0) && (evt->info.objModify.adminOwnerId != 0)); - SaBoolT isSpApplRtu = ((evt->info.objModify.ccbId == 0) && (evt->info.objModify.adminOwnerId == 0)); + bool isPrtAttrs = ((evt->info.objModify.ccbId == 0) && (evt->info.objModify.adminOwnerId != 0)); + bool isSpApplRtu = ((evt->info.objModify.ccbId == 0) && (evt->info.objModify.adminOwnerId == 0)); /* Can be a PRTO or a config obj with PRTAttrs. */ TRACE_ENTER(); if(isSpApplRtu) {TRACE_3("imma_proc_obj_modify CCBID==0 admoId=0 SPECIAL applier RTA modify");} @@ -1209,7 +1209,7 @@ static void imma_proc_obj_modify(IMMA_CB /* Allocate the Callback info */ - callback = calloc(1, sizeof(IMMA_CALLBACK_INFO)); + callback = (IMMA_CALLBACK_INFO *) calloc(1, sizeof(IMMA_CALLBACK_INFO)); if (callback) { /* Fill the Call Back Info */ if(isPrtAttrs) { @@ -1273,7 +1273,7 @@ void imma_proc_free_pointers(IMMA_CB *cb p->paramName.buf = NULL; p->paramName.size = 0; } - immsv_evt_free_att_val(&(p->paramBuffer), p->paramType);/*free-4 */ + immsv_evt_free_att_val(&(p->paramBuffer), (SaImmValueTypeT) p->paramType);/*free-4 */ p->next = NULL; free(p); /*free-2 */ } @@ -1289,7 +1289,7 @@ void imma_proc_free_pointers(IMMA_CB *cb p->paramName.buf = NULL; p->paramName.size = 0; } - immsv_evt_free_att_val(&(p->paramBuffer), p->paramType);/*free-4 */ + immsv_evt_free_att_val(&(p->paramBuffer), (SaImmValueTypeT) p->paramType);/*free-4 */ p->next = NULL; free(p); /*free-2 */ } @@ -1631,11 +1631,11 @@ uint32_t imma_proc_resurrect_client(IMMA Arguments : cb - ptr to the IMMA control block immHandle - IMM OM service handle - Return Values : NCSCC_RC_SUCCESS/NCSCC_RC_FAILURE + Return Values : SA_AIS_OK/SA_AIS_ERR_<ERROR> Notes : None ******************************************************************************/ -uint32_t imma_hdl_callbk_dispatch_one(IMMA_CB *cb, SaImmHandleT immHandle) +SaAisErrorT imma_hdl_callbk_dispatch_one(IMMA_CB *cb, SaImmHandleT immHandle) { IMMA_CALLBACK_INFO *callback = NULL; IMMA_CLIENT_NODE *cl_node = NULL; @@ -1697,11 +1697,11 @@ uint32_t imma_hdl_callbk_dispatch_one(IM Arguments : cb - ptr to the IMMA control block immHandle - IMM OM service handle - Return Values : NCSCC_RC_SUCCESS/NCSCC_RC_FAILURE + Return Values : SA_AIS_OK/SA_AIS_ERR_<ERROR> Notes : None ******************************************************************************/ -uint32_t imma_hdl_callbk_dispatch_all(IMMA_CB *cb, SaImmHandleT immHandle) +SaAisErrorT imma_hdl_callbk_dispatch_all(IMMA_CB *cb, SaImmHandleT immHandle) { IMMA_CALLBACK_INFO *callback = NULL; IMMA_CLIENT_NODE *cl_node = NULL; @@ -1775,11 +1775,11 @@ uint32_t imma_hdl_callbk_dispatch_all(IM Arguments : cb - ptr to the IMMA control block immHandle - immsv handle - Return Values : NCSCC_RC_SUCCESS/NCSCC_RC_FAILURE + Return Values : SA_AIS_OK/SA_AIS_ERR_<ERROR> Notes : None ******************************************************************************/ -uint32_t imma_hdl_callbk_dispatch_block(IMMA_CB *cb, SaImmHandleT immHandle) +SaAisErrorT imma_hdl_callbk_dispatch_block(IMMA_CB *cb, SaImmHandleT immHandle) { IMMA_CALLBACK_INFO *callback = NULL; SYSF_MBX *callbk_mbx = NULL; @@ -2020,7 +2020,7 @@ static bool imma_process_callback_info(I callback->params); } } else { - SaAisErrorT error = IMMSV_IMPOSSIBLE_ERROR; + SaAisErrorT error = (SaAisErrorT) IMMSV_IMPOSSIBLE_ERROR; if(!isExtendedNameValid) { if (osaf_is_extended_names_enabled()) { TRACE_3("Object name is too long: %s", osaf_extended_name_borrow(&callback->name)); @@ -2335,21 +2335,21 @@ static bool imma_process_callback_info(I if(localEr == SA_AIS_OK) { attrDataSize = sizeof(SaImmAttrValuesT_2 *) * (noOfAttributes + 1); - attr = calloc(1, attrDataSize); /*alloc-1 */ + attr = (SaImmAttrValuesT_2 **) calloc(1, attrDataSize); /*alloc-1 */ for (; i < noOfAttributes && p; i++, p = p->next) { IMMSV_ATTR_VALUES *q = &(p->n); - attr[i] = calloc(1, sizeof(SaImmAttrValuesT_2)); /*alloc-2 */ - attr[i]->attrName = malloc(q->attrName.size + 1); /*alloc-3 */ + attr[i] = (SaImmAttrValuesT_2 *) calloc(1, sizeof(SaImmAttrValuesT_2)); /*alloc-2 */ + attr[i]->attrName = (char *) malloc(q->attrName.size + 1); /*alloc-3 */ strncpy(attr[i]->attrName, (const char *)q->attrName.buf, q->attrName.size + 1); attr[i]->attrName[q->attrName.size] = 0; /*redundant. */ attr[i]->attrValuesNumber = q->attrValuesNumber; attr[i]->attrValueType = (SaImmValueTypeT)q->attrValueType; if (q->attrValuesNumber) { attr[i]->attrValues = - calloc(q->attrValuesNumber, sizeof(SaImmAttrValueT)); /*alloc-4 */ + (SaImmAttrValueT *) calloc(q->attrValuesNumber, sizeof(SaImmAttrValueT)); /*alloc-4 */ /*alloc-5 */ attr[i]->attrValues[0] = - imma_copyAttrValue3(q->attrValueType, &(q->attrValue)); + imma_copyAttrValue3((SaImmValueTypeT) q->attrValueType, &(q->attrValue)); if (q->attrValuesNumber > 1) { int ix; @@ -2357,7 +2357,7 @@ static bool imma_process_callback_info(I for (ix = 1; ix < q->attrValuesNumber; ++ix) { osafassert(r); attr[i]->attrValues[ix] = /*alloc-5 */ - imma_copyAttrValue3(q->attrValueType, &(r->n)); + imma_copyAttrValue3((SaImmValueTypeT) q->attrValueType, &(r->n)); r = r->next; }//for }//if @@ -2753,12 +2753,12 @@ static bool imma_process_callback_info(I } /*alloc-1 */ - attr = calloc(noOfAttrMods + 1, sizeof(SaImmAttrModificationT_2 *)); + attr = (SaImmAttrModificationT_2 **) calloc(noOfAttrMods + 1, sizeof(SaImmAttrModificationT_2 *)); p = callback->attrMods; for (; i < noOfAttrMods; i++, p = p->next) { - attr[i] = calloc(1, sizeof(SaImmAttrModificationT_2)); /*alloc-2 */ - attr[i]->modType = p->attrModType; + attr[i] = (SaImmAttrModificationT_2 *) calloc(1, sizeof(SaImmAttrModificationT_2)); /*alloc-2 */ + attr[i]->modType = (SaImmAttrModificationTypeT) p->attrModType; attr[i]->modAttr.attrName = p->attrValue.attrName.buf; /* steal/alloc-3 */ p->attrValue.attrName.buf = NULL; @@ -2768,10 +2768,10 @@ static bool imma_process_callback_info(I p->attrValue.attrValueType; if (p->attrValue.attrValuesNumber) { attr[i]->modAttr.attrValues = - calloc(p->attrValue.attrValuesNumber, sizeof(SaImmAttrValueT)); /*alloc-4 */ + (SaImmAttrValueT *) calloc(p->attrValue.attrValuesNumber, sizeof(SaImmAttrValueT)); /*alloc-4 */ /*alloc-5 */ attr[i]->modAttr.attrValues[0] = - imma_copyAttrValue3(p->attrValue.attrValueType, &(p->attrValue.attrValue)); + imma_copyAttrValue3((SaImmValueTypeT) p->attrValue.attrValueType, &(p->attrValue.attrValue)); if (p->attrValue.attrValuesNumber > 1) { int ix; @@ -2779,7 +2779,7 @@ static bool imma_process_callback_info(I for (ix = 1; ix < p->attrValue.attrValuesNumber; ++ix) { osafassert(r); attr[i]->modAttr.attrValues[ix] = - imma_copyAttrValue3(p->attrValue.attrValueType, &(r->n));/*alloc-5 */ + imma_copyAttrValue3((SaImmValueTypeT) p->attrValue.attrValueType, &(r->n));/*alloc-5 */ r = r->next; }//for all extra values @@ -3009,7 +3009,7 @@ static bool imma_process_callback_info(I p = p->next; } osafassert(noOfAttrNames); /*alloc-1 */ - attributeNames = calloc(noOfAttrNames + 1, sizeof(SaImmAttrNameT)); + attributeNames = (SaImmAttrNameT *) calloc(noOfAttrNames + 1, sizeof(SaImmAttrNameT)); p = callback->attrNames; for (; i < noOfAttrNames; i++, p = p->next) { attributeNames[i] = p->name.buf; @@ -3246,9 +3246,9 @@ SaAisErrorT imma_evt_fake_evs(IMMA_CB *c } /* Encode non-flat since we broadcast to unknown receivers. */ - rc = immsv_evt_enc(i_evt, &uba); - - if (rc != NCSCC_RC_SUCCESS) { + proc_rc = immsv_evt_enc(i_evt, &uba); + + if (proc_rc != NCSCC_RC_SUCCESS) { TRACE_2("ERR_LIBRARY: Failed to pre-pack"); rc = SA_AIS_ERR_LIBRARY; goto fail; @@ -3257,7 +3257,7 @@ SaAisErrorT imma_evt_fake_evs(IMMA_CB *c size = uba.ttl; /*NOTE: should check against "payload max-size" */ - tmpData = malloc(size); + tmpData = (char *) malloc(size); data = m_MMGR_DATA_AT_START(uba.start, size, tmpData); memset(&fevs_evt, 0, sizeof(IMMSV_EVT)); diff --git a/osaf/libs/agents/saf/imma/imma_proc.h b/osaf/libs/agents/saf/imma/imma_proc.h --- a/osaf/libs/agents/saf/imma/imma_proc.h +++ b/osaf/libs/agents/saf/imma/imma_proc.h @@ -72,7 +72,7 @@ typedef struct imma_callback_info { } IMMA_CALLBACK_INFO; void imma_process_evt(IMMA_CB *cb, IMMSV_EVT *evt); -uint32_t imma_version_validate(SaVersionT *version); +SaAisErrorT imma_version_validate(SaVersionT *version); uint32_t imma_callback_ipc_init(IMMA_CLIENT_NODE *client_info); void imma_callback_ipc_destroy(IMMA_CLIENT_NODE *client_info); @@ -98,9 +98,9 @@ void imma_proc_free_pointers(IMMA_CB *cb /* callback prototypes */ IMMA_CALLBACK_INFO *imma_callback_ipc_rcv(IMMA_CLIENT_NODE *clnd); -uint32_t imma_hdl_callbk_dispatch_one(IMMA_CB *cb, SaImmHandleT immHandle); -uint32_t imma_hdl_callbk_dispatch_all(IMMA_CB *cb, SaImmHandleT immHandle); -uint32_t imma_hdl_callbk_dispatch_block(IMMA_CB *cb, SaImmHandleT immHandle); +SaAisErrorT imma_hdl_callbk_dispatch_one(IMMA_CB *cb, SaImmHandleT immHandle); +SaAisErrorT imma_hdl_callbk_dispatch_all(IMMA_CB *cb, SaImmHandleT immHandle); +SaAisErrorT imma_hdl_callbk_dispatch_block(IMMA_CB *cb, SaImmHandleT immHandle); /* Admin operation continuation functions */ int imma_popAsyncAdmOpContinuation(IMMA_CB *cb, ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi _______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel