c-简单UDP服务器回复不起作用

Question

现在我只是想让我的"服务器"在收到UDP数据报后立即回复（为更复杂的事情做准备）。

但是，我在sendTo上（在运行时）一直收到一个Invalid Argument错误。有人能帮忙指出原因吗？我写过其他sendTo，也看过互联网上的示例代码，但无法确定原因。

recvfrom工作正常。

谢谢！

void *receive(void *socket)
{
    int* socket_ptr = (int *) socket;
    int socket_desc = *socket_ptr;
    int recv_length;
    socklen_t addrlen = sizeof (struct sockaddr_in);
    while(1){
             struct Message msg;
             struct sockaddr_in * incoming = malloc (sizeof (struct sockaddr_in));
             if ((recv_length = recvfrom (socket_desc, &msg, sizeof(msg), 0,
                 (struct sockaddr *) incoming, &addrlen))
                  == -1)
             {
             }else
             {
                  struct CoordMessage * cm = malloc(sizeof (struct CoordMessage));
                  cm->msgID = BEGINSUCCESS;
                  cm->tid = msg.tid;
                   if (sendto(socket_desc, cm, sizeof(struct CoordMessage), 0, (struct sockaddr *) incoming, sizeof(incoming)) < 0){
                           perror("TEMP Send failed ");
                           exit(1);
                   }
             }
   }
}  

Answer 1

sendto调用中的sizeof(incoming)只是指针的大小，您需要传递从recvfrom调用中获得的结构的大小。此外，malloc会导致内存泄漏，因为您永远不会free内存。

我建议将结构声明为

struct sockaddr_in incoming = {0};

然后将结构的地址传递给recvfrom和sendto，例如

if ((recv_length = recvfrom (socket_desc, &msg, sizeof(msg), 0,
    (struct sockaddr *) &incoming, &addrlen))
     == -1)

和

if (sendto(socket_desc, cm, sizeof(struct CoordMessage), 0, (struct sockaddr *) &incoming, addrlen) < 0)

同样的事情也适用于cm变量，如果你malloc它，你需要free它。但你可以声明它，然后传递地址。