已合并
修复UpdateApplicationInfoInstalled未查询分身的问题 #20237
修复UpdateApplicationInfoInstalled未查询分身的问题 #20237
已合并
SKY2001创建于 4 天前
SKY2001
4 天前

IssueNo:
https://gitcode.com/openharmony/ability_ability_runtime/issues/16003?ref=&did=4260220#tid-4260220
Description:

稳定性自检:

自检项 自检结果
涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发 pass
成员变量进行赋值或创建需要排查并发 pass
谨慎在lambda表达式中使用引用捕获 pass
谨慎在未经拷贝的情况下使用外部传入的string、C字符串 pass
map\vector\list\set等stl模板类使用时需要排查并发 pass
谨慎考虑加锁范围 pass
在IPC通信中谨慎使用同步通信方式 pass
禁止传递this指针至其他模块或线程(特别是eventhandler任务) pass
禁止将外部传入的裸指针在内部直接构造智能指针 pass
禁止多个独立创建的智能指针管理同一地址 pass
禁止在析构函数中抛异步任务 pass
禁止js对象在非js线程(例如在IPC线程)创建、使用或销毁 pass
禁止在对外接口中未经判空直接使用外部传入的指针 pass
禁止接口返回局部变量引用 pass
禁止在信号函数中加锁 pass
禁止在关键流程(SA启动、应用启动等主流程)执行耗时的操作 pass
禁止将同一个cpp编译在不同的so中 pass

安全编码自检:

自检项 自检结果
裸指针避免通过隐式转换构造为sptr pass
json对象在取值之前必须先判断类型,避免类型不匹配 pass
序列化时必须对传入的数组大小进行校验,避免出现超大数组 pass
避免使用未明确位宽的整型,选择使用int8_t、uint8_t等类型 pass
外部传入的路径要做规范化校验,对路径中的.、..、../等特殊字符严格校验 pass
指针变量、表示资源描述符的变量、bool变量必须赋初值 pass
readParcelable获取的对象使用前需要判空 pass
分配和释放内存的函数需要成对出现 pass
申请内存后异常退出前需要及时进行内存释放 pass
内存申请前必须对内存大小进行合法性校验 pass
内存分配后必须判断是否成功 pass
禁止使用realloc、alloca函数 pass
禁止打印文件路径、口令等敏感信息,如有需要,使用private修饰 pass
禁止打印内存地址 pass
整数之间运算时必须严格检查,确保不会出现溢出、反转、除0 pass
禁止对有符号整数进行位操作符运算 pass
禁止对指针进行逻辑或位运算 pass
循环次数如果收外部数据控制,需要检验其合法性 pass
禁止使用内存操作类危险函数,需要使用安全函数 pass
谨慎使用不可重入函数 pass
必须检查安全函数的返回值,并进行正确处理 pass
禁止仅通过TokenType类型判断绕过权限校验 pass

TDD Result:
pass
XTS Result:
pass

是否已执行L0用例

AI检视评分(使用本地代码检视skills扫描):

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 SKY2001 的贡献)
SSKY2001
4 天前 关联了issue:[Bug]: 修复UpdateApplicationInfoInstalled未查询分身的问题
afwk_helper成员
4 天前 评论:

开始进行AI检视!

AI review has been started, please wait...

likedislike
afwk_helper成员
4 天前 评论:
check type result report
start ai_review pass -
likedislike
openharmony_ciopenharmony_ci成员
4 天前 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
4 天前 评论:

感谢提交 Pull Requests!如果您提交的PR已经开发完毕,请评论 "start build" 触发门禁,更多交互操作,请访问OpenHarmony社区支持命令清单。如果需要调整订阅PR、Issue的变更状态,请访问订阅链接


Thanks for submitting the pull request. If your Pull Request has already been developed, you can leave a "start build" comment to trigger the gated system. For more commands, please visit OpenHarmony Command List. If you need to change the subscription of a Pull Request or Issue, please visit the link.

likedislike
openharmony_ciopenharmony_ci成员
4 天前 添加了label:dco检查成功
afwk_helper成员
4 天前 评论:

