| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
!20296 merge pc into master pc告警修改 Created-by: zhangyuhang72 Commit-by: zhangyuhang72 Merged-by: openharmony_ci Description: https://gitcode.com/openharmony/ability_ability_runtime/issues/16061 # 代码检视报告 — commit 0358babd "pc告警修改"（Round 3 / 最新提交） > 统一报告由 codecheck 工作台生成，**用于门禁管控**。Round 3 在 Round 2 修复 3 项发现后重检。 --- ## 报告元数据 <!-- codecheck-report-metadata:start --> yaml codecheck_report: schema_version: "1.0" scope: "commit-0358babd" round: 3 commit_id: "0358babd259cba46ca71418fd870edffd2730262" change_id: "Ibe1d2c17709d8e86f3a4e422935674ce9c28248c" report_id: "Ibe1d2c17709d8e86f3a4e422935674ce9c28248c-R3" date: "2026-08-25" gate_decision: "approve" risk_level: "low" score: 95 dimensions_required: ["api-scanner", "security-scanner", "logic-scanner"] dimensions_executed: ["api-scanner", "security-scanner", "logic-scanner"] findings_total: 1 findings_by_severity: {P0: 0, P1: 0, P2: 1, P3: 0} gate_blockers: [] must_fix: [] followups: ["API-04/SEC-01b"] <!-- codecheck-report-metadata:end --> --- ## 1. 门禁结论 | 项目 | 结论 | |---|---| | 决策 | **approve** | | 风险等级 | :large_green_circle: low | | 评分 | **95/100**（R1: 73 → R2: 83 → R3: 95） | | 阻塞项 | 无 | | 必须修复（P0/P1） | 0 项 | | 建议跟进（P2/P3） | 1 项 | **一句话结论**：Round 2 的 4 项发现中 3 项已修复（4 函数遗漏判空、ANI 侧 bundleName 不一致、callback 判空但无 env 判空）；仅剩 API-04/SEC-01b（AsyncCallback 工具层判空移除致跨文件缺口）为用户有意保留的已知项，评分 95。 --- ## 3. 必须立即处理（P0/P1） **无。** --- ## 4. 建议跟进（P2/P3） | ID | 优先级 | 问题 | 建议行动 | 排期 | |---|---|---|---|---| | API-04/SEC-01b | P2 | AsyncCallback 无工具层 call 判空，23 个 ANI 文件 200+ 调用点无入口判空 | **用户有意保留**。如需消除：恢复 AsyncCallback 两个重载的 call == nullptr 判空（一行改动覆盖全部 200+ 调用点） | 跟踪 | --- ## 5. 分维度速览 | 维度 | 结果 | 关键说明 | |---|---|---| | security-scanner | 通过 | Round 2 的 SEC-01a（4 函数遗漏判空）已修复（callback 判空 27→31）；SEC-01b（跨文件缺口）为用户保留项 | | logic-scanner | 通过 | LOG-R2-01（4 函数 callback 判空但无 env 判空）已修复——4 函数均补 env 判空 | | api-scanner | 通过 | API-01（4 函数遗漏）已修复；API-05（ANI 侧 bundleName.empty() 不一致）已修复（7→10）；API-04 为用户保留项 | --- ## 6. 关键发现详情 ### [API-04/SEC-01b] AsyncCallback 无工具层 call 判空，200+ 调用点无入口判空 (P2, scanner=security+api) - **位置**：frameworks/ets/ani/ani_common/src/ani_common_util.cpp:572, 606（两个重载均无 call 判空）+ 23 个 ANI 文件 - **触发路径**：23 个 ANI 文件（如 ets_agent_manager.cpp、ets_ability_context.cpp、ets_ability_manager.cpp 等）的 200+ AsyncCallback 调用点均未在上游判空 callback → 若任一收到 nullptr callback → 崩溃 - **影响**：大面积 null-callback 暴露面。ANI 运行时正常不产生 nullptr ani_object，触发概率低，但 200+ 调用点使聚合风险不可忽视。ets_app_manager.cpp 已通过 31 个入口判空覆盖（fail-fast），其余 23 个文件无入口判空 - **证据**：ani_common_util.cpp 两个 AsyncCallback 重载（:572, :606）仅检 env 不检 call；23 个 ANI 文件 200+ 调用点均无 callback == nullptr 入口判空 - **建议**：恢复 AsyncCallback 工具层 call == nullptr 判空（两个重载均加），以最小改动覆盖全部 200+ 调用点。ets_app_manager.cpp 的 31 个入口判空可保留（fail-fast），两者并存不冲突 - **当前状态**：**用户有意保留**——基于"入口判空覆盖本文件、ANI 运行时不产生 nullptr"的判断。此为已知风险项，非阻塞 --- ## 附：Round 2 → Round 3 修复验证 | Round 2 ID | 等级 | 修复内容 | 验证 | |-----------|------|---------|------| | API-01/SEC-01a | P2 | 4 函数补 callback 判空（GetRunningProcessInfoByBundleNameAndUserId、GetRunningProcessInformationByBundleType、NativeIsSharedBundleRunning、NativeSetKeepAliveForBundle） | callback null ptr 计数 27→31 ✓ | | API-05 | P2 | 3 函数补 bundleName.empty()（KillProcessesByBundleNameInner、KillProcessWithAccountInner、IsApplicationRunning） | bundleName.empty() 计数 7→10 ✓ | | LOG-R2-01 | P3 | 4 函数补 env 判空（GetRunningProcessInfoByBundleName、OnOffInner、NativeKillProcessesByBundleName、NativeKillProcessWithAccount） | 4 函数均有 env==nullptr + callback==nullptr ✓ | See merge request: openharmony/ability_ability_runtime!20296 | 10 天前 | |
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 天前 | |
!20308 merge set into master set pid Created-by: xhz-sz Commit-by: xhz-sz Merged-by: openharmony_ci Description: **IssueNo**: https://gitcode.com/openharmony/ability_ability_runtime/issues/16071 **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扫描）: # 代码检视报告 — PR #20308 set pid（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: "PR-20308/frameworks/ets/ani" round: 1 commit_id: "09810552fca5a180b130bf70c3d8e72f929ec648" change_id: "N/A" report_id: "09810552-R1" date: "2026-08-26" gate_decision: "approve" risk_level: "low" score: 94 dimensions_required: ["security-scanner", "logic-scanner", "api-scanner"] dimensions_executed: ["security-scanner", "logic-scanner", "api-scanner"] findings_total: 3 findings_by_severity: {P0: 0, P1: 0, P2: 0, P3: 3} gate_blockers: [] must_fix: [] followups: ["LOG-01", "LOG-02", "LOG-03"] <!-- codecheck-report-metadata:end --> --- ## 1. 门禁结论 | 项目 | 结论 | |---|---| | 决策 | **approve** | | 风险等级 | 🟢 low | | 评分 | **94/100** | | 阻塞项 | 无 | | 必须修复（P0/P1） | 0 项 | | 建议跟进（P2/P3） | 3 项 | **一句话结论**：PR 在 ETS/ANI 侧 3 个文件中新增 SetCallerPid 镜像 JS 侧行为，改动范围小且模式一致；security 与 logic 双维度均无 PR 引入的缺陷，3 项 P3 均为预先存在的 ETS↔JS 对等差距或冗余代码，建议作为 follow-up 跟踪。 --- ## 3. 必须立即处理（P0/P1） **无。** --- ## 4. 建议本轮或下一补档处理（P2/P3） | ID | 优先级 | 问题 | 建议行动 | 排期 | |---|---|---|---|---| | LOG-01 | P3 | ETS CreateUIWindow 缺少 JS 对等的 SetDensity/SetIsDensityFollowHost/SetDisplayId/SetConstrainedModal/SetStartModalExtensionTimeStamp | 后续补齐 ETS CreateUIWindow 的全部 option 设置，达到与 JS CreateUIWindow 完全对等 | 下一补档 | | LOG-02 | P3 | ETS CreateWindowOption 缺少 JS 对等的 launchTimestamp 计算 + SetStartModalExtensionTimeStamp | 后续将 UIEXTENSION_LAUNCH_TIMESTAMP_HIGH/LOW → launchTimestamp 逻辑移植到 ETS CreateWindowOption | 下一补档 | | LOG-03 | P3 | HandleSessionCreate 中 sessionInfo 判空重复（lines 716-718 与 722-724 相同检查） | 清理重复的判空块 | 下一补档 | --- ## 5. 分维度速览 | 维度 | 结果 | 关键说明 | |---|---|---| | security-scanner | ✅ 通过 | 11 项检查全部 clean/N/A；sharedWant 判空已由 PR 补充（ets_auto_fill_extension.cpp:460-463），want 为 C++ 引用不可为空，无新增死锁/泄漏/反序列化面 | | logic-scanner | ✅ 通过（3 项 P3 follow-up） | PR 引入的逻辑正确、一致：所有调用方已同步更新签名，GetIntParam 默认 -1 与 JS 侧逐字节一致，锁内调用无非阻塞风险；3 项 P3 为预先存在的 ETS↔JS 对等差距 | | api-scanner | N/A | 变更仅涉及 private 方法签名（CreateWindowOption/CreateUIWindow/CreateNewWindow），无公共 API 签名变更；ANI 为静态语言实现，api-scanner 默认跳过 | --- ## 6. 关键发现详情 ### [LOG-01] ETS CreateUIWindow 缺少多个 option 设置（与 JS 对等差距） (P3, scanner=logic-scanner) - **位置**：frameworks/ets/ani/ui_extension_ability/src/ets_ui_extension.cpp:636-658 vs frameworks/native/ability/native/ui_extension_ability/js_ui_extension.cpp:690-730 - **触发路径**：ETS UIExtension 窗口创建时，CreateUIWindow 未调用 SetDensity/SetIsDensityFollowHost/SetDisplayId/SetConstrainedModal/SetStartModalExtensionTimeStamp，而 JS 对等函数调用了这些 setter。 - **影响**：ETS UIExtension 窗口缺少 density、displayId、modal 约束、launch timestamp 等配置，两种运行时行为存在差异。此为预先存在的差距，PR 仅缩小（新增 SetCallerPid）但未完全消除。 - **证据**：JS CreateUIWindow（js_ui_extension.cpp:709-727）调用 6 个 ETS 缺失的 setter；ETS CreateUIWindow（ets_ui_extension.cpp:648-655）仅设置到 SetUIExtensionUsage + SetCallerPid。 - **建议**：后续补齐 ETS CreateUIWindow 全部 option 设置以达到 JS 对等。 ### [LOG-02] ETS CreateWindowOption 缺少 launchTimestamp 逻辑（与 JS 对等差距） (P3, scanner=logic-scanner) - **位置**：frameworks/ets/ani/ui_extension_base/src/ets_ui_extension_base.cpp:683-712 vs frameworks/native/ability/native/ui_extension_base/js_ui_extension_base.cpp:745-783 - **触发路径**：ETS CreateWindowOption 未从 UIEXTENSION_LAUNCH_TIMESTAMP_HIGH/LOW 计算 launchTimestamp，也未调用 SetStartModalExtensionTimeStamp，而 JS 对等函数有完整逻辑（lines 768-781）。 - **影响**：ETS UIExtensionBase 窗口缺少启动时间戳标记，影响 start-modal-extension 的时序诊断/追踪。预先存在，PR 新增的 SetCallerPid（line 710）与 JS（line 782）一致。 - **证据**：JS CreateWindowOption 在 SetIsBlockSubwindow 后、SetCallerPid 前有 launchTimestamp 计算块（768-781）；ETS 对等位置（706-710）无此块。 - **建议**：后续将 launchTimestamp 计算逻辑移植到 ETS CreateWindowOption。 ### [LOG-03] HandleSessionCreate 中 sessionInfo 判空重复（冗余代码） (P3, scanner=logic-scanner) - **位置**：frameworks/ets/ani/ui_extension_base/src/ets_ui_extension_base.cpp:716-724 - **触发路径**：HandleSessionCreate 在 lines 716-718 检查 sessionInfo == nullptr || sessionInfo->uiExtensionComponentId == 0，随后在 lines 722-724 重复相同检查。第二次检查恒为通过（若第一次通过），为死逻辑。 - **影响**：无功能缺陷（第二次检查为 no-op），但在 PR 修改函数的直接调用路径中存在冗余代码。 - **证据**：lines 716-718 与 722-724 的条件表达式完全相同。 - **建议**：后续清理重复判空块（lines 722-725）。 See merge request: openharmony/ability_ability_runtime!20308 | 9 天前 | |
pc告警修改 Signed-off-by: zhangyuhang72 <zhangyuhang72@h-partners.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: opencode (glm-5.2) <ai@local> Change-Id: Ibe1d2c17709d8e86f3a4e422935674ce9c28248c | 10 天前 | |
!20208 merge master into master Description:add DMA appTelemetry Created-by: EurusHomles-zH Commit-by: EurusHomles-zH Merged-by: openharmony_ci Description: **IssueNo**: https://gitcode.com/openharmony/ability_ability_runtime/issues/15978 **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用例 - [ ] 已验证 - [x] 不涉及。如不涉及，请写明理由 ### AI检视评分（使用本地代码检视skills扫描）: See merge request: openharmony/ability_ability_runtime!20208 | 10 天前 | |
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 天前 |