已合并
sync client code: connect hub manages multi render-process connections keyed by tokenMaskId #30606
sync client code: connect hub manages multi render-process connections keyed by tokenMaskId #30606
已合并
LyBbq创建于 6月8日
LyBbq成员
6月8日

Description:

问题背景:

客户端连接中心同步(内部代码同步至社区仓):RSRenderServiceConnectHub 原以 map<token, renderProcess> 管理渲染进程连接,仅支持单连接视角,缺少多渲染进程(同层渲染等场景)的选路与死亡感知能力。

修复方案:

  1. connRenderProcesses_ 改为 map<tokenMaskId, RenderProcessInfo>(token / connectToRenderRemote / renderProcess / deathRecipient / clientToRenderConnection 打包),按 tokenMaskId 管理多条渲染进程连接
  2. IPC 接口适配:RSIConnectToRenderProcess::CreateRenderConnection 增加 tokenMaskId 入参并返回 {connection, tokenMaskId},RemoveConnection 改按 tokenMaskId 删除,proxy/stub 同步修改
  3. 新增 RSConnectRenderProcessDeathRecipient:渲染进程死亡时按 tokenMaskId 清理对应连接并触发 SetOnRenderProcessDiedCallback(tokenMaskId, cb) 注册的回调;ConnectRenderProcessDied(tokenMaskId) 完成善后
  4. 新增按 tokenMaskId 的查询/选路接口:GetClientToRenderConnection(tokenMaskId)、GetDefaultTokenMaskId()、GetRenderProcessTokenMaskId()、FindClientToRenderConnection() 等,未适配多实例的旧接口暂走默认连接

Issue number:https://gitcode.com/openharmony/graphic_graphic_2d/issues/24329

Test & Result:

  • 门禁通过:编译、静态检查、冒烟测试均成功,已合入 master
  • 多渲染进程连接建立 / 按 tokenMaskId 选路 / 进程死亡回调与连接清理功能验证无回归

CodeCheck:

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

L0新增用例自检结果

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 LyBbq 的贡献)
LLyBbq成员
6月8日 关联了issue:[Bug]: SyncClient
LLyBbq成员
6月8日 审查状态已重置,审查人: aslklw,huang_kaixing,white-dragon-tiger
LLyBbq成员
6月8日 推送  64 个提交:60b8c9ef-63 commits from branch master,a8ce8223-conflict
openharmony_ciopenharmony_ci成员
6月8日 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
6月8日 评论:

感谢提交 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
此处折叠了58条消息 查看更多
openharmony_ciopenharmony_ci成员
6月10日 合入了pull request,合并节点 SHA:9562d2c2f759bc70ee47bcf483e375725b133b68
openharmony_ciopenharmony_ci成员
6月10日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
6月10日 添加了label:merged
LLyBbq成员
8月31日 修改标题为 “sync client code: connect hub manages multi render-process connections keyed by tokenMaskId”,原标题为“sync code”
LLyBbq成员
8月31日 修改了pull request 的描述