closesocket函数的bug

发布于 2019-02-26 19:03:33 浏览：2077 订阅该版

最近在项目中使用AT socket模块，网络应用层使用socket接口进行通信，测试发现当底层sal_closesocket调用失败时，导致socket文件不用被释放，

经过排查发现net_sockets.c函数中closesocket()函数中处理有问题，closesocket中应该修改成无认底层sal_closesocket关闭成功还是失败，
都应该对socket文件进行翻译，应该修改成如下就解决问题。

int closesocket(int s)
{
    int  error = 0;
    int socket = dfs_net_getsocket(s);
    struct dfs_fd *d;
    d = fd_get(s);
    if(!d)
    {
        return -1;
    }
    if (sal_closesocket(socket) == 0)
    {
        error = 0;
    }
    else
    {
        error = -1;
    }
    /* socket has been closed, delete it from file system fd */
    /*zhaoshimin 20190226 modify sometime sal_closesocket faile delete thn file system fd*/
    fd_put(d);
    fd_put(d);
    return error;
}

7 个回答

lihanchao90 2019-02-27

这家伙很懒，什么也没写！

[i=s] 本帖最后由 lihanchao90 于 2019-2-27 10:26 编辑 [/i]

victory这个原因找了好久都没有找到，膜拜下

flyboy 2019-02-27

Do my self();

膜拜楼主，顺便问一句楼主，提PR了吗;P

fhqmcu 认证专家 2019-03-05

个人博客：https://blog.csdn.net/fhqlongteng

膜拜楼主，顺便问一句楼主，提PR了吗

没有提交pr,怎么提交还不会？

Cheney_Chen 2019-03-05

这家伙很懒，什么也没写！

没有提交pr,怎么提交还不会？

这样关闭失败之后没有 fd_put(d) 会有什么异常现象吗？

Cheney_Chen 2019-03-05

这家伙很懒，什么也没写！

[i=s] 本帖最后由 Cheney_Chen 于 2019-3-5 19:41 编辑 [/i]

感谢楼主反馈，问题已经修复，可以查看这个 PR https://github.com/RT-Thread/rt-thread/pull/2382/files

fhqmcu 认证专家 2019-03-06

个人博客：https://blog.csdn.net/fhqlongteng

硬件连接的2G 或 4G模块，由于网络和模本身死机等问题很容易会出现底层socket关闭失败的情况，这样的话socket句柄就没办法释放，rtt系统默认最多分配16个socket，当出现16次异常关闭后，你就没法新分配socket了，你就再也连接不上网络了。

Cheney_Chen 2019-03-09

这家伙很懒，什么也没写！

硬件连接的2G 或 4G模块，由于网络和模本身死机等问题很容易会出现底层socket关闭失败的情况，这样的话sock …

嗯，上面的 PR 已经合并了，你更新最新 RT-Thread 源码看改动能不能解决你的问题。

撰写答案

关注者

0

被浏览

2.1k

关于作者

fhqmcu

个人博客：https://blog.csdn.net/fhqlongteng

提问

回答

被采纳

相关问题

1

请问用closesocket关闭不成功，返回值-1，是什么原因？

2

Closesocket报错

3

closesocket出错

4

closesocket函数调用会阻塞

5

长时间运行无法联网之sal_closesocket函数的bug

6

使用dfs v1， net_socket组件，socket ，closesocke,函数重复运行，发现内存会一直增加的问题。

本月文章贡献

出出啊

1 篇文章 6 次点赞

小小李sunny

1 篇文章 1 次点赞

张世争

1 篇文章 3 次点赞

crystal266

2 篇文章 1 次点赞

whj467467222

2 篇文章 1 次点赞

回到
顶部

发布
问题

手机
浏览

扫码手机浏览

回到
底部