TCP-IP套接字C：尝试连接到服务器套接字时出现错误的地址错误

Question

我正在尝试创建客户端服务器井字游戏作为家庭作业。 我在单独的功能中而不是在主程序中设置服务器侦听器和连接。 服务器套接字创建成功，但是客户端连接失败，我认为这是导致随后的seg错误的原因。

我已经查找了类似的问题，并尽可能地对代码进行了调整，例如为sizeof（cli_addr）设置变量，而不是直接将其传递给accept（）。 我不知道为什么它不起作用。

这是建立连接的功能

void setup_connections(int serversocket,int *clientsocket,int portnum){
int cli_size,con_num = 0;
struct sockaddr_in cli_addr;

//server is listening for clients
listen(serversocket,5);

while(con_num<2){


    cli_addr.sin_family = AF_INET;
    cli_addr.sin_addr.s_addr = INADDR_ANY;
    cli_addr.sin_port = htons(portnum);

    //accept connection while creating client socket
    cli_size = sizeof(cli_addr);
    clientsocket[con_num] = accept(serversocket,(struct sockaddr *) &cli_addr,cli_size);

    if(clientsocket[con_num]<0){
        perror("Error: ");
    }

    con_num++;

    }
}

编译cli_size类型时收到警告。

*警告：不兼容的整数到指针的转换，将'int'传递给'socklen_t *'类型的参数（又名'unsigned int *'）[-Wint-conversion] clientsocket [con_num] = accept（serversocket，（struct sockaddr ）＆cli_addr，cli_size ）;

但是，我检查的所有示例都通过了这样的检查。

这是设置服务器套接字的功能。

int setup_server(int portnum){
int serversocket,serv_bind;
struct sockaddr_in server_addr;


serversocket = socket(AF_INET, SOCK_STREAM, 0);
if(serversocket<0){
    printf("Failed to create server\n");
}

//set up server info
server_addr.sin_family = AF_INET;
server_addr.sin_addr.s_addr = INADDR_ANY;
server_addr.sin_port = htons(portnum); //port number given by user

//link server address to socket
serv_bind = bind(serversocket,(struct sockaddr *) &server_addr,sizeof(server_addr));
if(serv_bind<0){
    printf("Failed to bind\n");
}

return serversocket;
}

这是main.c中调用它们的代码：

int portnum,serversocket, clientsocket[2]; //sockets


portnum = atoi(argv[1]);
if(argc < 2){
    printf("Port number not given");
}

//create sockets
serversocket = setup_server(portnum);
setup_connections(serversocket,clientsocket);

运行该程序后，我得到的是：

错误：地址错误

错误：地址错误

细分错误：11

如果您可以向我解释我做错了什么，那将非常有帮助。

Answer 1

编译cli_size类型时收到警告。

*警告：不兼容的整数到指针的转换，将'int'传递给'socklen_t *'类型的参数（又名'unsigned int *'）[-Wint-conversion] clientsocket [con_num] = accept（serversocket，（struct sockaddr）＆cli_addr，cli_size ）;

但是，我检查过的所有示例都像这样通过

不，他们没有。 如果这样做，它们在编译时将得到与您相同的错误。

这是可以accept的原型

int accept(int sockfd, struct sockaddr *addr, socklen_t *addrlen);
//                                                      ^ Note the asterisk

您可以清楚地看到它期望指向socklen_t 而不是 socklen_t的指针。 您的accept行应如下所示：

clientsocket[con_num] = accept(serversocket,(struct sockaddr *) &cli_addr,&cli_size); 
//                                                                        ^ Note the pointer indirection

如果要使用cli_addr结构，则应检查返回的大小是否大于传递的大小，因为如果是，则意味着地址结构已被截断。

这是Linux手册页所说的

addrlen参数是一个值结果参数：调用者必须对其进行初始化，以包含addr指向的结构的大小（以字节为单位）； 返回时，它将包含对等地址的实际大小。

Answer 2

您应该注意收到的警告：

警告：指针转换不兼容的整数，将'int'传递给类型为'socklen_t *'的参数（又名'unsigned int *'）[-Wint-conversion] clientsocket [con_num] = accept（serversocket，（struct sockaddr）＆cli_addr，cli_size） ;

打折是错误的：

但是，我检查的所有示例都通过了这样的检查。

如果您查看的所有示例都使用与您类似的代码，那么您迫切需要找到更好的示例。 但是，更重要的是，您需要学会依赖文档 ，最好是作为第一个求助手段，但是至少要消除不确定性，例如“此警告是否值得我关注？” （提示：直到并且除非您可以表达基于文档的原因，否则答案始终是“是，我应该注意警告”。）

快速浏览一下accept()的文档，您会发现第三个参数应该是指向 包含您传递的地址对象大小的变量的指针 ，并且该函数将更新该变量（通过指针）以包含返回地址的实际长度。 相反，您传递的是大小本身，几乎可以肯定，转换为指针将产生无效的指针。 您实际上很幸运，它确实这样做了，所以您遇到了运行时错误，因为另一种选择是您在程序的某个随机位置无提示地产生了内存损坏。

请注意， bind()是不同的。 它要求您直接而不是间接传递地址的大小。 这是明智的，因为没有理由要修改大小。