openharmony/security_certificate_manager
1
0
Fork 0
mirror of https://gitee.com/openharmony/security_certificate_manager synced 2024-11-23 07:29:54 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix codecheck

Browse Source 
Signed-off-by: LeadroyaL <leadroyal@qq.com>
Change-Id: Id2bd6be0d1ff6065a0d1e192130ac309c60c411e
This commit is contained in:
LeadroyaL 2024-10-29 08:51:37 +08:00
parent 678a4b20f8
commit 4e75356e90
1 changed files with 15 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
interfaces/kits/cj/src/cj_cert_manager_ffi.c
View File

@ -41,8 +41,11 @@ int32_t FfiCertManagerGetAppCert(const struct CmBlob *keyUri, const uint32_t sto
free(credential.credData.data);
return errCode;
}
// ATTENTION: resource will be released by caller.
// Caller will ensure `retObj` is always not null.
/* ATTENTION:
* 1. Resource will be released by caller.
* 2. strdup may return nullptr, but caller will handle nullptr
* 3. Caller will ensure `retObj` is always not null.
*/
retObj->isExist = credential.isExist;
retObj->type = strdup(credential.type);
retObj->alias = strdup(credential.alias);
@ -104,8 +107,11 @@ int32_t FfiCertManagerGetUserCertList(const uint32_t store, uint32_t *retCount,
return CMR_ERROR_MALLOC_FAIL;
}
*retCount = certificateList.certsCount;
for (int i = 0; i < certificateList.certsCount; ++i) {
// ATTENTION: resource will be released by caller.
for (uint32_t i = 0; i < certificateList.certsCount; ++i) {
/* ATTENTION:
* 1. Resource will be released by caller.
* 2. strdup may return nullptr, but caller will handle nullptr
*/
(*retObj)->uri = strdup(certificateList.certAbstract[i].uri);
(*retObj)->certAlias = strdup(certificateList.certAbstract[i].certAlias);
(*retObj)[i].status = certificateList.certAbstract[i].status;
@ -130,8 +136,11 @@ int32_t FfiCertManagerGetUserCertInfo(const struct CmBlob *certUri, const uint32
free(info.certInfo.data);
return errCode;
}
// ATTENTION: resource will be released by caller.
// Caller will ensure `retObj` is always not null.
/* ATTENTION:
* 1. Resource will be released by caller.
* 2. strdup may return nullptr, but caller will handle nullptr
* 3. Caller will ensure `retObj` is always not null.
*/
retObj->uri = strdup(info.uri);
retObj->certAlias = strdup(info.certAlias);
retObj->status = info.status;