已合并
提供快照dfx #20157
提供快照dfx #20157
已合并
zhangtao创建于 19 天前
zhangtao
zhangtao
19 天前

IssueNo:
https://gitcode.com/openharmony/ability_ability_runtime/issues/15933

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扫描):

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 zhangtao 的贡献)
afwk_helper成员
19 天前 评论:

开始进行AI检视!

AI review has been started, please wait...

likedislike
afwk_helper成员
19 天前 评论:
check type result report
start ai_review pass -
likedislike
afwk_helper成员
19 天前 评论:

⏱️ AI检视超时通知 / AI Review Timeout Notification

AI代码检视任务执行时间过长(已耗时: 0:00:10),任务已超时终止。

The AI code review task has been running for too long (elapsed: 0:00:10) and has timed out.

建议 / Suggestions:

  1. 稍后重试 / Try again later
  2. 如果问题持续,请联系管理员 / If the issue persists, please contact the administrator
  3. 检查AI服务状态 / Check AI service status

错误详情 / Error Details: HTTPSConnectionPool(host='openharmonyinsight.cn', port=443): Max retries exceeded with url: /api/v1/review (Caused by ConnectTimeoutError(<HTTPSConnection(host='openharmonyinsight.cn', port=443) at 0x7f42e8664970>, 'Connection to openharmonyinsight.cn timed out. (connect timeout=10)'))

likedislike
openharmony_ciopenharmony_ci成员
19 天前 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
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
此处折叠了324条消息 查看更多
openharmony_ci
openharmony_ci成员
17 小时前 评论:

您好,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成员
17 小时前 关闭了关联的issue
openharmony_ciopenharmony_ci成员
17 小时前 合入了pull request,合并节点 SHA:ecedeb8d89a5ef5976d506d7e2c4555f595b77f4
openharmony_ciopenharmony_ci成员
17 小时前 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
17 小时前 添加了label:merged