0

我正在使用 UDP 套接字将许多客户端连接到一台服务器。在recvfrom()ing 客户端的登录数据包之后，我将它们存储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 条消息，而不是每个客户端都收到一条。我在这里做错了什么？

4

2 回答 2

2

基本问题是 struct addrinfo 包含一个指向struct sockaddr 的指针，显然您只初始化了一次。因此，每个 recvfrom() 都会覆盖相同的 sockaddr 数据（addrinfo.ai_addr），并且每个 userAddr[] 项都指向同一个。

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

于 2012-10-29T00:44:11.107 回答
1

除了其他人所说的，您正在实施的基本上是手动多播，那么为什么不直接使用实际的多播呢？这样，您的服务器可以将单个消息“sendto()”发送到单个多播 IP 组，并且操作系统将处理将消息的副本发送到已为您订阅该多播 IP 组的每个客户端。

于 2012-10-30T05:10:01.437 回答