⚠️ 🤖 AI 代码检视报告 ⚠️

总体评估: NEEDS_ATTENTION

问题统计:

  • 总问题数: 3
  • 严重问题: 0
  • 高危问题: 0

摘要:
修复了UpdateApplicationInfoInstalled未查询分身应用信息的问题,核心逻辑正确,但GetNameAndIndexForUid成功后缺少对resolveBundleName的空值校验,且错误日志缺少上下文信息,建议补充防御性检查。

📊 详细报告
查看完整的审查详情,包括具体的问题描述、建议和代码位置:
🔗 查看详细报告


此评论由 OpenHarmony Insight 代码审查系统自动生成

likedislike
afwk_helper成员
4 天前 评论:

⚠️ AI检视服务网关超时(504),检视可能在服务端已完成。

请访问 openharmonyinsight.cn 查看检视结果。

AI review gateway timed out. The review may have completed on the server.

likedislike
SKY2001
3 天前 评论:

static-check

likedislike
openharmony_ci
openharmony_ci成员
3 天前 评论:


静态检查:

# check type result report
1 codeCheck pass >>>

likedislike
SKY20013 天前进行代码检视1
test/unittest/app_mgr_service_inner_seventh_test/app_mgr_service_inner_seventh_test.cpp
@@ -242,0 +249,4 @@
249+ AAFwk::MyStatus::GetInstance().getCallingUid_ = 0;
250+ AAFwk::MyStatus::GetInstance().getNameAndIndexForUidRet_ = ERR_INVALID_OPERATION;
251+ 
252+ std::string bundleName = "com.test.app";
SKY20013 天前评论:

用例检视pass

likedislike
SKY2001
3 天前 评论:

pass

likedislike
afwk_helper成员
3 天前 评论:
AI代码检视(AI Code Check)
报告标题 报告来源 提交时间 评分 ID匹配 检查结果
- - - - - fail

AI代码检视失败。
AI Code Check failed.

未找到AI代码检视报告。请使用本地代码检视工具(skills/codecheck)对代码进行检视,生成检视报告后将其粘贴到PR评论或PR提交描述中。
No AI code review report found. Please run local code check skills (skills/codecheck), and paste the report into PR comments or PR description.

若已处理完毕,请评论 "check ai-code-check" 重新触发检查。
或联系以下人员评论 "force ai-code-check" 强制通过:
duansizhao: @dsz2025
xialiangwei: @xialiangwei
wangkailong: @wkljy
liqiang: @jayleehw
管控详情:https://gitcode.com/dsz2025/AfwkHelper/blob/main/PR管控/PR管控说明.md

likedislike
afwk_helper成员
3 天前 评论:

自定义PR管控检查报告 / Custom PR Control Check Report

检查项 检查结果 处理建议
TDD检查 ✓ 通过 -
检视意见 ✓ 通过 -
AI代码检视 ✗ 失败 未找到AI代码检视报告。请使用本地代码检视工具(skills/codecheck)对代码进行检视,生成检视报告后将其粘贴到PR评论或PR提交描述中。
likedislike
jsjzjujsjzju成员
2 天前 通过审查
openharmony_ci
openharmony_ci成员
2 天前 评论:

ISSUE关联关系有变化,之前验证结果无效,需要重新触发构建

likedislike
SKY2001
2 天前 评论:

代码检视报告 — 20237 (UpdateApplicationInfoInstalled 多 AppIndex 支持)(Round 1 / 最新提交)

统一报告由 codecheck 工作台生成,用于门禁管控


报告元数据

codecheck_report:
  schema_version: "1.0"
  scope: "20237_UpdateApplicationInfoInstalled"
  round: 1
  commit_id: "195f6e237b3011c9c3925ca8143a42b71bd21518"
  change_id: "20237"
  report_id: "20237-R1"
  date: "2026-08-19"
  gate_decision: "approve"
  risk_level: "low"
  score: 98
  dimensions_required: ["security", "logic"]
  dimensions_executed: ["security", "logic"]
  findings_total: 1
  findings_by_severity: {"P0": 0, "P1": 0, "P2": 0, "P3": 1}
  gate_blockers: 0
  must_fix: 0
  followups: 1

