C:sendto()到多个客户端 - 所有消息都转到同一个客户端?

时间:2012-10-28 23:42:51

标签: c sockets udp

我使用UDP套接字将许多客户端连接到一台服务器。在recvfrom()客户端的登录数据包之后,我将他们的struct addrinfo *存储在一个数组中,如下所示:

struct addrinfo * userAddrs[64];

recvfrom(sockfd, req, 4096, 0, serverAddr->ai_addr, &serverAddr->ai_addrlen);
userAddrs[userindex] = (struct addrinfo *) malloc(sizeof(struct addrinfo));
memcpy(userAddrs[userindex], serverAddr, sizeof(struct addrinfo));

然后服务器从客户端接收消息并将其发送给该频道上的每个人:

// user x channel matrix. 1 means listening on that channel.
int userchannel_matrix[64][64];
for(int i = 0; i < 64; i++){
    if(userchannel_matrix[i][channelindex] == 1){
       sendto(sockfd, &textsay, sizeof (struct text_say), 0, userAddrs[i]->ai_addr, userAddrs[i]->ai_addrlen);
    }
}

然而,这最终会将所有消息发送到一个客户端。例如:如果三个客户端订阅了第4个通道,其中一个客户端发送了一条消息,则该客户端将收到所有3条消息,而不是每个客户端接收一条消息。我在这里做错了什么?

2 个答案:

答案 0 :(得分:2)

基本问题是struct addrinfo包含一个指针到struct sockaddr,你只清楚它只初始化一次。所以每个recvfrom()都会覆盖同一块sockaddr数据(addrinfo.ai_addr),每个userAddr []项都指向同一个数据。

正如@Troy建议的那样,你应该使用struct sockaddr。或者首先在数组中分配所有addrinfo结构,,每个结构都有自己的struct sockaddr * ai_addr指针。

答案 1 :(得分:1)

除了别人说的,你实现的基本上是手动组播,那么为什么不使用实际的组播呢?这样,您的服务器可以将单个消息“sendto()”发送到单个多播IP组,操作系统将处理将消息的副本发送给已为您订阅该多播IP组的每个客户端。