已合并
fix screen power status issue #17704
fix screen power status issue #17704
已合并
wulong158创建于 1月23日
wulong158
wulong158
1月23日

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 的贡献)
wulong158wulong158
1月23日 关联了issue:[Bug]: fix screen power status issue
openharmony_ciopenharmony_ci成员
1月23日 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
1月23日 评论:

感谢提交 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成员
1月23日 添加了label:dco检查成功
wulong158
wulong158
1月23日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
1月23日 评论:

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

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

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

静态检查:

# check type result report
1 codeCheck pass >>>

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

likedislike
wulong158
wulong1581月23日进行代码检视2
window_scene/screen_session_manager/src/screen_session_manager.cpp
已过期
@@ -14437,2 +14468,4 @@
14468+}
1443714469// LCOV_EXCL_STOP
1443814470} // namespace OHOS::Rosen
wulong158
wulong1581月23日评论:

日志带上关键信息

likedislike
System
系统消息系统
1月23日 评论:

changed this line on f2826621 view diff detail

wulong158wulong158
1月23日 强制推送  1 个提交:f2826621-fix screen power status issue
wulong158wulong158
1月23日 解决了最后一个问题
openharmony_ciopenharmony_ci成员
1月23日 删除了label:静态检查成功
openharmony_ciopenharmony_ci成员
1月23日 删除了label:编译失败
openharmony_ciopenharmony_ci成员
1月23日 删除了label:冒烟测试失败
openharmony_ci
openharmony_ci成员
1月23日 评论:

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

likedislike
wulong158
wulong158
1月23日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
1月23日 评论:

感谢提交 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_ci
openharmony_ci成员
1月23日 评论:

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

likedislike
wulong158wulong158
1月23日 强制推送  1 个提交:2622a115-fix screen power status issue
openharmony_ci
openharmony_ci成员
1月23日 评论:

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

likedislike
openharmony_ci
openharmony_ci成员
1月23日 评论:

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

start build

likedislike
openharmony_ci
openharmony_ci成员
1月23日 评论:

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

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

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/697325ce64650f998b2997dc/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 success(IGNORE) >>>
5 master_inner_build failed(IGNORE)(联合门禁失败) NA >>>
6 mac-sdk success NA >>>
7 dayu200-codearts failed(IGNORE)(skip build) NA >>>

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

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


Hello, Committer @nyankomiya @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
openharmony_dcp
openharmony_dcp成员
1月23日 评论:

提交次数已达到或超过3次,为了保证上库效率和减少构建资源浪费,请点击链接观看学习视频,谢谢!https://dcp.openharmony.cn/workbench/video/videoDisplay

likedislike
hejunfei1991hejunfei1991成员
1月23日 通过审查
openharmony_ciopenharmony_ci成员
1月23日 关闭了关联的issue
openharmony_ciopenharmony_ci成员
1月23日 合入了pull request,合并节点 SHA:4fe4fe501c90fbee9c5e17ab63f716b2f9969c71
openharmony_ciopenharmony_ci成员
1月23日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
1月23日 添加了label:merged