[HUST CSE][components][net][sal] harden socket lifetime and close handling#11383
[HUST CSE][components][net][sal] harden socket lifetime and close handling#11383Telecaster2147 wants to merge 1 commit intoRT-Thread:masterfrom
Conversation
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
📌 Code Review Assignment🏷️ Tag: componentsReviewers: @Maihuanyi Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2026-05-10 23:20 CST)
📝 Review Instructions
|
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
当前 SAL socket 路径里,socket 描述符查表、对象使用、以及 close/reuse 之间缺少统一的生命周期约束。原有实现至少存在两类问题:
这类问题集中出现在
components/net/sal/src/sal_socket.c的 socket lookup、delete、close 和多条 API 入口路径中,影响 SAL 的健壮性和并发安全性,值得单独修复。你的解决方案是什么 (what is your solution)
本 PR 对 SAL socket 生命周期做了统一收敛,并补充对应回归测试:
struct sal_socket中增加显式生命周期状态和引用计数;sal_get_socket()改为“查找并持有临时引用”,新增sal_socket_put()负责释放;struct sal_socket的整个期间都成对持有/释放引用;components/net/utest/tc_sal_socket.c,覆盖 close 后继续访问、double close、close/use 竞态、socket cache reuse stress 等回归场景。该修改不改变正常 socket 路径的接口语义,主要把原先不受约束的关闭/复用竞态收敛为明确、可验证的失败返回和生命周期行为。
请提供验证的bsp和config (provide the config and bsp)
BSP:
bsp/qemu-vexpress-a9.config:
bsp/qemu-vexpress-a9/.config做基础编译回归验证;.config基础上追加:examples/utest/configs/default.cfgexamples/utest/configs/components/sal.cfgCONFIG_RT_USING_UTESTCASES=yCONFIG_RT_UTEST_TC_USING_SAL=y]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up