已合并
闪控球/窗更新时保留绑定属性 #19762
闪控球/窗更新时保留绑定属性 #19762
已合并
loumou创建于 7月23日
loumou
7月23日

Description:
https://gitcode.com/openharmony/window_window_manager/pull/19762
Issue number:
#15209
Test & Result:

CodeCheck:

不涉及不涉及
类型自检项自检结果
多线程相关在类的成员变量中定义了vector/map/list等容器类型,且在多个成员函数中有操作时,需要加锁保护自检结果:PASS
定义全局变量,在多个函数中都有操作时,需要加锁保护自检结果: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
(感谢 loumou 的贡献)
Lloumou
7月23日 关联了issue:[Bug]: 闪控球/窗更新时保留绑定属性
openharmony_ci
openharmony_ci成员
7月23日 评论:

感谢提交 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成员
7月23日 添加了label:waiting_on_author
openharmony_ciopenharmony_ci成员
7月23日 添加了label:dco检查失败
Lloumou
7月23日 修改了pull request 的描述
此处折叠了57条消息 查看更多
openharmony_ci
openharmony_ci成员
7月25日 评论:

您好,Committer @nyankomiya @logn @chyyy0213 @hejunfei1991 @stupidb @lollipopolive @JUGaaab ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @nyankomiya @logn @chyyy0213 @hejunfei1991 @stupidb @lollipopolive @JUGaaab . 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
JJUGaaab成员
7月25日 通过审查
openharmony_ciopenharmony_ci成员
7月25日 合入了pull request,合并节点 SHA:1d26c39c642d3a0de34f5ec27c3fd91d13126abd
openharmony_ciopenharmony_ci成员
7月25日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
7月25日 添加了label:merged