2

我正在编写一个应该执行一些操作的函数，然后返回（使用它的参数）与函数内部交互（即使用的sendto）设备的地址recvfrom。

这是我调用函数的方式，cliaddr之前实例化。

struct sockaddr_in cliaddr;
memset((void *) &cliaddr, 0, sizeof(cliaddr));
rcv_string(sockfd, &return_string, &cliaddr);
// Here I'll need to use cliaddr, that's why I need it outside too

下面是函数的实现：

int rcv_string(int sockfd, char **return_string, struct sockaddr_in *sndaddr) {
    // ...
    char buff[PACKETSZ + 2];
    memset(&buff, 0, sizeof(buff)); // Clean buffer
    socklen_t *plen = malloc(sizeof(struct sockaddr *));
    if ((recvfrom(sockfd, buff, sizeof(buff), 0, (struct sockaddr *) sndaddr, plen)) < 0) {
            perror("recvfrom");
            return -1;
    }
    char *snd_ip = malloc(INET_ADDRSTRLEN * sizeof(char));
    if (inet_ntop(AF_INET, &((*sndaddr).sin_addr.s_addr), snd_ip, INET_ADDRSTRLEN * sizeof(char)) == NULL) {
            perror("inet_ntop");
            return -1;
    }
    printf("Received '%s' from '%s'.\n", buff, snd_ip);
    // ...
}

现在，即使发送设备的 IP 地址是192.168.1.251我得到以下输出：

Received '0packetx' from '0.0.0.0'.

接收到的缓冲区格式正确，但地址明显错误。为什么？和函数外的地址变量的定义有关系吗？

编辑

如果在之后memset我cliaddr还添加了这 3 行：

cliaddr.sin_family = AF_INET;
cliaddr.sin_addr.s_addr = htonl(INADDR_ANY);
cliaddr.sin_port = htons(SERV_PORT);

我得到一个随机的行为。有时我得到0.0.0.0，有时127.0.0.1有时是正确的地址（192.168.1.251）。

4

2 回答 2

4

您将错误的长度传递给 recvfrom

  socklen_t *plen = malloc(sizeof(struct sockaddr *));
    if ((recvfrom(sockfd, buff, sizeof(buff), 0, (struct sockaddr *) sndaddr, plen))

为什么你 malloc 是一个谜，但你需要

socklen_t *plen = malloc(sizeof(socklen_t));
*plen = sizeof(struct sockaddr_in );

更简单的是

      socklen_t plen = sizeof(struct sockaddr_in);
   if ((recvfrom(sockfd, buff, sizeof(buff), 0, (struct sockaddr *) sndaddr, &plen))
于 2018-01-22T18:56:51.970 回答
-3

如果您用有效的 sizeof 数据填充 plen，它应该可以工作。

*plen = sizeof(struct sockaddr_in);

你可以把它放在 recvfrom 调用之前。

于 2018-01-22T18:44:07.240 回答