已合并
7.0 #861
已合并
hlzhangaj创建于 8月5日
hlzhangaj
hlzhangaj
8月5日

IssueNo: https://gitcode.com/openharmony/ability_ability_base/issues/1224

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用例

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 hlzhangaj 的贡献)
openharmony_ciopenharmony_ci成员
8月5日 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
8月5日 评论:

感谢提交 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成员
8月5日 添加了label:dco检查成功
littlejerry1littlejerry1成员
20 天前 通过审查
openharmony_ci
openharmony_ci成员
20 天前 评论:

该提交没有关联任何Issue,请先关联一个Issue后重新触发构建或合入
The PR is not associated with any Issue, Please associate the PR with one Issue and retrigger build or submit.

likedislike
hlzhangajhlzhangaj
14 天前 修改了pull request 的描述
hlzhangaj
hlzhangaj
14 天前 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
14 天前 评论:

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

likedislike
openharmony_ciopenharmony_ci成员
14 天前 添加了label:编译成功
openharmony_ciopenharmony_ci成员
14 天前 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
14 天前 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
14 天前 通过测试
openharmony_ci
openharmony_ci成员
14 天前 评论:

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

静态检查:

# check type result report
1 codeCheck pass >>>

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

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

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


Hello, Committer @jayleehw @uniquexch @littlejerry1 @ccllee1 @yangxuguang-huawei . 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成员
14 天前 关闭了关联的issue
openharmony_ciopenharmony_ci成员
14 天前 合入了pull request,合并节点 SHA:c9fca7d46eaaee68faafe728635d32c1ce092c2c
openharmony_ciopenharmony_ci成员
14 天前 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
14 天前 添加了label:merged