已合并
fix: 加固安全敏感路径的错误处理 #103
fix: 加固安全敏感路径的错误处理 #103
已合并
yangqi创建于 5 天前
yangqi
5 天前

修改概述

  • 加强通信、共享内存、UBSE 和 dlock 路径中的空指针、数量、权限及整数边界校验
  • 确保错误路径正确释放或回滚锁、动态库状态、已分配列表、日志文件及其他资源
  • 将守护进程锁文件迁移至受保护的 systemd 运行时目录,并在进程生命周期内保持锁文件描述符有效
  • 移除私钥路径日志,并限制动态加载的 OpenSSL 符号作用域
  • 经全仓引用扫描确认无生产调用后,删除 ExecutorServicedg_ring_buffer、包装入口及仅验证其自身的单元测试
  • 为超长消息、空通道和损坏的记录 ID 链增加回归测试

关联并关闭:#61

兼容性影响

不涉及公共 API 或配置格式变更。IpcProxy::SyncCall 及内部转换辅助函数改用引用传参,属于内部实现调整。被删除的执行器和环形缓冲区代码不存在生产调用方。

验证情况

  • 使用 clang-format 18.1.8 完成格式化检查
  • 完成被删除执行器及环形缓冲区相关符号的全仓引用扫描,无残留引用
  • git diff --check 通过
  • GitCode 服务端推送钩子通过
  • openEuler CI 总门禁 #434 通过
  • x86_64 编译 #434 通过
  • aarch64 编译 #466 通过
  • DT #376 通过

协作者

Co-authored-by: Codex noreply@openai.com

likedislike
Pull Request已成功合入, 合并人@openeuler-ci-bot
(感谢 yangqi 的贡献)
Yyangqi
5 天前 创建了 pull request,commit 1fe00bfb
Yyangqi
5 天前 关联了issue:security: harden boundary checks and error-path cleanup
openeuler-ci-botopeneuler-ci-bot成员
5 天前 将hlinbo,shw030915,Gao_Victor,qmyyxtl,caowaterman设为审查人
openeuler-ci-botopeneuler-ci-bot成员
5 天前 添加了label:sig/sig-UB-ServiceCore
openeuler-ci-bot
openeuler-ci-bot成员
5 天前 评论:

Welcome To openEuler Community

Hey @yangqi12138 , thanks for your contribution to the community.

Bot Usage Manual

I'm the Bot here serving you. You can find the instructions on how to interact with me at Here . That means you can comment below every pull request or issue to trigger Bot Commands. You can self-configure the PR merge rules for this repository. For more details, please refer to Here.

Contact Guide

If you have any questions, please contact the SIG: sig-UB-ServiceCore ,
and any of the maintainers: @Gao_Victor, @caowaterman, @hlinbo, @qmyyxtl, @shw030915 ,
and any of the committers: @deepsleep_mem, @liu1103xwxw .

likedislike
此处折叠了152条消息 查看更多
deepsleep_mem
deepsleep_mem成员
2 天前 评论:

/lgtm

likedislike
openeuler-ci-botopeneuler-ci-bot成员
2 天前 添加了label:lgtm-deepsleep_mem
openeuler-ci-bot
openeuler-ci-bot成员
2 天前 评论:

Review Code Feedback

  • The label lgtm-deepsleep_mem was added to this pull request. It means that deepsleep_mem reviewed the code changes. 👋
Tips
  • If this pull request is not merged while all conditions are met, comment /check-pr to try again. 😄
likedislike
openeuler-ci-botopeneuler-ci-bot成员
2 天前 关闭了关联的issue
openeuler-ci-botopeneuler-ci-bot成员
2 天前 合入了pull request,合并节点 SHA:c73619da27ad211bc02db1045a7e2363a5eb87fa