Socket编程：封装函数后出现bad file descriptor错误求助

你遇到的确实是C语言值传递的基础坑，这在重新捡起来写代码的初学者里太常见了，咱们一步步拆解原因和解决办法：

问题根源：值传递导致的变量副本失效 #

在C语言里，默认的参数传递是值传递——函数会创建传入变量的副本，函数内部对副本的修改完全不会影响到外面的原始变量。你的open_socket函数犯了两个核心错误：

1. socket文件描述符s_fd是值传递：函数内部通过s_fd = socket(...)给副本赋了有效的文件描述符，但外面的socket_fd还是初始的垃圾值，调用sendto时自然会报"无效文件描述符"。
2. struct sockaddr_in s_addr也是值传递：你在函数里修改了地址结构体的内容，但这些修改只作用在副本上，外面的serv_addr还是未正确初始化的状态。

修复方案：用指针传递实现外部变量修改 #

要让函数能真正修改外部的变量，必须传递变量的指针（也就是内存地址），这样函数内部就能直接操作原始变量的内存空间。

修改后的open_socket函数 #

// 改为返回int类型，用错误码标识操作结果，方便调用处判断
int open_socket( int *s_fd, struct sockaddr_in *s_addr, struct timeval tv, char* ID_IP, int ID_Port) {
    // 通过指针操作外部的socket_fd变量
    if ( (*s_fd = socket(AF_INET, SOCK_DGRAM, 0) ) < 0 ) { 
        perror("ERROR opening socket");
        return -1;
    }
    // 操作指针指向的外部s_addr结构体
    s_addr->sin_family = AF_INET;
    s_addr->sin_port = htons(ID_Port);
    // 原代码里先设INADDR_ANY再用inet_pton覆盖，这步完全多余，直接删掉
    setsockopt(*s_fd, SOL_SOCKET, SO_RCVTIMEO, (const char *)&tv , sizeof (struct timeval));
    
    if(inet_pton(AF_INET, ID_IP, &s_addr->sin_addr)<=0) { 
        perror("\nInvalid address/ Address not supported");
        close(*s_fd); // 失败时要关闭已创建的socket，避免资源泄漏
        return -1;
    }
    if (connect(*s_fd, (struct sockaddr *)s_addr, sizeof(*s_addr)) < 0) { 
        perror("\nConnection Failed");
        close(*s_fd);
        return -1;
    }
    printf("Socket Connected\n" );
    return 0; // 成功返回0
}

修改后的调用代码 #

// SOCKET 1
struct sockaddr_in serv_addr; 
int serv_len=sizeof(serv_addr);
int socket_fd; // 无需提前赋值，函数会通过指针修改它
struct timeval tv;
tv.tv_sec=1 ; tv.tv_usec=0;

// 传递变量的地址（用&符号取地址）
if(open_socket( &socket_fd, &serv_addr, tv, SERVER, PORT) != 0){
    printf("Failed to open socket\n");
    return -1;
}

// 现在sendto可以正常使用了
if ( (sendto(socket_fd , cmd_final, BUFFLEN , 0 , (struct sockaddr *)&serv_addr, serv_len) ) < 0 ) {
    perror("ERROR > send cmd failed : ");
    close(socket_fd); // 出错时记得关闭socket
    return 0;
}

额外优化小建议 #

• 用perror代替单纯的printf：它会自动拼接系统错误信息，排查问题更高效。
• 错误处理时记得释放资源：比如socket创建成功后，后续步骤失败一定要close，避免资源泄漏。

内容的提问来源于stack exchange，提问作者HacHac