已合并
add screen power manager #19706
add screen power manager #19706
已合并
wulong158创建于 7月18日
wulong158
wulong158
7月18日

Description:

Issue number:

Test & Result:

CodeCheck:

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

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

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

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

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

当前检测到如下Commits的Signed-off-by邮箱未签署DCO协议:


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 Signed-off-by emails in the following commits have not signed the DCO:

likedislike
openharmony_ciopenharmony_ci成员
7月18日 添加了label:waiting_on_author
openharmony_ciopenharmony_ci成员
7月18日 添加了label:dco检查失败
wulong158wulong158
7月18日 关联了issue:[Bug]: add screen power manager
wulong158wulong158
7月18日 强制推送  1 个提交:8b5dcdf4-add screen power manager
openharmony_ci
openharmony_ci成员
7月18日 评论:

感谢提交 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成员
7月18日 删除了label:dco检查失败
openharmony_ciopenharmony_ci成员
7月18日 添加了label:dco检查成功
wulong158
wulong158
7月18日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
7月18日 评论:

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

likedislike
openharmony_ciopenharmony_ci成员
7月18日 添加了label:编译失败
openharmony_ciopenharmony_ci成员
7月18日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
7月18日 添加了label:冒烟测试成功
openharmony_ci
openharmony_ci成员
7月18日 评论:

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

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 ohos-sdk success NA >>>
2 dayu200 success success >>>
3 dayu200_tdd failed(compile failed) NA >>>
4 part_compile success(IGNORE) NA >>>
5 master_inner_build failed(IGNORE)(联合门禁失败) NA >>>
6 mac-sdk success NA >>>
7 ohos-host_mini_tdd success NA >>>
8 x86_64_virt success NA >>>

likedislike
wulong158wulong158
7月18日 强制推送  1 个提交:bbc30c22-add screen power manager
openharmony_ciopenharmony_ci成员
7月18日 删除了label:编译失败
openharmony_ciopenharmony_ci成员
7月18日 删除了label:静态检查成功
openharmony_ciopenharmony_ci成员
7月18日 删除了label:冒烟测试成功
openharmony_ci
openharmony_ci成员
7月18日 评论:

代码有更新,重置PR验证状态

likedislike
openharmony_ci
openharmony_ci成员
7月18日 评论:

感谢提交 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
wulong158
wulong158
7月18日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
7月18日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译和测试【dayu200_tdd编译, ohos-host_mini_tdd编译, x86_64_virt编译, master_inner_build编译, ohos-sdk编译, part_compile编译, dayu200测试, dayu200编译, mac-sdk编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a5aeba864650f998b4cbbfd/runlist

likedislike
wulong158wulong158
7月18日 强制推送  1 个提交:47882557-add screen power manager
openharmony_ci
openharmony_ci成员
7月18日 评论:

监测到PR: https://gitcode.com/openharmony/window_window_manager/merge_requests/19706 代码有更新,当前正在执行的门禁任务已自动中止,如有需要,可重新触发

likedislike
openharmony_ci
openharmony_ci成员
7月18日 评论:

感谢提交 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
wulong158
wulong158
7月18日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
7月18日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译和测试【dayu200编译, master_inner_build编译, x86_64_virt编译, part_compile编译, dayu200_tdd编译, mac-sdk编译, dayu200测试, ohos-sdk编译, ohos-host_mini_tdd编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a5af16c64650f998b4e8488/runlist

likedislike
openharmony_ciopenharmony_ci成员
7月18日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
7月18日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
7月18日 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
7月18日 通过测试
openharmony_ci
openharmony_ci成员
7月18日 评论:

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

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 ohos-sdk success NA >>>
2 dayu200 success success >>>
3 dayu200_tdd success NA >>>
4 part_compile success(IGNORE) NA >>>
5 master_inner_build failed(IGNORE)(联合门禁失败) NA >>>
6 mac-sdk success NA >>>
7 ohos-host_mini_tdd success NA >>>
8 x86_64_virt success NA >>>

likedislike
openharmony_ciopenharmony_ci成员
7月18日 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
7月18日 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
7月18日 评论:

您好,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
zhaoliang成员7月18日进行代码检视1
window_scene/screen_session_manager/infra/src/fold_screen/fold_screen_base_policy.cpp
@@ -124,6 +124,7 @@ void FoldScreenBasePolicy::SetTpFeatureConfig(int32_t tpType, const std::string&
124124{
125125#ifdef TP_FEATURE_ENABLE
126126 if (isDefaultConfigType) {
127+ PreProcessTP();
zhaoliang7月18日评论:

前置处理问什么加到这里?

likedislike
Zzhaoliang成员
7月18日 解决了最后一个问题
Zzhaoliang成员
7月18日 通过审查
openharmony_ciopenharmony_ci成员
7月18日 关闭了关联的issue
openharmony_ciopenharmony_ci成员
7月18日 合入了pull request,合并节点 SHA:3ad2e26d774ac86385a8873e525a69806377e463
openharmony_ciopenharmony_ci成员
7月18日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
7月18日 添加了label:merged