mirror of
https://github.com/openharmony/ability_ability_runtime.git
synced 2026-08-24 22:21:36 -04:00
!17910 merge cr2 into master
fix_memory2 Created-by: RuiChen_01 Commit-by: RuiChen_01 Merged-by: openharmony_ci Description: **IssueNo**: **Description**: **稳定性自检：** | 自检项 | 自检结果 | | ------------------------------------------------------------ | -------- | | 涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发 | | | 成员变量进行赋值或创建需要排查并发 | | | 谨慎在lambda表达式中使用引用捕获 | | | 谨慎在未经拷贝的情况下使用外部传入的string、C字符串 | | | map\vector\list\set等stl模板类使用时需要排查并发 | | | 谨慎考虑加锁范围 | | | 在IPC通信中谨慎使用同步通信方式 | | | 禁止传递this指针至其他模块或线程（特别是eventhandler任务） | | | 禁止将外部传入的裸指针在内部直接构造智能指针 | | | 禁止多个独立创建的智能指针管理同一地址 | | | 禁止在析构函数中抛异步任务 | | | 禁止js对象在非js线程（例如在IPC线程）创建、使用或销毁 | | | 禁止在对外接口中未经判空直接使用外部传入的指针 | | | 禁止接口返回局部变量引用 | | | 禁止在信号函数中加锁 | | | 禁止在关键流程（SA启动、应用启动等主流程）执行耗时的操作 | | | 禁止将同一个cpp编译在不同的so中 | | **安全编码自检：** | 自检项 | 自检结果 | | -------------------------------------------------------------- | -------- | | 裸指针避免通过隐式转换构造为sptr | | | json对象在取值之前必须先判断类型，避免类型不匹配 | | | 序列化时必须对传入的数组大小进行校验，避免出现超大数组 | | | 避免使用未明确位宽的整型，选择使用int8_t、uint8_t等类型 | | | 外部传入的路径要做规范化校验，对路径中的.、..、../等特殊字符严格校验 | | | 指针变量、表示资源描述符的变量、bool变量必须赋初值 | | | readParcelable获取的对象使用前需要判空 | | | 分配和释放内存的函数需要成对出现 | | | 申请内存后异常退出前需要及时进行内存释放 | | | 内存申请前必须对内存大小进行合法性校验 | | | 内存分配后必须判断是否成功 | | | 禁止使用realloc、alloca函数 | | | 禁止打印文件路径、口令等敏感信息，如有需要，使用private修饰 | | | 禁止打印内存地址 | | | 整数之间运算时必须严格检查，确保不会出现溢出、反转、除0 | | | 禁止对有符号整数进行位操作符运算 | | | 禁止对指针进行逻辑或位运算 | | | 循环次数如果收外部数据控制，需要检验其合法性 | | | 禁止使用内存操作类危险函数，需要使用安全函数 | | | 谨慎使用不可重入函数 | | | 必须检查安全函数的返回值，并进行正确处理 | | | 禁止仅通过TokenType类型判断绕过权限校验 | | **TDD Result**: **XTS Result**: ### 是否已执行L0用例 - [ ] 已验证 - [ ] 不涉及。如不涉及，请写明理由 See merge request: openharmony/ability_ability_runtime!17910
This commit is contained in:
openharmony_ci
@@ -368,6 +368,7 @@ private:
|
||||
TAG_LOGD(AAFwkTag::ABILITYMGR, "called");
|
||||
NapiAsyncTask::CompleteCallback complete =
|
||||
[](napi_env env, NapiAsyncTask &task, int32_t status) {
|
||||
HandleScope handleScope(env);
|
||||
std::vector<AAFwk::AbilityRunningInfo> infos;
|
||||
auto errcode = AbilityManagerClient::GetInstance()->GetAbilityRunningInfos(infos);
|
||||
if (errcode == 0) {
|
||||
@@ -410,6 +411,7 @@ private:
|
||||
|
||||
NapiAsyncTask::CompleteCallback complete =
|
||||
[upperLimit](napi_env env, NapiAsyncTask &task, int32_t status) {
|
||||
HandleScope handleScope(env);
|
||||
std::vector<AAFwk::ExtensionRunningInfo> infos;
|
||||
auto errcode = AbilityManagerClient::GetInstance()->GetExtensionRunningInfos(upperLimit, infos);
|
||||
if (errcode == 0) {
|
||||
@@ -500,6 +502,7 @@ private:
|
||||
#endif
|
||||
NapiAsyncTask::CompleteCallback complete =
|
||||
[](napi_env env, NapiAsyncTask &task, int32_t status) {
|
||||
HandleScope handleScope(env);
|
||||
AppExecFwk::ElementName elementName = AbilityManagerClient::GetInstance()->GetTopAbility();
|
||||
#ifdef ENABLE_ERRCOE
|
||||
task.ResolveWithNoError(env, CreateJsElementName(env, elementName));
|
||||
|
||||
@@ -49,6 +49,7 @@ napi_value CreateJSToken(napi_env env, const sptr<IRemoteObject> target)
|
||||
|
||||
napi_value CreateJsAbilityRunningInfoArray(napi_env env, const std::vector<AAFwk::AbilityRunningInfo> &infos)
|
||||
{
|
||||
HandleEscape handleEscape(env);
|
||||
napi_value arrayValue = nullptr;
|
||||
napi_status createStatus = napi_create_array_with_length(env, infos.size(), &arrayValue);
|
||||
if (createStatus != napi_ok || arrayValue == nullptr) {
|
||||
@@ -59,7 +60,7 @@ napi_value CreateJsAbilityRunningInfoArray(napi_env env, const std::vector<AAFwk
|
||||
for (const auto &runningInfo : infos) {
|
||||
napi_set_element(env, arrayValue, index++, CreateJsAbilityRunningInfo(env, runningInfo));
|
||||
}
|
||||
return arrayValue;
|
||||
return handleEscape.Escape(arrayValue);
|
||||
}
|
||||
|
||||
napi_value CreateJsElementName(napi_env env, const AppExecFwk::ElementName &elementName)
|
||||
@@ -69,6 +70,7 @@ napi_value CreateJsElementName(napi_env env, const AppExecFwk::ElementName &elem
|
||||
|
||||
napi_value CreateJsExtensionRunningInfoArray(napi_env env, const std::vector<AAFwk::ExtensionRunningInfo> &infos)
|
||||
{
|
||||
HandleEscape handleEscape(env);
|
||||
napi_value arrayValue = nullptr;
|
||||
napi_status createStatus = napi_create_array_with_length(env, infos.size(), &arrayValue);
|
||||
if (createStatus != napi_ok || arrayValue == nullptr) {
|
||||
@@ -79,11 +81,12 @@ napi_value CreateJsExtensionRunningInfoArray(napi_env env, const std::vector<AAF
|
||||
for (const auto &runningInfo : infos) {
|
||||
napi_set_element(env, arrayValue, index++, CreateJsExtensionRunningInfo(env, runningInfo));
|
||||
}
|
||||
return arrayValue;
|
||||
return handleEscape.Escape(arrayValue);
|
||||
}
|
||||
|
||||
napi_value CreateJsAbilityRunningInfo(napi_env env, const AAFwk::AbilityRunningInfo &info)
|
||||
{
|
||||
HandleEscape handleEscape(env);
|
||||
napi_value objValue = nullptr;
|
||||
napi_status createStatus = napi_create_object(env, &objValue);
|
||||
if (createStatus != napi_ok || objValue == nullptr) {
|
||||
@@ -98,11 +101,12 @@ napi_value CreateJsAbilityRunningInfo(napi_env env, const AAFwk::AbilityRunningI
|
||||
napi_set_named_property(env, objValue, "processName", CreateJsValue(env, info.processName));
|
||||
napi_set_named_property(env, objValue, "startTime", CreateJsValue(env, info.startTime));
|
||||
napi_set_named_property(env, objValue, "abilityState", CreateJsValue(env, info.abilityState));
|
||||
return objValue;
|
||||
return handleEscape.Escape(objValue);
|
||||
}
|
||||
|
||||
napi_value CreateJsExtensionRunningInfo(napi_env env, const AAFwk::ExtensionRunningInfo &info)
|
||||
{
|
||||
HandleEscape handleEscape(env);
|
||||
napi_value objValue = nullptr;
|
||||
napi_status createStatus = napi_create_object(env, &objValue);
|
||||
if (createStatus != napi_ok || objValue == nullptr) {
|
||||
@@ -118,7 +122,7 @@ napi_value CreateJsExtensionRunningInfo(napi_env env, const AAFwk::ExtensionRunn
|
||||
napi_set_named_property(env, objValue, "processName", CreateJsValue(env, info.processName));
|
||||
napi_set_named_property(env, objValue, "startTime", CreateJsValue(env, info.startTime));
|
||||
napi_set_named_property(env, objValue, "clientPackage", CreateNativeArray(env, info.clientPackage));
|
||||
return objValue;
|
||||
return handleEscape.Escape(objValue);
|
||||
}
|
||||
|
||||
napi_value AbilityStateInit(napi_env env)
|
||||
|
||||
Reference in New Issue
Block a user