| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
fix/double free Signed-off-by: Alang <gaoyulang@huawei.com> Change-Id: I8927a620a00b7d9c9cdaffcee0039a907f8428a4 | 5 天前 | |
perf(build): enable size_optimize_config on 130 SO targets Add a new services/common:size_optimize_config (same flags as optimize_config minus the explicit -flto=full, so the toolchain's global ThinLTO mode is preserved) and apply it to the 130 ohos_shared_library targets that previously shipped without any optimization config, so unreferenced functions/data are dropped at link time and ROM shrinks without overriding the global LTO policy. -Wl,--gc-sections is an ELF-only linker flag: Apple ld rejects it, so both optimize_config and size_optimize_config now add it only when current_os != "mac", keeping the darwin SDK host build (which links dylibs with -dead_strip) working. Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (glm-5.3) <ai@local> | 15 天前 | |
applicationContext在有更新时复用缓存对象而非创建新对象 Co-Authored-By: Agent Signed-off-by: 任国军 <renguojun1@h-partners.com> | 3 天前 | |
!20343 merge fix_delegator into master fix getAbilityState Created-by: ProfaneAria Commit-by: 任国军 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用例 - [ ] 已验证 - [ ] 不涉及。如不涉及，请写明理由 ### AI检视评分（使用本地代码检视skills扫描）: # 代码检视报告 — js_ability_delegator / OnGetAbilityState（Round 1 / 最新提交） > 统一报告由 codecheck 工作台生成，**用于门禁管控**。所有 codecheck 报告（含 orchestrator 合并出的统一报告、单 scanner 直接产出的统一报告）必须遵循本模板：章节顺序、字段名、报告元数据块、评分与门禁规则均为**固定格式**，跨报告保持一致，便于门禁脚本解析与历史对比。 > 生成入口：[ README.md](skills/codecheck/README.md) → Step 5；合并逻辑见 [orchestrator/SKILL.md](skills/codecheck/orchestrator/SKILL.md)。 > 权威评分与门禁规则为**通用规则**，不在输出报告中呈现；生成时必须按 [conventions.md](skills/codecheck/conventions.md) §7（等级归一化）/ §8（必检维度）/ §9（评分与决策矩阵）计算，不得自创分值。 --- ## 报告元数据 > **门禁脚本只读取本 YAML 块**。字段名与取值域为固定合约，禁止改名、增删或自定义取值。人工阅读部分从「1. 门禁结论」开始。 <!-- codecheck-report-metadata:start --> yaml codecheck_report: schema_version: "1.0" scope: "js_ability_delegator (OnGetAbilityState)" round: 1 commit_id: "f3e217c1f6fe6776d2c40fb56234a54cd48373eb" change_id: "N/A" report_id: "f3e217c1-R1" date: "2026-09-01" gate_decision: "approve" risk_level: "low" score: 93 dimensions_required: ["security-scanner", "logic-scanner", "api-scanner"] dimensions_executed: ["security-scanner", "logic-scanner", "api-scanner"] findings_total: 2 findings_by_severity: {P0: 0, P1: 0, P2: 1, P3: 1} gate_blockers: [] must_fix: [] followups: ["API-01", "LOG-01"] <!-- codecheck-report-metadata:end --> --- ## 1. 门禁结论 | 项目 | 结论 | |---|---| | 决策 | **approve** | | 风险等级 | 🟢 low | | 评分 | **93/100** | | 阻塞项 | 无 | | 必须修复（P0/P1） | 0 项 | | 建议跟进（P2/P3） | 2 项 | **一句话结论**：本次将 OnGetAbilityState 三处错误返回统一为 ThrowJsError 模式，与文件内其他 23+ 同步函数保持一致，属一致性修复；参数校验前置、无内存/并发/IPC 安全风险；无 P0/P1，可上库。 --- ## 2. 扣分原因（仅 gate_decision=block 时呈现；approve/conditional/insufficient 时本节省略） > 本报告 gate_decision=approve，本节省略。 --- ## 3. 必须立即处理（P0/P1） **无。** 详情见第 6 节。 --- ## 4. 建议本轮或下一补档处理（P2/P3） | ID | 优先级 | 问题 | 建议行动 | 排期 | |---|---|---|---|---| | API-01 | P2 | getAbilityState 资料仅声明 getAbilityState(ability: UIAbility): int，未描述错误抛出行为（401/16000100） | 补充接口文档说明错误抛出场景与错误码；补充错误路径测试用例覆盖 | 下一补档 | | LOG-01 | P3 | null delegator 路径返回值由 null 变为 undefined（ENABLE_ERRCODE 未定义构建下，ThrowJsError 固定 return CreateJsUndefined） | 如需严格保留 null 语义可在该路径显式返回 CreateJsNull；当前影响极小，登记观察 | 跟踪 | --- ## 5. 分维度速览 | 维度 | 结果 | 关键说明 | |---|---|---| | security-scanner | 通过 | 无崩溃/UAF/OOM/死锁/权限风险；argc 越界已前置防护（:792 先检查再访问 argv[0]，:798）；ThrowJsError 受 ENABLE_ERRCODE 宏控制，napi_throw 为受控 JS 异常非 native crash；无内存/并发/IPC 面 | | logic-scanner | 通过（正面修复） | 本次使 OnGetAbilityState 与文件内其他 23+ 同步函数错误处理一致（26 处已用 ThrowJsError）；错误码选择合理（401 参数错误 / 16000100 运行时失败）；三处校验顺序合理 argc→parse→delegator null；存在 null→undefined 细微差异（P3，ThrowJsError 模式固有，非本次引入） | | api-scanner | 跟进 | API 签名未变（仍 (ability: UIAbility): int）；资料 AbilityDelegator.ets:90,160 未描述 throw 行为（pre-existing，文件同类函数同此情况）；无错误路径测试覆盖（ability_delegator 测试通常在测试框架自身） | --- ## 6. 关键发现详情 > 无 P0/P1。以下为 P2/P3 finding 卡片。 ### [API-01] getAbilityState 资料未描述错误抛出行为 (P2, scanner=api-scanner) - **位置**：frameworks/ets/ets/application/AbilityDelegator.ets:90（接口声明）与 :160（native 声明） - **触发路径**：调用方传参不足 / ability 解析失败 / delegator 为空 → OnGetAbilityState（js_ability_delegator.cpp:794/800/806）返回 ThrowJsError(env, 401|16000100, ...) → ENABLE_ERRCODE 已定义时 ThrowJsError（:72-80）调用 napi_throw（:77）抛 JS 异常 - **影响**：接口文档与实现不一致——资料仅声明返回 int，未说明错误路径会抛 401/16000100。调用方无法从文档预知错误处理方式。属 pre-existing（文件其他同类函数同此情况），本次使 getAbilityState 加入既有模式，未恶化。 - **证据**： - 资料：AbilityDelegator.ets:90 getAbilityState(ability: UIAbility): int;，无 [@throws](https://gitcode.com/throws) / 错误码说明 - 实现：js_ability_delegator.cpp:794 return ThrowJsError(env, INCORRECT_PARAMETERS, "Parameter error. Too few parameters."); - 错误码：js_ability_delegator.cpp:59 INCORRECT_PARAMETERS = 401；:67 COMMON_FAILED = 16000100（ENABLE_ERRCODE 已定义） - throw 机制：:77 napi_throw(env, error);（ENABLE_ERRCODE 已定义分支） - **建议**：在接口资料补充错误抛出场景（参数不足→401、delegator 为空→16000100）与对应错误码；补充错误路径的测试用例覆盖。不阻塞本次上库。 ### [LOG-01] null delegator 路径返回值 null→undefined 细微差异 (P3, scanner=logic-scanner) - **位置**：frameworks/js/napi/app/ability_delegator/js_ability_delegator.cpp:806 - **触发路径**：delegator 为空（:804）→ 原 return CreateJsNull(env) 改为 return ThrowJsError(env, COMMON_FAILED, ...) → ENABLE_ERRCODE 未定义时 ThrowJsError 不走 throw 分支，固定 return CreateJsUndefined(env)（:79） - **影响**：ENABLE_ERRCODE 未定义构建下，该路径返回值由 null 变为 undefined。null 与 undefined 在 JS 中均 falsy，调用方极少用 === null 严格区分，且 ability_delegator 为测试工具、仅错误路径触发，影响极小。此差异是 ThrowJsError 模式固有特性（:79 固定 return CreateJsUndefined），非本次引入。 - **证据**： - 改动：js_ability_delegator.cpp:806 return ThrowJsError(env, COMMON_FAILED, "Calling GetAbilityState failed."); - ThrowJsError 实现：:72-80，#ifdef ENABLE_ERRCODE 分支才 napi_throw，:79 无条件 return CreateJsUndefined(env) - 文件一致性：26 处 ThrowJsError 使用均同此特性 - **建议**：如需严格保留 null 语义，可在 ENABLE_ERRCODE 未定义时该路径显式返回 CreateJsNull；当前影响极小，登记为观察项即可。不阻塞上库。 See merge request: openharmony/ability_ability_runtime!20343 | 4 天前 | |
applicationContext在有更新时复用缓存对象而非创建新对象 Co-Authored-By: Agent Signed-off-by: 任国军 <renguojun1@h-partners.com> | 3 天前 | |
perf(build): enable size_optimize_config on 130 SO targets Add a new services/common:size_optimize_config (same flags as optimize_config minus the explicit -flto=full, so the toolchain's global ThinLTO mode is preserved) and apply it to the 130 ohos_shared_library targets that previously shipped without any optimization config, so unreferenced functions/data are dropped at link time and ROM shrinks without overriding the global LTO policy. -Wl,--gc-sections is an ELF-only linker flag: Apple ld rejects it, so both optimize_config and size_optimize_config now add it only when current_os != "mac", keeping the darwin SDK host build (which links dylibs with -dead_strip) working. Co-Authored-By: Agent Signed-off-by: RuiChen_01 <chenrui193@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: claude (glm-5.3) <ai@local> | 15 天前 |