ability_ability_runtime/frameworks/native · linjunjie6/ability_ability_runtime

Ddenganliangrefactor(ability_runtime): improve DumpArktsHeapSize robustness

文件	最后提交记录	最后更新时间
ability	!20315 merge stoi into master fix stoi Created-by: xhz-sz Commit-by: xhd-sz Merged-by: openharmony_ci Description: IssueNo: https://gitcode.com/openharmony/ability_ability_runtime/issues/16079 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 #20315 fix stoi（Round 1） > 范围：PR #20315（commit `cce6d1f7`），标题 "fix stoi"，3 文件 +179/-3。 > 检视重点：仅 PR 自身修改/新增代码。 --- ## 报告元数据 <!-- codecheck-report-metadata:start --> `yaml codecheck_report: schema_version: "1.0" scope: "PR-20315-fix-stoi" round: 1 commit_id: "cce6d1f7f3a4f8c0c52a5cb2c6193be933752cf9" change_id: "N/A" report_id: "cce6d1f7-R1" date: "2026-08-27" gate_decision: "approve" risk_level: "low" score: 98 dimensions_required: ["security-scanner", "logic-scanner"] dimensions_executed: ["security-scanner", "logic-scanner"] findings_total: 1 findings_by_severity: {P0: 0, P1: 0, P2: 0, P3: 1} gate_blockers: [] must_fix: [] followups: ["LOG-01"]` <!-- codecheck-report-metadata:end --> --- ## 1. 门禁结论 \| 项目 \| 结论 \| \|---\|---\| \| 决策 \| approve \| \| 风险等级 \| 🟢 low \| \| 评分 \| 98/100 \| \| 阻塞项 \| 无 \| \| 必须修复（P0/P1） \| 0 项 \| \| 建议跟进（P2/P3） \| 1 项 \| 一句话结论：PR 将 `ConvertStringToUint32` 从 `std::stoi`（异常 + 静默溢出 + 部分解析）替换为 `std::from_chars`（无异常 + 严格全消费 + 正确 uint32 范围），修复了负数静默溢出和 INT_MAX 以上值误拒两个缺陷，与同仓库已有的 `StringToUint32`（`context_impl.cpp`）行为对齐，测试覆盖 10 个用例，变更质量高。 --- ## 2. 扣分原因（仅 gate_decision=block 时呈现；本报告省略） --- ## 3. 必须立即处理（P0/P1）无。 --- ## 4. 建议本轮或下一补档处理（P2/P3） \| ID \| 优先级 \| 问题 \| 建议行动 \| 排期 \| \|---\|---\|---\|---\|---\| \| LOG-01 \| P3 \| `ConvertStringToUint32`（`resource_config_helper.cpp`）与 `StringToUint32`（`context_impl.cpp`）为两个独立但行为相同的解析函数，未统一 \| 后续可考虑合并为一个公共工具函数，避免未来分叉 \| 下一补档 \| --- ## 5. 分维度速览 \| 维度 \| 结果 \| 关键说明 \| \|---\|---\|---\| \| security-scanner \| 无发现 \| `from_chars` 无异常抛出，消除 `catch(...)` 吞没异常的隐患；`source` 按值传递，`source.data()` 安全；无 IPC/持久化/权限变更 \| \| logic-scanner \| 1 条 (P3:1) \| 行为变更为"收紧"（拒绝空白/尾部非数字/负数），与已有 `StringToUint32` 对齐；测试 10 用例全覆盖；两个解析器未统一为 P3 观察项 \| --- ## 6. 关键发现详情 ### [LOG-01] 两处 uint32 解析函数未统一 (P3, scanner=logic-scanner) - 位置：`frameworks/native/ability/native/resource_config_helper.cpp:188`（`ConvertStringToUint32`）与 `frameworks/native/appkit/ability_runtime/context/context_impl.cpp:102`（`StringToUint32`） - 触发路径：不涉及触发——两者当前行为已一致（均使用 `from_chars` + 全消费检查），但作为独立函数存在维护分叉风险 - 影响：未来修改一处可能遗漏另一处，导致行为不一致 - 证据：两个函数签名和实现几乎相同，分属不同文件和命名空间 - 建议：后续提取为公共工具函数（如 `ability_runtime::utils::StringToUint32Strict`），两处调用方统一引用 --- ## 附录：PR 变更概要 ### 变更摘要 \| 类别 \| 修改内容 \| 涉及文件 \| \|------\|---------\|---------\| \| 解析器替换 \| `std::stoi`（try/catch + `static_cast<uint32_t>`）→ `std::from_chars`（无异常 + 直接解析 uint32 + 全消费检查） \| `resource_config_helper.cpp` \| \| 修复缺陷 \| 负数静默溢出（`-1`→4294967295）；INT_MAX 以上值误拒（`stoi` 抛 `out_of_range`） \| 同上 \| \| 行为收紧 \| 拒绝前导空白、拒绝尾部非数字字符（与 `context_impl.cpp::StringToUint32` 对齐） \| 同上 \| \| 测试 \| 新增 10 个 UT 用例，覆盖正常值/边界/溢出/负数/空串/非数字/尾部字符/前导空白 \| `resource_config_helper_test.cpp` \| \| 构建配置 \| 新增 `resource_config_helper_test` 测试目标到 BUILD.gn \| `BUILD.gn` \| ### 行为变更分析 \| 输入 \| 旧行为（stoi） \| 新行为（from_chars） \| 风险评估 \| \|------\|---------------\|----------------------\|---------\| \| `"100"` \| 100 ✓ \| 100 ✓ \| 无变化 \| \| `"0"` \| 0 ✓ \| 0 ✓ \| 无变化 \| \| `"4294967295"` \| stoi 返回 -1 → cast → 4294967295 ✓（侥幸正确） \| 4294967295 ✓ \| 无变化 \| \| `"4294967296"` \| stoi 返回 -2 → cast → 4294967294 ✗（错误值） \| fail ✓ \| 修复 \| \| `"-1"` \| stoi 返回 -1 → cast → 4294967295 ✗（静默溢出） \| fail ✓ \| 修复 \| \| `"3000000000"` \| stoi 抛 `out_of_range` → catch → fail ✗（误拒） \| 3000000000 ✓ \| 修复 \| \| `""` \| stoi 抛 `invalid_argument` → fail \| fail \| 无变化 \| \| `"abc"` \| stoi 抛 → fail \| fail \| 无变化 \| \| `"123abc"` \| 123 ✗（部分解析，静默接受尾部垃圾） \| fail ✓ \| 收紧（改进） \| \| `" 100"` \| 100 ✗（跳过空白） \| fail ✓ \| 收紧，数据源为可信系统服务，不含空白 \| ### 验证命令 `bash ./build.sh --product-name <product> --build-target ability_runtime run -t UT -ts frameworks_kits_ability_native_test` ### 兼容性/权限/DFX 影响 - 兼容性：无公共 API 变更，无 SDK 接口签名变更。行为收紧（拒绝空白/尾部垃圾）为配置解析场景的合理改进，数据源为可信系统服务。 - 权限：无 PermissionVerification 变更。 - DFX：无 hisysevent/error-code 变更。 See merge request: openharmony/ability_ability_runtime!20315	1 个月前
appkit	!20270 merge feat-arktsheap into master feat(ability_runtime): add ARKTS_HEAP dump type for hidumper --show-arktsheap Created-by: denganliang Commit-by: denganliang Merged-by: openharmony_ci Description: ### 修改说明为 `hidumper --mem pid --show-arktsheap` 提供 ability_runtime 侧支持： - `MemDumpType` 新增 `ARKTS_HEAP = 5` 枚举值 - `DumpRuntimeHelper` 新增 `DumpArktsHeapSize` 方法 - `THREAD_NAME_MAX_LEN = 16` 命名常量 + `pthread_getname_np` 返回值检查 - 新增 `DumpMem_0500` UT ### 关联 Issue - https://gitcode.com/openharmony/hiviewdfx_hidumper/issues/618 ### 配套 PR - hidumper: https://gitcode.com/openharmony/hiviewdfx_hidumper/pull/802 - selinux_adapter: https://gitcode.com/openharmony/security_selinux_adapter/merge_requests/8191 ### AI检视评分（使用本地代码检视skills扫描）: <!-- codecheck-report-metadata:start --> yaml codecheck_report: schema_version: "1.0" scope: "ability_runtime PR #20270 (DumpArktsHeapSize)" round: 1 commit_id: "3be6ac53298841ec86eaa7ca4b92a3075290bb03" change_id: "I5a23ab4a9fa6ade31464c0d3ed5d407495f996cf" report_id: "I5a23ab4a9fa6ade31464c0d3ed5d407495f996cf-R1" date: "2026-08-27" gate_decision: "approve" risk_level: "低" score: 95 dimensions_required: ["security", "logic", "api"] dimensions_executed: ["security", "logic", "api"] findings_total: 2 findings_by_severity: {"P0": 0, "P1": 0, "P2": 0, "P3": 2} gate_blockers: 0 must_fix: 0 followups: 2 <!-- codecheck-report-metadata:end --> 评分：95/100 \| 门禁决策：approve \| 风险等级：低扫描维度：security + logic + api \| 原始发现 8 条 → Refute 后 P0=0 P1=0 P2=0 P3=2 详细报告见上方评论。 See merge request: openharmony/ability_ability_runtime!20270	1 个月前
child_process	补充查询子进程的接口 Signed-off-by: SKY2001 <songtianyu10@huawei.com> AI[47%] Human Fixed[0%] Human[53%] AI Adopted[100%]	1 个月前
insight_intent	前台组件BTI适配 Co-Authored-By: Liuzongze Signed-off-by: liuzongze@h-partners.com <liuzongze@h-partners.com> 🤖 AI[0%] 🔧 Human Fixed[0%] 🧑 Human[100%] 👌 AI Adopted[0%] Change-Id: I679d5a5acdc3173856c6a91f21839a50bd7ca899	2 个月前
runtime	!20051 merge 84runtime into master add isDebugApp Created-by: wlh2624_DS Commit-by: wlh2624_DS Merged-by: openharmony_ci Description: IssueNo: https://gitcode.com/openharmony/arkcompiler_runtime_core/issues/12418 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扫描）: See merge request: openharmony/ability_ability_runtime!20051	2 个月前