我使用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条消息,而不是每个客户端接收一条消息。我在这里做错了什么?
答案 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组的每个客户端。