diff --git a/samgr/source/samgr_lite.c b/samgr/source/samgr_lite.c index d7491c5ac51db5782607eadb17a968f984e164b2..012cd4d7d3cef749b27acd51bf4bde3f5ed2277c 100755 --- a/samgr/source/samgr_lite.c +++ b/samgr/source/samgr_lite.c @@ -466,7 +466,7 @@ static void AddTaskPool(ServiceImpl *service, TaskConfig *cfg, const char *name) } if (service->taskPool == NULL) { - HILOG_ERROR(HILOG_MODULE_SAMGR, "Service create taskPool failed!", name, cfg->taskFlags); + HILOG_ERROR(HILOG_MODULE_SAMGR, "Service create taskPool failed!", name, cfg->taskFlags); } } @@ -500,7 +500,7 @@ static void HandleInitRequest(const Request *request, const Response *response) { ServiceImpl *serviceImpl = (ServiceImpl *)request->data; if (serviceImpl == NULL) { - HILOG_ERROR(HILOG_MODULE_SAMGR, "Init service Request:<%d,%d>, Response:<%p,%d>!", + HILOG_ERROR(HILOG_MODULE_SAMGR, "Init service Request:<%d,%u>, Response:<%p,%d>!", request->msgId, request->msgValue, response->data, response->len); return; } diff --git a/samgr/source/task_manager.c b/samgr/source/task_manager.c index e72f35e4e459d1a39e1db6ca7dd5dad738bdc41f..4128bfc05f3f20f86bb5efe86b6733711859d1ba 100755 --- a/samgr/source/task_manager.c +++ b/samgr/source/task_manager.c @@ -45,13 +45,13 @@ TaskPool *SAMGR_CreateFixedTaskPool(const TaskConfig *config, const char *name, MQueueId queueId = (MQueueId)QUEUE_Create(name, sizeof(Exchange), config->queueSize); if (queueId == NULL) { - HILOG_ERROR(HILOG_MODULE_SAMGR, "Create Queue<%s> size:%d failed!", name, config->queueSize); + HILOG_ERROR(HILOG_MODULE_SAMGR, "Create Queue<%s> size:%hu failed!", name, config->queueSize); return NULL; } TaskPool *taskPool = (TaskPool *)SAMGR_Malloc(sizeof(TaskPool) + sizeof(ThreadId) * size); if (taskPool == NULL) { - HILOG_ERROR(HILOG_MODULE_SAMGR, "Create TaskPool<%s> size:%d failed!", name, config->queueSize); + HILOG_ERROR(HILOG_MODULE_SAMGR, "Create TaskPool<%s> size:%hu failed!", name, config->queueSize); QUEUE_Destroy(queueId); return NULL; } @@ -81,7 +81,7 @@ int32 SAMGR_StartTaskPool(TaskPool *pool, const char *name) while (pool->top < pool->size) { register ThreadId threadId = (ThreadId)THREAD_Create(TaskEntry, pool->queueId, &attr); if (threadId == NULL) { - HILOG_ERROR(HILOG_MODULE_SAMGR, "Start Task<%s, %d, %d> failed!", name, pool->stackSize, pool->priority); + HILOG_ERROR(HILOG_MODULE_SAMGR, "Start Task<%s, %hu, %hhu> failed!", name, pool->stackSize, pool->priority); break; } pool->tasks[pool->top] = threadId; diff --git a/samgr_endpoint/source/endpoint.c b/samgr_endpoint/source/endpoint.c index 170894ad19791a39acb8d4508acc636afe8c2e3f..068b9580137610224ab1a802a9b4ae0fb719f48b 100755 --- a/samgr_endpoint/source/endpoint.c +++ b/samgr_endpoint/source/endpoint.c @@ -217,7 +217,7 @@ static int SendGetAllSysCapsRequest(const Endpoint *endpoint, uint32 startIdx, I SvcIdentity samgr = {SAMGR_HANDLE, SAMGR_TOKEN, SAMGR_COOKIE}; int ret = Transact(endpoint->context, samgr, INVALID_INDEX, &req, reply, LITEIPC_FLAG_DEFAULT, (uintptr_t *)replyBuf); - HILOG_DEBUG(HILOG_MODULE_SAMGR, "SendGetAllSysCapsRequest startIdx:%d, ret:%d!", startIdx, ret); + HILOG_DEBUG(HILOG_MODULE_SAMGR, "SendGetAllSysCapsRequest startIdx:%u, ret:%d!", startIdx, ret); return -ret; } @@ -455,7 +455,7 @@ static void HandleIpc(const Request *request, const Response *response) if ((strcmp(router->saName.service, SAMGR_SERVICE) != 0) && !JudgePolicy(uid, (const PolicyTrans *)(router->policy), router->policyNum)) { FreeBuffer(endpoint->context, ipcMsg); - HILOG_ERROR(HILOG_MODULE_SAMGR, "Consumer uid<%d> has no permission to access<%s, %d, %d>!", + HILOG_ERROR(HILOG_MODULE_SAMGR, "Consumer uid<%u> has no permission to access<%s, %d, %d>!", uid, router->saName.service, router->identity.serviceId, router->identity.featureId); return; } diff --git a/samgr_server/source/samgr_server.c b/samgr_server/source/samgr_server.c index 107dd0b2c2a9647c2b5e1e524f51bc494c12fb83..af681ad40536b216368d40def59ac8875b315e6e 100755 --- a/samgr_server/source/samgr_server.c +++ b/samgr_server/source/samgr_server.c @@ -181,7 +181,7 @@ static int ProcEndpoint(SamgrServer *server, int32 option, void *origin, IpcIo * } MUTEX_Unlock(server->mtx); IpcIoPushUint32(reply, handle.handle); - HILOG_INFO(HILOG_MODULE_SAMGR, "Register Endpoint<%d, %d, %d>", handle.pid, handle.handle, handle.deadId); + HILOG_INFO(HILOG_MODULE_SAMGR, "Register Endpoint<%d, %u, %u>", handle.pid, handle.handle, handle.deadId); return EC_SUCCESS; } @@ -229,7 +229,7 @@ static int32 ProcPutFeature(SamgrServer *server, const void *origin, IpcIo *req, ret = SASTORA_Save(&server->store, service, feature, identity); MUTEX_Unlock(server->mtx); - HILOG_DEBUG(HILOG_MODULE_SAMGR, "Register Feature<%s, %s> pid<%d>, id<%d, %d> ret:%d", + HILOG_DEBUG(HILOG_MODULE_SAMGR, "Register Feature<%s, %s> pid<%d>, id<%u, %u> ret:%d", service, feature, pid, identity->handle, identity->token, ret); TransmitPolicy(ret, identity, reply, policy, policyNum); SAMGR_Free(policy); @@ -294,7 +294,7 @@ static int32 ProcGetFeature(SamgrServer *server, const void *origin, IpcIo *req, *identity = SASTORA_Find(&server->store, service, feature); if (identity->handle == INVALID_INDEX) { MUTEX_Unlock(server->mtx); - HILOG_DEBUG(HILOG_MODULE_SAMGR, "Cannot Find Feature<%s, %s> id<%d, %d> ret:%d", + HILOG_DEBUG(HILOG_MODULE_SAMGR, "Cannot Find Feature<%s, %s> id<%u, %u> ret:%d", service, feature, identity->handle, identity->token, EC_NOSERVICE); return EC_NOSERVICE; } @@ -440,7 +440,7 @@ void ProcGetAllSysCap(const SamgrServer *server, IpcIo *req, IpcIo *reply) } int32 nextRequestIdx = startIdx; int32 replyNum = GetReplyNumAndNextReqIdx(sysCapablitys, startIdx, &nextRequestIdx); - HILOG_DEBUG(HILOG_MODULE_SAMGR, "ProcGetAllSysCap replyNum: %d, size: %d, startIdx: %d, nextRequestIdx: %d", + HILOG_DEBUG(HILOG_MODULE_SAMGR, "ProcGetAllSysCap replyNum: %d, size: %d, startIdx: %u, nextRequestIdx: %d", replyNum, size, startIdx, nextRequestIdx); IpcIoPushInt32(reply, EC_SUCCESS); // indicate is the last reply