Star20
573
代码
代码
Issues160
Pull Requests133
流水线
项目讨论
Wiki
分析
Star20
573
已开启
Message :重试获取查杀原因 #20322
zhengdongdong12创建于 1 天前
Message :重试获取查杀原因 #20322
已开启
zhengdongdong12创建于 1 天前
从zhengdongdong12/ability_ability_runtime_7358master合入到openharmony/ability_ability_runtimemaster
zhengdongdong12
zhengdongdong12
1 天前

IssueNo: https://gitcode.com/openharmony/hiviewdfx_hicollie/issues/518

Description: 重试获取查杀原因

稳定性自检：

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

安全编码自检：

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

TDD Result: OK

XTS Result: OK

是否已执行L0用例

AI检视评分（使用本地代码检视skills扫描）:

likedislike
合并受阻
A
afwk_helper成员
1 天前 评论：

开始进行AI检视！

AI review has been started, please wait...

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

感谢提交 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 天前 添加了label：dco检查成功
A
afwk_helper成员
1 天前 评论：

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

总体评估: NEEDS_ATTENTION

问题统计:

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

摘要:
本次PR仅包含在4个C++文件的include区域添加空行的非功能性修改，未包含PR标题所描述的“重试获取查杀原因”相关的任何业务逻辑代码。

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

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

likedislike
A
afwk_helper成员1 天前进行代码检视1
services/appdfr/src/appfreeze_manager.cpp
已过期
@@ -24,6 +24,7 @@
2424#include <sstream>
2525#include <sys/ioctl.h>
2626 
27+ 
A
afwk_helper1 天前评论：

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

🟡 PR内容与标题不符，缺少核心功能代码

位置: L27 | 严重程度: Medium

❓ 问题描述

PR标题声明为“Message :重试获取查杀原因”，但实际代码变更仅为在多个文件中添加空行，没有包含任何与业务逻辑相关的代码。这可能是提交时遗漏了核心修改文件，导致无法进行功能性代码审查。

💡 修复建议

修改建议：请检查并补充本PR遗漏的“重试获取查杀原因”相关的核心功能代码文件，确保提交内容完整。

27: // Add actual retry logic for getting kill reason here

likedislike
zhengdongdong12zhengdongdong12
1 天前 审查状态已重置，审查人: afwk_helper
zhengdongdong12zhengdongdong12
1 天前 推送  1 个提交:c0c18d8d-update: 更新文件 appfreeze_manager.h
openharmony_ci
openharmony_ci成员
1 天前 评论：

感谢提交 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
zhengdongdong12zhengdongdong12
1 天前 审查状态已重置，审查人: afwk_helper
zhengdongdong12zhengdongdong12
1 天前 推送  1 个提交:032f2b29-update: 更新文件 appfreeze_manager.cpp
openharmony_ci
openharmony_ci成员
1 天前 评论：

感谢提交 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
zhengdongdong12zhengdongdong12
1 天前 审查状态已重置，审查人: afwk_helper
zhengdongdong12zhengdongdong12
1 天前 推送  1 个提交:ca7f2dbd-update: 更新文件 app_mgr_service_inner.h
openharmony_ci
openharmony_ci成员
1 天前 评论：

感谢提交 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
zhengdongdong12zhengdongdong12
1 天前 审查状态已重置，审查人: afwk_helper
zhengdongdong12zhengdongdong12
1 天前 推送  1 个提交:114631ad-update: 更新文件 app_mgr_service_inner.h
openharmony_ci
openharmony_ci成员
1 天前 评论：

感谢提交 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
zhengdongdong12zhengdongdong12
1 天前 关闭了 pull request
zhengdongdong12zhengdongdong12
1 天前 重新打开了 pull request
zhengdongdong12zhengdongdong12
1 天前 关闭了 pull request
zhengdongdong12zhengdongdong12
1 天前 重新打开了 pull request
zhengdongdong12zhengdongdong12
1 天前 审查状态已重置，审查人: afwk_helper
zhengdongdong12zhengdongdong12
1 天前 强制推送  1 个提交:affad999-Message: 重试获取查杀原因
openharmony_ci
openharmony_ci成员
1 天前 评论：

感谢提交 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
zhengdongdong12zhengdongdong12
1 天前 审查状态已重置，审查人: afwk_helper
zhengdongdong12zhengdongdong12
1 天前 强制推送  1 个提交:70e3d4e1-Message: 重试获取查杀原因
openharmony_ci
openharmony_ci成员
1 天前 评论：

感谢提交 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
zhengdongdong12zhengdongdong12
1 天前 审查状态已重置，审查人: afwk_helper
zhengdongdong12zhengdongdong12
1 天前 强制推送  1 个提交:91aa7857-Message: 重试获取查杀原因
openharmony_ci
openharmony_ci成员
1 天前 评论：

感谢提交 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
zhengdongdong12
zhengdongdong12
1 天前 评论：

start build

likedislike
A
afwk_helper成员
1 天前 评论：

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

likedislike
openharmony_ci
openharmony_ci成员
1 天前 评论：

该提交没有关联任何Issue,请先关联一个Issue后再触发构建
The PR is not associated with any Issue, Please associate the PR with one Issue before trigger build.

likedislike
zhengdongdong12zhengdongdong12
1 天前 修改了pull request 的描述
zhengdongdong12zhengdongdong12
1 天前 修改了pull request 的描述
openharmony_ci
openharmony_ci成员
1 天前 评论：

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

likedislike
openharmony_ciopenharmony_ci成员
1 天前 添加了label：编译失败
openharmony_ciopenharmony_ci成员
1 天前 添加了label：静态检查成功
openharmony_ciopenharmony_ci成员
1 天前 添加了label：冒烟测试失败
openharmony_ci
openharmony_ci成员
1 天前 评论：

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

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result package
1 ohos-sdk success >>>
2 dayu200 failed(compile failed) >>>
3 dayu200_tdd failed(compile failed) >>>
4 master_inner_build failed(IGNORE)(联合门禁失败) >>>
5 mac-sdk success >>>
6 ohos-host_mini_tdd success >>>
7 dayu600_7885 failed(compile failed) >>>
8 x86_64_virt failed(compile failed) >>>

likedislike