已合并
delete redundant code #18378
delete redundant code #18378
已合并
wulong158创建于 3月10日
wulong158
wulong158成员
3月10日

Description:

Issue number:

Test & Result:

CodeCheck:

类型自检项自检结果
多线程相关在类的成员变量中定义了vector/map/list等容器类型,且在多个成员函数中有操作时,需要加锁保护自检结果:pass
定义全局变量,在多个函数中都有操作时,需要加锁保护自检结果:
内存相关调用外部接口时,确认是否对返回值做了判断,尤其外部接口返回了nullptr的情况,避免进程崩溃自检结果:pass
调用安全函数时,如memcpy_s等,是否检查其返回值自检结果:pass
检查函数中是否涉及了内存或资源申请(如文件句柄),注意每个异常退出流程,是否都已经将资源释放(推荐使用RAII)自检结果:pass
隐式内存分配场景:realpath、ReadParcelable序列化、cJSON相关函数时等,需主动释放或使用智能指针自检结果:pass
校验外部输入使用nlohmann:json解析外部输入时,需判断参数类型是否符合预期自检结果:pass
所有外部输入均不可信,需判断外部输入是否直接作为内存分配的大小,数组下标、循环条件、SQL查询等自检结果:pass
外部输入的路径不可信,需使用realpath做标准化处理,并判断路径的合法性自检结果:pass
外部输入包括对外提供的接口,IPC的proxy/stub接口,序列化/反序列化接口等自检结果:pass
数学运算代码中是否混合了加减乘除等运算,需检查是否可能导致整数溢出或符号翻转自检结果:pass
需检查代码是否有高精度数字转换为低精度的操作,如果必须,建议使用C++安全类型转换接口自检结果:pass
检查代码在计算时是否有除零操作(包括除数是计算出来的结果可能为0的情况)自检结果:pass
权限相关作为系统服务对外提供了接口,是否做了权限保护和校验(如需要),只允许申请了权限的应用访问自检结果:pass
提供给其他系统服务的接口默认需要做SA服务校验自检结果:pass
跨进程通信优先使用异步IPC,若必须使用同步IPC需要考虑对端卡死或高延时影响自检结果:pass
序列化/反序列化中数据读写顺序要严格对齐自检结果:pass
likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 wulong158 的贡献)
wulong158wulong158成员
3月10日 关联了issue:[Bug]: 删除DMS冗余代码
openharmony_ciopenharmony_ci成员
3月10日 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
3月10日 评论:

感谢提交 Pull Requests !此PR未通过DCO校验。
校验失败可能原因:

1. 未签署“DCO协议”(开发者原创声明协议),在线签署、查看签署状态。

2. Commits 中未包含 Signed-off-by信息,参考FAQ处理。

修复上述问题后,在PR的评论框输入“check dco” ,单击”评论”,系统将再次进行DCO校验。

当前检测到如下Commits 未包含Signed-off-by信息:


Thanks for submitting a pull request. This pull request has not passed the DCO check.
Possible causes:

1. You have not signed the Developer Certificate of Origin (DCO). Sign the DCO and check DCO status.

2. The commits do not contain the Signed-off-by information. To resolve this issue, see FAQs.

After resolving the preceding issues, enter check dco in the comment box of this pull request and click Comment. The system will check DCO status again.

The following commits do not contain the Signed-off-by information:

likedislike
openharmony_ciopenharmony_ci成员
3月10日 添加了label:dco检查失败
wulong158wulong158成员
3月10日 强制推送  1 个提交:d834284e-delete redundant code
此处折叠了49条消息 查看更多
hejunfei1991hejunfei1991
3月11日 通过审查
openharmony_ciopenharmony_ci成员
3月11日 关闭了关联的issue
openharmony_ciopenharmony_ci成员
3月11日 合入了pull request,合并节点 SHA:ac0a5d3e72b782ca6d122a2d913ce8e9ecda7bc8
openharmony_ciopenharmony_ci成员
3月11日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
3月11日 添加了label:merged