1. 门禁结论

项目 结论
决策 approve
风险等级 🟢 low
评分 98/100
阻塞项
必须修复(P0/P1) 0 项
建议跟进(P2/P3) 1 项

一句话结论UpdateApplicationInfoInstalledGetApplicationInfo 改为 GetNameAndIndexForUid + GetApplicationInfoWithAppIndex 以支持多 AppIndex,逻辑正确、错误处理完整、安全性提升(不信任入参 bundleName,改用 uid 反查),测试覆盖 4 条新路径;仅 1 项 P3 测试覆盖建议。


2. 扣分原因

gate_decision=approve,本节省略。


3. 必须立即处理(P0/P1)

无。 详情见第 6 节。


4. 建议本轮或下一补档处理(P2/P3)

ID 优先级 问题 建议行动 排期
LOG-001 P3 测试未验证"入参 bundleName 被忽略、改用 uid 反查"的语义变更 补充用例:入参 bundleName 与 uid 反查结果不一致时验证以 uid 反查为准 下一补档

5. 分维度速览

维度 结果 关键说明
security ✅ 通过 11 项 Core Scan 全部不命中;修改是安全改进(uid 反查替代信任入参 bundleName,防伪造);上游有 VerifyRequestPermission() 权限校验保护 IPC 入口;G 类横扫无命中
logic ✅ 通过 控制流/数据流/边界/错误处理均正确;auto errCode + errCode != ERR_OK 类型匹配(bool 转换 bug 已修复);变量遮蔽已修复(resolveBundleName);4 条新路径测试覆盖良好;仅 1 项 P3 测试覆盖建议

6. 关键发现详情

[LOG-001] 测试未验证入参 bundleName 被忽略的语义变更 (P3, scanner=logic)

  • 位置test/unittest/app_mgr_service_inner_seventh_test/app_mgr_service_inner_seventh_test.cpp:244-327(用例 004-007)
  • 触发路径:用例 004-007 传入 bundleName="com.test.app",但 mock 的 getNameForUid_ 默认为空字符串 → resolveBundleName="",与入参不一致。GetApplicationInfoWithAppIndex mock 不依赖参数(直接返回 getApplicationInfoWithAppIndexRet_),因此测试通过但无法验证"入参被忽略"这一行为变更。
  • 影响:语义变更(从"按入参 bundleName 查询"改为"按 uid 反查")未被测试覆盖。若未来 GetApplicationInfoWithAppIndex 的真实实现依赖 resolveBundleName 参数,空字符串可能导致查询失败,而当前测试无法发现。
  • 证据mock_bundle_mgr_helper.cpp:219-224GetApplicationInfoWithAppIndex 直接返回 getApplicationInfoWithAppIndexRet_,不读取 appName 参数;mock_bundle_mgr_helper.cpp:48-51GetNameAndIndexForUid 设置 bundleName = getNameForUid_(默认空),不使用入参 bundleName="com.test.app"
  • 建议:补充用例:设置 MyStatus::getNameForUid_ = "com.resolved.app"(与入参 bundleName="com.test.app" 不同),验证 GetApplicationInfoWithAppIndex 收到的 appName"com.resolved.app" 而非 "com.test.app"(可通过在 mock 中记录参数值实现)。
likedislike
SKY2001
2 天前 评论:

pass

likedislike
afwk_helper成员
2 天前 评论:
AI代码检视(AI Code Check)
报告标题 报告来源 提交时间 评分 ID匹配 检查结果
⭐ 代码检视报告 — 20237 (UpdateApplicationInfoInstalled 多 AppIndex 支持)(Round 1 / 最新提交) 评论 2026-08-19 19:59:47 98/100 fail

AI代码检视失败。
AI Code Check failed.

找到AI检视报告,但commit-id或Change-Id与本PR不匹配。请确保报告对应本PR的最新提交。
AI review report found, but commit-id or Change-Id does not match this PR.

