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_大小类型时收到警告。

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

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

这是设置服务器套接字的函数。

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;
}

这是主要的代码。这就叫它们：

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

您注意到收到的警告是正确的：

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

你打折扣是错误的：

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

如果你看过的所有例子都使用与你类似的代码，那么你迫切需要找到更好的例子。然而，更重要的是，你需要学会依赖留档，最好是作为你的第一个求助对象，但至少要清除不确定性，比如“这个警告是我应该关心的吗？”（提示：除非你能表达一个基于留档的原因，否则答案总是“是的，我应该关心警告”。）

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

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

Answer 2

我在编译cli_大小类型时收到警告。

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

然而，我检查的所有示例都是这样通过的

不，他们不会。如果他们这样做了，他们会在编译时得到与您相同的错误。

这是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指向的结构的大小（字节）；返回时，它将包含对等地址的实际大小。