已关闭
feat(render_service): add delayed cleanup for render nodes after process death #30729
feat(render_service): add delayed cleanup for render nodes after process death #30729
已关闭
huaixu-y创建于 6月15日关闭于 7月14日
huaixu-y
huaixu-y
6月15日

Description:

Issue number:

Test & Result:

CodeCheck:

类型 自检项 自检结果
多线程 在类的成员变量中定义了vector/map/list等容器类型,且在多个成员函数中有操作时,需要加锁保护 自检结果:
定义全局变量,在多个函数中都有操作时,需要加锁保护 自检结果:
内存操作 调用外部接口时,确认是否对返回值做了判空判断,尤其外部接口返回了nullptr的情况,避免进程崩溃 自检结果:
内存操作优先使用安全函数,并检查其返回值 自检结果:
注意每个异常退出流程,是否都已经将资源释放(推荐使用RAII) 自检结果:
隐式内存分配场景:realpath、ReadParcelable序列化、cJSON相关函数时等,需主动释放或使用智能指针 自检结果:
外部输入 所有外部输入均不可信,需判断外部输入是否直接作为内存分配的大小,数组下标、循环条件、SQL查询等 自检结果:
注意外部字符串数据有无尾0 自检结果:
外部输入的路径不可信,需使用realpath做标准化处理,并判断路径的合法性 自检结果:
敏感信息 注意日志中打印敏感信息需匿名化 自检结果:
数学运算 代码中是否混合了加减乘除等运算,需检查是否可能导致整数溢出或符号翻转 自检结果:
初始化 类成员、局部变量使用前需初始化 自检结果:
权限管理 作为系统服务对外提供了接口(或RSCmd),是否做了权限保护和校验,只允许申请了权限的应用访问 自检结果:

L0新增用例自检结果

likedislike
当前Pull Request已关闭, 关闭人@huaixu-y
huaixu-yhuaixu-y
6月16日 审查状态已重置,审查人: aslklw,liuchao-huawei,white-dragon-tiger
huaixu-yhuaixu-y
6月16日 强制推送  1 个提交:cec13b3a-feat(render_service): add delayed cleanup for render nodes after process death
huaixu-yhuaixu-y
6月16日 审查状态已重置,审查人: aslklw,liuchao-huawei,white-dragon-tiger
huaixu-yhuaixu-y
6月16日 强制推送  1 个提交:495392e8-feat(render_service): add delayed cleanup for render nodes after process death
huaixu-yhuaixu-y
6月16日 审查状态已重置,审查人: aslklw,liuchao-huawei,white-dragon-tiger
此处折叠了61条消息 查看更多
openharmony_ci
openharmony_ci成员
6月26日 评论:

ISSUE关联关系有变化,之前验证结果无效,需要重新触发构建

likedislike
chuchengcheng
chuchengcheng成员
6月26日 评论:

🚨 🤖 AI 代码检视报告 🚨

总体评估: CRITICAL

问题统计:

  • 总问题数: 7
  • 严重问题: 3
  • 高危问题: 4

摘要:
该PR引入了延迟清理机制,但存在多处空指针解崩风险、多进程模式下缺少状态同步回退逻辑、以及清理任务状态管理逻辑错误等严重问题,需立即修复。

📊 详细报告
查看完整的审查详情,包括具体的问题描述、建议和代码位置:
🔗 查看详细报告


此评论由 OpenHarmony Insight 代码审查系统自动生成

likedislike
chuchengcheng
chuchengcheng成员6月26日进行代码检视1
rosen/modules/render_service/main/render_server/transaction/rs_client_to_service_connection.cpp
@@ -2639,0 +2703,4 @@
2703+ return ERR_INVALID_VALUE;
2704+ }
2705+ renderServiceAgent_->SetDelayedCleanupEnabled(remotePid_, enabled);
2706+ BroadcastDelayedCleanupEnabled(remotePid_, enabled);
chuchengcheng
chuchengcheng6月26日评论:

这里如果存在多个RP时,会往多个RP中插入RemotePid,但是RemotePid清理不掉,可能存在内存泄露

likedislike
openharmony_dcp
openharmony_dcp成员
7月13日 评论:

您好, @huaixu-y 该PR需要您响应,已过去25天未响应,请根据检视意见进行修改,如5天内未响应检视意见,此PR会被自动关闭。关闭后的PR,如有需要,您可以自行打开该PR。

likedislike
huaixu-yhuaixu-y
7月14日 关闭了 pull request