🐞 fix(at_socket):修复socket 在开启server下的connect连接#10967
Conversation
修复accept在错误执行过程中返回无效socket问题
|
👋 感谢您对 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)
🏷️ Tag: components_net_atReviewers: @Ryan-CW-Code @lizhen9880 Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2025-11-21 11:25 CST)
📝 Review Instructions
|
| char *socket_info = RT_NULL; | ||
| int base_socket = 0; | ||
|
|
||
| rt_sscanf(buff, "SOCKET:%d", &base_socket); |
There was a problem hiding this comment.
这里直接读取缓冲区是没法兼容很多AT设备的
There was a problem hiding this comment.
我只是把下面的 往前提了一下 ,您说的应该是当前组件的问题 ,不是我新引入的问题
There was a problem hiding this comment.
哦哦,那应该没什么问题。 我看目前也就esp适配了at_server接口,可以先合并进去。
修复accept在错误执行过程中返回无效socket问题
拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
🐞 fix(at_socket):修复socket 在开启server下的connect连接
修复accept在错误执行过程中返回无效socket问题
你的解决方案是什么 (what is your solution)
1.创建socket connect时已经创建了 连接成功指示时应当先检查是否存在
2. 当异常执行时 accept 返回的空指针指向的socket为无效数值,应返回<0的数值
在esp32下测试通过
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 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