| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
!20738 merge Asyn into master fix: 接口同步改异步 Created-by: SkyQAQ Commit-by: songkeyuan Merged-by: openharmony_ci Description: **IssueNo**: https://gitcode.com/openharmony/ability_ability_runtime/issues/16490 **Description**: fix: 接口同步改异步 **稳定性自检:** | 自检项 | 自检结果 | | ------------------------------------------------------------ | -------- | | 涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发 | | | 成员变量进行赋值或创建需要排查并发 | | | 谨慎在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扫描): # 代码检视报告 — 接口同步改异步(StartRemoteIntent / SendIntentResult / WantReceiver)(Round 1 / 最新提交) > 统一报告由 codecheck 工作台生成,**用于门禁管控**。审查对象为最新提交 fa0372489b(fix: 接口同步改异步)。 > 说明:LOG-01(返回值契约破坏)与 LOG-02(SendIntentResult 失败检测失效)经调用方与单测对齐实现后已屏蔽,本轮仅剩 LOG-03。 --- ## 报告元数据 <!-- codecheck-report-metadata:start --> yaml codecheck_report: schema_version: "1.0" scope: "接口同步改异步(StartRemoteIntent/SendIntentResult/WantReceiver)" round: 1 commit_id: "de7edca85877a4c66f0a8b841b83e37b8344c8e3" change_id: "N/A" report_id: "de7edca8-R1" date: "2026-10-20" gate_decision: "approve" risk_level: "low" score: 98 dimensions_required: ["api-scanner", "security-scanner", "logic-scanner", "input-scanner"] dimensions_executed: ["api-scanner", "security-scanner", "logic-scanner", "input-scanner"] findings_total: 1 findings_by_severity: {P0: 0, P1: 0, P2: 0, P3: 1} gate_blockers: [] must_fix: [] followups: ["LOG-03"] <!-- codecheck-report-metadata:end --> --- ## 1. 门禁结论 | 项目 | 结论 | | ----------------- | ----------- | | 决策 | **approve** | | 风险等级 | 🟢 low | | 评分 | **98/100** | | 阻塞项 | 无 | | 必须修复(P0/P1) | 0 项 | | 建议跟进(P2/P3) | 1 项 | **一句话结论**:本次将跨设备意图相关 IPC 与 WantReceiver 由同步改异步,方向合理且未触碰权限校验与序列化格式;原 P1/P2 项(返回值契约、失败检测)已由调用方与单测对齐实现并屏蔽,仅剩 1 项 P3 文档化建议,可上库。 --- ## 3. 必须立即处理(P0/P1) **无。** --- ## 4. 建议本轮或下一补档处理(P2/P3) | ID | 优先级 | 问题 | 建议行动 | 排期 | | ------ | ------ | ------------------------------------------------------------ | ---------------------------------------------- | -------- | | LOG-03 | P3 | 新公共头宏 PARCEL_TRANSACT_ASYNC_RET_INT 无注释说明 fire-and-forget 语义,reply 形参未使用易被后续误用 | 补充宏注释,明确 async 语义与 reply 未使用约定 | 下一补档 | --- ## 5. 分维度速览 | 维度 | 结果 | 关键说明 | | ---------------- | ------------------- | ------------------------------------------------------------ | | logic-scanner | 🟢 1×P3 | 返回值契约(LOG-01)、结果投递失败感知(LOG-02)已对齐实现并屏蔽;仅剩宏语义未文档化(LOG-03) | | security-scanner | ✅ 无发现 | 未触碰权限校验(ability_manager_service.cpp:15600 的 PERMISSION_EXECUTE_DISTRIBUTED_INTENT 校验仍保留)、未改序列化格式、无崩溃/UAF/权限绕过/敏感数据风险 | | input-scanner | N/A | 本次 diff 仅改 IPC 发送侧(同步→异步),无「外部输入→持久化」链路,无适用面 | | api-scanner | ✅ 无签名/一致性缺陷 | distributed_parcel_helper.h 仅新增宏(向后兼容),公共方法签名未变;返回语义变化已对齐处理 | --- ## 6. 关键发现详情 ### [LOG-03] 新公共头宏缺注释,async 语义未文档化 (P3, scanner=api-scanner) - **位置**:interfaces/kits/native/ability/native/distributed_ability_runtime/distributed_parcel_helper.h:100-109 - **触发路径**:PARCEL_TRANSACT_ASYNC_RET_INT 形参 reply 未使用,宏体无注释说明 fire-and-forget 语义,后续调用方可能误以为仍返回远程结果。 - **影响**:潜在误用风险,当前不可达。 - **证据**:distributed_parcel_helper.h:100-109 宏体。 - **建议**:补充宏注释,明确「async、成功即返回 ERR_OK、reply 未使用」。 See merge request: openharmony/ability_ability_runtime!20738 | 2 天前 | |
Update SetFont param count error message Remove the unused APPLICATION_FONT_ANIMATION key from the simulator configuration header. Signed-off-by: zhupeisong <zhupeisong@huawei.com> AI[100%] Human Fixed[0%] Human[0%] AI Adopted[80%] Co-authored-by: opencode (deepseek-flash) <ai@local> | 1 天前 | |
修复查询子进程相关错误码转化问题 Signed-off-by: SKY2001 <songtianyu10@huawei.com> AI[0%] Human Fixed[0%] Human[100%] AI Adopted[0%] | 1 个月前 | |
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> | 29 天前 | |
!20682 merge FixYuHangGaoJing into master 修复PC告警-yuhang Created-by: SKY2001 Commit-by: SKY2001 Merged-by: openharmony_ci Description: 修复PC告警-yuhang https://gitcode.com/openharmony/ability_ability_runtime/issues/16439?ref=&did=4408656#tid-4408656 See merge request: openharmony/ability_ability_runtime!20682 | 8 天前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 2 天前 | ||
| 1 天前 | ||
| 1 个月前 | ||
| 29 天前 | ||
| 8 天前 |