已合并
feat: AgentCli #20359
feat: AgentCli #20359
已合并
yangxuguang-huawei创建于 9月1日
yangxuguang-huawei
yangxuguang-huawei成员
9月1日

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

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
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 yangxuguang-huawei 的贡献)
yangxuguang-huaweiyangxuguang-huawei成员
9月1日 关联了issue:[新需求]: 提供agent-cli工具
afwk_helper成员
9月1日 评论:

开始进行AI检视!

AI review has been started, please wait...

likedislike
afwk_helper成员
9月1日 评论:
check type result report
start ai_review pass -
likedislike
openharmony_ciopenharmony_ci成员
9月1日 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
9月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
此处折叠了116条消息 查看更多
openharmony_ci
openharmony_ci成员
9月10日 评论:

您好,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成员
9月10日 关闭了关联的issue
openharmony_ciopenharmony_ci成员
9月10日 合入了pull request,合并节点 SHA:75d386bf64350053356bbe3ec1e31c3d926b6b50
openharmony_ciopenharmony_ci成员
9月10日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
9月10日 添加了label:merged