| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
fix: 统一意图 toolCallId 外层透传并补齐 ETS 支持 FunctionManager 进入 AMS 时将保留参数提取到 toolCallId_，清理业务参数副本中的 保留 key。通过外层 Want 传递并恢复该字段，公共执行器和日志统一读取独立字段， 移除向 insightIntentParam_ 注入标识及从中兜底读取的逻辑。 补齐 ETS Driver 参数解析与 InsightIntentContext 只读属性桥接，更新相关 UT， 覆盖外层参数往返、缺省值、业务参数隔离、调用间隔离及 ANI 错误处理。 验证：26 个宿主用例通过（使用平台依赖桩，启用 ASan/UBSan），差异格式检查通过。 当前缺少完整 OpenHarmony 构建环境，平台编译、UT 和 ETS VM 验证尚未执行。 Signed-off-by: zexin_c <chenzexin14@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: codex (gpt-6-astra) <ai@local> | 2 天前 | |
!20583 merge fix_testrunner into master 修复GetTestRunnerPath中erase崩溃场景 Created-by: ProfaneAria Commit-by: renguojun 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扫描）: # 代码检视报告 — commit 75ddcf2bc5（Round 2 / 最新提交） > 统一报告由 codecheck 工作台生成，**用于门禁管控**。所有 codecheck 报告（含 orchestrator 合并出的统一报告、单 scanner 直接产出的统一报告）必须遵循本模板：章节顺序、字段名、报告元数据块、评分与门禁规则均为**固定格式**，跨报告保持一致，便于门禁脚本解析与历史对比。 > 生成入口：[ README.md](README.md) → Step 5；合并逻辑见 [orchestrator/SKILL.md](orchestrator/SKILL.md)。 > 权威评分与门禁规则为**通用规则**，不在输出报告中呈现；生成时必须按 [conventions.md](conventions.md) §7（等级归一化）/ §8（必检维度）/ §9（评分与决策矩阵）计算，不得自创分值。 --- ## 报告元数据 > **门禁脚本只读取本 YAML 块**。字段名与取值域为固定合约，禁止改名、增删或自定义取值。人工阅读部分从「1. 门禁结论」开始。 <!-- codecheck-report-metadata:start --> yaml codecheck_report: schema_version: "1.0" scope: "commit 75ddcf2bc5 — Round 2: rfind/erase npos 守卫统一修复（3 文件）" round: 2 commit_id: "75ddcf2bc59ff20a92d9cacf8783b4ee1ce8fa7f" change_id: "N/A" report_id: "75ddcf2bc5-R2" date: "2026-09-24" gate_decision: "approve" risk_level: "low" score: 93 dimensions_required: ["security-scanner", "logic-scanner"] dimensions_executed: ["security-scanner", "logic-scanner"] findings_total: 3 findings_by_severity: P0: 0 P1: 0 P2: 1 P3: 2 gate_blockers: 0 must_fix: 0 followups: 3 <!-- codecheck-report-metadata:end --> --- ## 1. 门禁结论 | 项目 | 结论 | |---|---| | 决策 | **approve** | | 风险等级 | 🟢 low | | 评分 | **93/100** | | 阻塞项 | 无 | | 必须修复（P0/P1） | 0 项 | | 建议跟进（P2/P3） | 3 项 | **一句话结论**：Round 1 的 4 个 follow-up（SEC-001/002/003/LOG-002）已全部修复，修复正确且未引入新问题。全库 erase(rfind(".")) 无守卫模式 0 处残留。剩余 1 个 P2（simulator 测试缺失）和 2 个预存 follow-up 不阻塞合入。 --- ## 2. 扣分原因 > gate_decision=approve，本节省略。评分扣分来自 1 个 P2（simulator 测试缺失），共扣 5 分。 --- ## 3. 必须立即处理（P0/P1） **无。** --- ## 4. 建议本轮或下一补档处理（P2/P3） | ID | 优先级 | 问题 | 建议行动 | 排期 | |---|---|---|---|---| | LOG-003 | P2 | simulator.cpp 两处 rfind/erase npos 守卫无测试覆盖 | 补充 ParseAbilityInfo 在 abilitySrcPath 不含 . 时的测试 | 下一补档 | | SEC-R2-001 | P2 | 3 个预存测试 use-after-move UB（0100/Test_0400/Test_0600），SEC-003 修复未覆盖 | 将 pTestRunner 赋值移至 std::move 之前，与 0200 修复方式一致 | 下一补档 | | LOG-001 | P3 | [Round 1 已知] rfind(".") 搜索整个 result 含模块名，可能误截 | 将 rfind 限定在路径部分 | 后续改善 | --- ## 5. 分维度速览 | 维度 | 结果 | 关键说明 | |---|---|---| | security-scanner | ✅ 通过 | Round 1 的 SEC-001/002/003 全部修复正确。全库 erase(rfind 无守卫模式 0 残留。test 0100 UB 残留为 P3（预存，非本次引入） | | logic-scanner | ✅ 通过 | 三处 npos 守卫修复在所有输入场景下行为正确，无调用链回归。边界条件（空路径/无扩展名/无"/"）均安全处理。simulator 测试缺失为 P2 follow-up | --- ## 6. 关键发现详情 ### [LOG-003] simulator.cpp 两处修复无测试覆盖 (P2, scanner=logic-scanner) - **位置**：frameworks/simulator/ability_simulator/src/simulator.cpp:337,435 - **触发路径**：ParseAbilityInfo 在 abilitySrcPath 不含 . 时 / LoadAbilityStage 在 srcEntrance 不含 . 时 — npos 守卫路径无测试验证 - **影响**：修复正确但无回归测试保护，后续重构可能回退 - **证据**：js_test_runner.cpp 有 4 个测试覆盖各边界（0200/0300/0400/0500），simulator.cpp 的两处修复无对应测试 - **建议**：补充 ParseAbilityInfo 传入不含 . 的 abilitySrcPath 测试，验证不崩溃且 abilityNameFromPath 正确推导 ### [SEC-R2-001] 预存测试 use-after-move UB 未修复 (P2, scanner=security+logic) - **位置**：js_test_runner_test.cpp:280（Test_0400）、:322（Test_0600）、:365（GetTestRunnerPath_0100） - **触发路径**：std::move(testRunner) → testRunner 为空 → testRunner.get() 返回 nullptr → 对 nullptr 调用 ReportFinished()/ReportStatus()/GetTestRunnerPath() - **影响**：技术性 UB，被调函数不解引用 this，实践中不崩溃 - **证据**：Js_Test_Runner_Test_0400 line 282 EXPECT_EQ(testRunner, nullptr) 自证 testRunner 为 null - **建议**：将 pTestRunner 赋值移至 std::move(testRunner) 之前，与 0200/0300/0400/0500 修复方式一致 - **归属**：预存代码（3 个测试用例的代码行未被本次 commit 修改），SEC-003 修复时仅覆盖了 commit 范围内的测试 ### [LOG-001] rfind 作用于整个 result 含模块名 (P3, scanner=logic-scanner) - **位置**：frameworks/native/appkit/ability_delegator/runner_runtime/js_test_runner.cpp:266 - **触发路径**：GetTestModuleName() 返回含 . 的值 + GetTestRunnerClassName() 为空 + GetTestRunnerPath() 不含 . → rfind 命中模块名中的点 → 误截 - **影响**：路径构造错误，LoadModule 加载失败 - **建议**：将 rfind 限定在 args->GetTestRunnerPath() 返回值范围内 - **归属**：[Round 1 已知 follow-up] 预存在缺陷，本次未修复 See merge request: openharmony/ability_ability_runtime!20583 | 1 天前 | |
修复查询子进程相关错误码转化问题 Signed-off-by: SKY2001 <songtianyu10@huawei.com> AI[0%] Human Fixed[0%] Human[100%] AI Adopted[0%] | 29 天前 | |
fix: 统一意图 toolCallId 外层透传并补齐 ETS 支持 FunctionManager 进入 AMS 时将保留参数提取到 toolCallId_，清理业务参数副本中的 保留 key。通过外层 Want 传递并恢复该字段，公共执行器和日志统一读取独立字段， 移除向 insightIntentParam_ 注入标识及从中兜底读取的逻辑。 补齐 ETS Driver 参数解析与 InsightIntentContext 只读属性桥接，更新相关 UT， 覆盖外层参数往返、缺省值、业务参数隔离、调用间隔离及 ANI 错误处理。 验证：26 个宿主用例通过（使用平台依赖桩，启用 ASan/UBSan），差异格式检查通过。 当前缺少完整 OpenHarmony 构建环境，平台编译、UT 和 ETS VM 验证尚未执行。 Signed-off-by: zexin_c <chenzexin14@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[100%] Co-authored-by: codex (gpt-6-astra) <ai@local> | 2 天前 | |
!20561 merge master into master add PERMISSION_LOAD_CERTSIGN_LIBRARY permission verifiction Created-by: GroupM Commit-by: GroupM Merged-by: openharmony_ci Description: https://gitcode.com/openharmony/ability_ability_runtime/issues/16322 See merge request: openharmony/ability_ability_runtime!20561 | 4 天前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 2 天前 | ||
| 1 天前 | ||
| 29 天前 | ||
| 2 天前 | ||
| 4 天前 |