已合并
verify profile #313
verify profile #313
已合并
ren_ze_hua创建于 1月17日
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 ren_ze_hua 的贡献)
ren_ze_huaren_ze_hua成员
1月17日 关联了issue:增加签名块校验合法性接口
AsterNighT
AsterNighT
1月17日 评论:

AI检视任务后台执行中,稍候返回结果。

The AI Review task is being executed in the background and will return results shortly.

您可通过评论"list commands"来查询命令列表。
You can leave a "list commands" comment to get commands list.

likedislike
AsterNighT
AsterNighT
1月17日 评论:

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

总体评估: NEEDS_ATTENTION

问题统计:

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

摘要:
代码添加了签名块校验逻辑,但存在函数名拼写错误等需要修复的问题。

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


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

likedislike
AsterNighT
AsterNighT1月17日进行代码检视2
interfaces/inner_api/code_sign_utils/src/code_sign_utils.cpp
已过期
@@ -272,2 +272,3 @@
272272{
273- int ret = EnableKeyInProfileByRust(bundleName.c_str(), profileBuffer.GetBuffer(), profileBuffer.GetSize());
273+ Verify::ProvisionInfo info;
274+ int ret = Verify::VrifyProfileByP7bBlock(profileBuffer.GetSize(),
AsterNighT
AsterNighT1月17日评论:

🤖 AI 代码检视意见(回复本评论可解决检视意见,点击被检视代码行左侧的小头像可收起检视意见)


🟠 函数名拼写错误

位置: L274 | 严重程度: High

❓ 问题描述

函数名 'VrifyProfileByP7bBlock' 存在拼写错误,应该是 'VerifyProfileByP7bBlock'。这个拼写错误可能导致编译错误或链接错误。

💡 修复建议

修改建议:修正函数名拼写,将 Vrify 改为 Verify。

274: + int ret = Verify::VerifyProfileByP7bBlock(profileBuffer.GetSize(),


likedislike
System
系统消息系统
1月19日 评论:

changed this line on ea76e214 view diff detail

openharmony_ciopenharmony_ci成员
1月17日 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
1月17日 评论:

感谢提交 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成员
1月17日 添加了label:dco检查成功
wangtiantian
wangtiantian
1月17日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
1月17日 评论:

非PR提交人或管理员不允许触发门禁

likedislike
openharmony_ci
openharmony_ci成员
1月17日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译和测试【part_compile测试, dayu200编译, hispark_taurus_Linux编译, hispark_taurus_LiteOS测试, dayu200_tdd编译, part_compile编译, hispark_taurus_LiteOS编译, master_inner_build编译, dayu200-codearts编译, dayu200测试】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/696ba0c664650f998b2d7c6f/runlist

likedislike
openharmony_ciopenharmony_ci成员
1月17日 添加了label:编译失败
openharmony_ciopenharmony_ci成员
1月17日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
1月17日 添加了label:冒烟测试失败
openharmony_ci
openharmony_ci成员
1月17日 评论:

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

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 hispark_taurus_LiteOS success success >>>
2 hispark_taurus_Linux success NA >>>
3 dayu200 failed(compile failed) NA >>>
4 dayu200_tdd failed(compile failed) NA >>>
5 part_compile success success(IGNORE) >>>
6 master_inner_build failed(IGNORE)(联合门禁失败) NA >>>
7 dayu200-codearts failed(IGNORE)(skip build) NA >>>

likedislike
ren_ze_huaren_ze_hua成员
1月19日 强制推送  1 个提交:ea76e214-verify profile
openharmony_ciopenharmony_ci成员
1月19日 删除了label:静态检查成功
openharmony_ciopenharmony_ci成员
1月19日 删除了label:编译失败
openharmony_ciopenharmony_ci成员
1月19日 删除了label:冒烟测试失败
openharmony_ci
openharmony_ci成员
1月19日 评论:

代码有更新,重置PR验证状态

likedislike
ren_ze_hua
ren_ze_hua成员
1月19日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
1月19日 评论:

感谢提交 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_ci
openharmony_ci成员
1月19日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译和测试【hispark_taurus_LiteOS测试, hispark_taurus_Linux编译, dayu200-codearts编译, dayu200测试, part_compile编译, dayu200编译, dayu200_tdd编译, master_inner_build编译, hispark_taurus_LiteOS编译, part_compile测试】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/696d83f564650f998bcf0815/runlist

likedislike
openharmony_ciopenharmony_ci成员
1月19日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
1月19日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
1月19日 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
1月19日 通过测试
openharmony_ci
openharmony_ci成员
1月19日 评论:

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

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 hispark_taurus_LiteOS success success >>>
2 hispark_taurus_Linux success NA >>>
3 dayu200 success success >>>
4 dayu200_tdd success NA >>>
5 part_compile success success(IGNORE) >>>
6 master_inner_build failed(IGNORE)(联合门禁失败) NA >>>
7 dayu200-codearts failed(IGNORE)(skip build) NA >>>

likedislike
openharmony_ciopenharmony_ci成员
1月19日 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
1月19日 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
1月19日 评论:

您好,Committer @JerryH1011 @hehehe-li @LeonChan525 @zhu-sheng-le ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @JerryH1011 @hehehe-li @LeonChan525 @zhu-sheng-le . 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
JerryH1011JerryH1011成员
1月19日 通过审查
openharmony_ciopenharmony_ci成员
1月19日 删除了label:静态检查成功
openharmony_ciopenharmony_ci成员
1月19日 删除了label:编译成功
openharmony_ciopenharmony_ci成员
1月19日 删除了label:冒烟测试成功
openharmony_ci
openharmony_ci成员
1月19日 评论:

<ISSUE: 156>中PR:https://gitcode.com/openharmony/security_appverify/merge_requests/292 代码有更新,重置所有关联PR的验证状态

likedislike
openharmony_ci
openharmony_ci成员
1月19日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译和测试【dayu200_tdd编译, hispark_taurus_LiteOS测试, master_inner_build编译, hispark_taurus_LiteOS编译, dayu200测试, part_compile编译, part_compile测试, hispark_taurus_Linux编译, dayu200-codearts编译, dayu200编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/696da23264650f998b0de628/runlist

likedislike
openharmony_ciopenharmony_ci成员
1月19日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
1月19日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
1月19日 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
1月19日 通过测试
openharmony_ci
openharmony_ci成员
1月19日 评论:

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

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 hispark_taurus_LiteOS success success >>>
2 hispark_taurus_Linux success NA >>>
3 dayu200 success success >>>
4 dayu200_tdd success NA >>>
5 part_compile success success(IGNORE) >>>
6 master_inner_build success(IGNORE) NA >>>
7 dayu200-codearts failed(IGNORE)(skip build) NA >>>

likedislike
openharmony_ci
openharmony_ci成员
1月19日 评论:

您好,Committer @JerryH1011 @hehehe-li @LeonChan525 @zhu-sheng-le ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @JerryH1011 @hehehe-li @LeonChan525 @zhu-sheng-le . 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成员
1月19日 关闭了关联的issue
openharmony_ciopenharmony_ci成员
1月19日 合入了pull request,合并节点 SHA:3aa475630a5715d31a5f21125bdea6a84253ac60
openharmony_ciopenharmony_ci成员
1月19日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
1月19日 添加了label:merged
openharmony_dcp
openharmony_dcp成员
1月19日 评论:

提交次数已达到或超过3次,为了保证上库效率和减少构建资源浪费,请点击链接观看学习视频,谢谢!https://dcp.openharmony.cn/workbench/video/videoDisplay

likedislike