C语言跨平台socket编程connect()返回错误88（无效参数）排查

我正在实现一款用于传输简单数据的跨平台网络程序，Windows端采用Winsock开发，Unix/Linux端采用标准Linux套接字接口开发。目前Windows端程序运行完全正常，Linux端服务端程序也可正常工作，但Linux客户端调用connect()函数时执行失败，返回错误码88：无效参数（invalid argument），相关实现代码如下：

netsend.h #

#include <sys/socket.h>
#include <netinet/in.h>
#include <arpa/inet.h>
#include <unistd.h>

struct netsend_data {
    int fd, error_code, sock, new_sock, opt;
    struct sockaddr_in server, client;
};

// 调用其他接口前必须先初始化。返回0=成功，返回-1=出错（可检查error_code）。创建客户端socket时server参数传0，创建服务端socket时传1
int netsend_init(struct netsend_data* data, const char* peer_ip_addr, int port, int server);

// 仅客户端使用，连接远端服务端
int netsend_connect(struct netsend_data* data);

netsend.c #

// 调用其他接口前必须先初始化。返回0=成功，返回-1=出错（可检查error_code）。创建客户端socket时server参数传0，创建服务端socket时传1
int netsend_init(struct netsend_data* data, const char* peer_ip_addr, int port, int server) {
    data->sock = 0;
    if (server) {   
        if (data->fd = socket(AF_INET, SOCK_STREAM, 0) < 0) {
            data->error_code = errno;
            return -1;
        }
    }
    else {  
        if (data->sock = socket(AF_INET, SOCK_STREAM, 0) < 0) {
            data->error_code = errno;
            return -1;
        }
    }
    
    // 配置地址信息
    if (server) {   
        data->server.sin_addr.s_addr = INADDR_ANY;
    }
    else {  
        inet_pton(AF_INET, peer_ip_addr, &data->server.sin_addr);
    }
    data->opt = 1;
    data->server.sin_family = AF_INET;
    data->server.sin_port = htons(port);
    
    return 0;
}

// 仅客户端使用，连接远端服务端
int netsend_connect(struct netsend_data* data) {
    // 连接远端服务端
    if (connect(data->sock, (struct sockaddr*)&data->server, sizeof(data->server)) < 0) {
        data->error_code = errno;
        return -1;
    }   
    
    return 0;
}

client.c #

#include "netsend.h"
#include <stdio.h>

int main(int argc, char** argv) {   
    struct netsend_data data;   
    if (netsend_init(&data, "192.168.0.117", 8888, 0) < 0) {
        printf("Error: %i", data.error_code);
    }
    
    if (netsend_connect(&data) < 0) {
        printf("Error: %i", data.error_code);
    }
    
    if (netsend_send_data(&data, "Hello from client", 0) < 0) {
        printf("Error: %i", data.error_code);
    }
    
    char response[50];
    if (netsend_recv_data(&data, response, 50, 0) < 0) {
        printf("Error: %i", data.error_code);
    }

    printf("%s", response);
    
    netsend_close(&data);
}

核心是C语言运算符优先级的经典错误：比较运算符<优先级高于赋值运算符=，创建socket的判断逻辑写法有误。

看客户端分支的这行代码：

if (data->sock = socket(AF_INET, SOCK_STREAM, 0) < 0)

实际执行顺序是先判断socket()返回值是否小于0，得到0（假）或1（真）的布尔结果，再把这个结果赋值给data->sock。socket创建成功时返回值是正整数，判断结果为0，所以data->sock最终被赋值为0，不是真实的socket文件描述符。后续调用connect()时传入的第一个参数是0，对应标准输入流，不是合法套接字，内核直接返回错误码88（EINVAL，无效参数），和遇到的报错完全吻合。

服务端分支的data->fd = socket(...) <0存在完全相同的bug，只是目前还没执行到后续bind、accept逻辑，问题没有暴露。

给socket赋值语句加括号，强制先完成赋值再做错误判断，修改如下：

// 服务端socket创建判断
if ((data->fd = socket(AF_INET, SOCK_STREAM, 0)) < 0) {
    data->error_code = errno;
    return -1;
}

// 客户端socket创建判断
if ((data->sock = socket(AF_INET, SOCK_STREAM, 0)) < 0) {
    data->error_code = errno;
    return -1;
}

另外两个需要处理的潜在隐患：

• 初始化netsend_data结构体时，先用memset(data, 0, sizeof(*data))清空所有字段，避免栈上随机值引发未知问题
• 增加inet_pton()的返回值校验，IP地址格式非法时该函数会返回0，直接使用会导致后续connect失败

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