若已处理完毕,请评论 "check ai-code-check" 重新触发检查。
或联系以下人员评论 "force ai-code-check" 强制通过:
duansizhao: @dsz2025
xialiangwei: @xialiangwei
wangkailong: @wkljy
liqiang: @jayleehw
管控详情:https://gitcode.com/dsz2025/AfwkHelper/blob/main/PR管控/PR管控说明.md

likedislike
afwk_helper成员
2 天前 评论:

自定义PR管控检查报告 / Custom PR Control Check Report

检查项 检查结果 处理建议
TDD检查 ✓ 通过 -
检视意见 ✓ 通过 -
AI代码检视 ✗ 失败 找到AI检视报告,但commit-id或Change-Id与本PR不匹配。请确保报告对应本PR的最新提交。
likedislike
SKY2001
2 天前 评论:

start build

likedislike
afwk_helper成员
2 天前 评论:

门禁编译Tips:
1.触发门禁编译前先评论static-check进行静态检查
2.本地编译验证充分后再触发门禁
3.门禁编译时不要推送代码否则会终止上一次门禁编译。
失败超过3次将可能影响代码合入

likedislike
openharmony_ci
openharmony_ci成员
2 天前 评论:

首次触发
门禁构建开始,包含静态检查、代码编译和测试【dayu600_7885测试, dayu200_tdd编译, ohos-host_mini_tdd编译, ohos-sdk编译, mac-sdk编译, master_inner_build编译, dayu200编译, dayu600_7885编译, x86_64_virt编译, dayu200测试】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a859d9164650f998b95595c/runlist

likedislike
xialiangwei
xialiangwei成员
2 天前 评论:

force ai-code-check

likedislike
afwk_helper成员
2 天前 评论:

AI代码检视已强制通过,可继续联系审查人进行加分。
AI Code Check has been forcibly passed, and you may contact the reviewers to review.

likedislike
SKY2001
2 天前 评论:

pass

likedislike
Aafwk_helper成员
2 天前 通过审查
afwk_helper成员
2 天前 评论:

@SKY2001 评论pass通过, @afwk_helper 已加分, 满足合入条件后请联系Commiter审核加分

likedislike
afwk_helper成员
2 天前 评论:

自定义PR管控检查报告 / Custom PR Control Check Report

检查项 检查结果 处理建议
TDD检查 ✓ 通过 -
检视意见 ✓ 通过 -
AI代码检视 ⚠ 强制通过 -
likedislike
openharmony_ciopenharmony_ci成员
2 天前 添加了label:编译成功
openharmony_ciopenharmony_ci成员
2 天前 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
2 天前 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
2 天前 通过测试
openharmony_ci
openharmony_ci成员
2 天前 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a859d9164650f998b95595c/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 ohos-sdk success NA >>>
2 dayu200 success success >>>
3 dayu200_tdd success NA >>>
4 master_inner_build success(IGNORE) NA >>>
5 mac-sdk success NA >>>
6 ohos-host_mini_tdd success NA >>>
7 dayu600_7885 success success >>>
8 x86_64_virt success NA >>>

likedislike
openharmony_ciopenharmony_ci成员
2 天前 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
2 天前 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
2 天前 评论:

您好,Committer @jayleehw @zhangyafei-echo @littlejerry1 @jsjzju @wkljy @yzkp @ccllee1 @yangxuguang-huawei @li-weifeng2024 @wendel ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @jayleehw @zhangyafei-echo @littlejerry1 @jsjzju @wkljy @yzkp @ccllee1 @yangxuguang-huawei @li-weifeng2024 @wendel . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_id]", or you can comment "assign" to review the PR by yourself.

likedislike
openharmony_ciopenharmony_ci成员
2 天前 关闭了关联的issue
openharmony_ciopenharmony_ci成员
2 天前 合入了pull request,合并节点 SHA:3b479e822c69e05a788886c8e80e239a5ef263e7
openharmony_ciopenharmony_ci成员
2 天前 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
2 天前 添加了label:merged