我们正在尝试应用基于UDP的协议,并且在sendto()函数中存在一些问题.
当我们尝试用ack响应写请求时,我们从sendto()函数得到"无效参数"
这是我们的代码:
int sock; // Socket
sockaddr_in_t echoServAddr; // Local address
sockaddr_in_t echoClntAddr; // Client address
unsigned int cliAddrLen; // Length of incoming message
data_packet_t echoBuffer;
wrq_packet_t wrqBuffer;
unsigned short echoServPort; // Server port
int recvMsgSize; // Size of received message
ack_packet_t Ack;
struct timeval timeout;
fd_set fds;
/* Create socket for sending/receiving datagrams */
if ((sock = socket(AF_INET, SOCK_DGRAM, IPPROTO_UDP)) < 0) perror("TTFTPERROR: socket() failed");
/* Construct local address structure */
memset(&echoServAddr, 0, sizeof(echoServAddr));
echoServAddr.sin_family = AF_INET;
echoServAddr.sin_addr.s_addr = htonl(INADDR_ANY);
echoServAddr.sin_port = htons(echoServPort);
/*Bind to the local address */
if (bind(sock, (struct sockaddr *) &echoServAddr, sizeof(echoServAddr)) < 0) perror("TTFTPERROR: bind() failed");
FD_ZERO(&fds);
FD_SET(sock, &fds);
timeout.tv_sec = WAIT_FOR_PACKET_TIMEOUT;
timeout.tv_usec = 0;
while (1) {
recvMsgSize = recvfrom(sock, &wrqBuffer, FULL_PACKET_SIZE, 0, (struct sockaddr *) &echoClntAddr, &cliAddrLen);
if (recvMsgSize > 0) break; // we got something!
}
Ack = CreateAckPacket(0); // send ack 0
if (sendto(sock, &Ack, sizeof(Ack), 0, (struct sockaddr *) &echoClntAddr, sizeof(echoClntAddr)) == -1){
perror("TTFTPERROR: sendto() failed to send ack 0");
exit(-1);
}
Run Code Online (Sandbox Code Playgroud)
你能帮我们理解什么是错的吗?
可能的错误是sendto线和recvfrom上面几行的组合.
你正在做的是,你重复使用sockaddr recvfrom给你发送回复(在这种情况下是ACK).这绝对是合法和正确的,你怎么能在无连接协议上向某人发送回复(你没有其他方式知道发送答案的人,也没有协议版本!).
但是:当你正确地提供&cliAddrLen给recvfrom这样它可以返回的实际地址长度(这可能会改变,想到的IPv4 VS IPv6地址)被写入到缓冲区echoClntAddr,你再打sendto与sizeof(echoClntAddr)作为地址字段的大小.
因为echoClntAddr很可能是sockaddr_storage(无论如何),它的大小将大于任何有效协议的地址大小.因此,这是一个无效的论点.或者更确切地说,地址是.
你应该改为调用sendto与cliAddrLen(和初始化cliAddrLen到 sizeof(echoClntAddr)之前调用recvfrom).