mirror of
https://github.com/openharmony/ability_ability_runtime.git
synced 2026-08-24 22:21:36 -04:00
!18867 merge fix/startup-config-null-crash into master
fix: 修复 startupManager.run 不传 config 时 ANI undefined 导致的崩溃 Created-by: zhang_hao_zheng Commit-by: zhang_hao_zheng Merged-by: openharmony_ci Description: **IssueNo**: https://gitcode.com/openharmony/ability_ability_runtime/issues/14704 **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**: passs ### 是否已执行L0用例 - [ ] 已验证 - [√] 不涉及。如不涉及，请写明理由 See merge request: openharmony/ability_ability_runtime!18867
This commit is contained in:
openharmony_ci
@@ -211,10 +211,15 @@ int32_t ETSStartupManager::GetStartupTaskManager(ani_env *env, ani_object startu
|
||||
return ERR_STARTUP_INVALID_VALUE;
|
||||
}
|
||||
std::shared_ptr<StartupConfig> config;
|
||||
int32_t result = GetConfig(env, startupConfig, config);
|
||||
if (result != ERR_OK) {
|
||||
TAG_LOGE(AAFwkTag::STARTUP, "get config failed");
|
||||
return result;
|
||||
ani_boolean isConfigUndefined = ANI_TRUE;
|
||||
env->Reference_IsUndefined(reinterpret_cast<ani_ref>(startupConfig), &isConfigUndefined);
|
||||
int32_t result = ERR_OK;
|
||||
if (!isConfigUndefined) {
|
||||
result = GetConfig(env, startupConfig, config);
|
||||
if (result != ERR_OK) {
|
||||
TAG_LOGE(AAFwkTag::STARTUP, "get config failed");
|
||||
return result;
|
||||
}
|
||||
}
|
||||
bool supportFeatureModule = isDefaultContext != ANI_TRUE;
|
||||
result = DelayedSingleton<StartupManager>::GetInstance()->BuildAppStartupTaskManager(dependencies,
|
||||
|
||||
@@ -97,6 +97,12 @@ int32_t ETSStartupConfig::Init(ani_object config)
|
||||
TAG_LOGE(AAFwkTag::STARTUP, "null config");
|
||||
return ERR_STARTUP_INTERNAL_ERROR;
|
||||
}
|
||||
ani_boolean isUndefined = ANI_TRUE;
|
||||
env->Reference_IsUndefined(reinterpret_cast<ani_ref>(config), &isUndefined);
|
||||
if (isUndefined) {
|
||||
TAG_LOGE(AAFwkTag::STARTUP, "undefined config");
|
||||
return ERR_STARTUP_INTERNAL_ERROR;
|
||||
}
|
||||
|
||||
InitAwaitTimeout(env, config);
|
||||
InitListener(env, config);
|
||||
@@ -141,6 +147,10 @@ bool ETSStartupConfig::GetTimeoutMs(ani_env *env, ani_object config, int32_t &ti
|
||||
TAG_LOGE(AAFwkTag::STARTUP, "null env");
|
||||
return false;
|
||||
}
|
||||
if (config == nullptr) {
|
||||
TAG_LOGE(AAFwkTag::STARTUP, "null config");
|
||||
return false;
|
||||
}
|
||||
ani_status status = env->FindClass("@ohos.app.appstartup.StartupConfig.StartupConfig", &cls);
|
||||
if (status != ANI_OK || cls == nullptr) {
|
||||
TAG_LOGE(AAFwkTag::STARTUP, "Findclass failed, status: %{public}d", status);
|
||||
|
||||
@@ -190,7 +190,9 @@ int32_t StartupManager::BuildAutoAppStartupTaskManager(std::shared_ptr<AAFwk::Wa
|
||||
TAG_LOGD(AAFwkTag::STARTUP, "autoStartupTasksManager build, id: %{public}u, tasks num: %{public}zu",
|
||||
startupTaskManagerId, autoStartupTasks.size());
|
||||
startupTaskManager = std::make_shared<StartupTaskManager>(startupTaskManagerId, autoStartupTasks);
|
||||
startupTaskManager->SetConfig(startupConfig);
|
||||
if (startupConfig != nullptr) {
|
||||
startupTaskManager->SetConfig(startupConfig);
|
||||
}
|
||||
startupTaskManagerMap_.emplace(startupTaskManagerId, startupTaskManager);
|
||||
startupTaskManagerId++;
|
||||
return ERR_OK;
|
||||
@@ -288,7 +290,9 @@ int32_t StartupManager::BuildAppStartupTaskManager(const std::vector<std::string
|
||||
TAG_LOGD(AAFwkTag::STARTUP, "startupTasksManager build, id: %{public}u, tasks num: %{public}zu",
|
||||
startupTaskManagerId, currentStartupTasks.size());
|
||||
startupTaskManager = std::make_shared<StartupTaskManager>(startupTaskManagerId, currentStartupTasks);
|
||||
startupTaskManager->SetConfig(defaultConfig_);
|
||||
if (defaultConfig_ != nullptr) {
|
||||
startupTaskManager->SetConfig(defaultConfig_);
|
||||
}
|
||||
startupTaskManagerMap_.emplace(startupTaskManagerId, startupTaskManager);
|
||||
startupTaskManagerId++;
|
||||
return ERR_OK;
|
||||
|
||||
Reference in New Issue
Block a user