Search code examples
socketsnetwork-programmingserverudpsendto

send_to returns EINVAL in udp


I am creating a simple file transfer application, but send_to is returning EINVAL when i am trying to send from server side to client, while both send_to and recv_from are working fine on the client side. port_number, portno, hostname are passed as arguments.

Code to set up server:

portno=port_number;

    sockfd = socket(AF_INET, SOCK_DGRAM, 0);
    if (sockfd < 0)
      error("ERROR opening socket");


    optval = 1;

    setsockopt(sockfd, SOL_SOCKET, SO_REUSEADDR,
         (const void *)&optval , sizeof(int));


    bzero((char *) &serveraddr, sizeof(serveraddr));
    serveraddr.sin_family = AF_INET;
    serveraddr.sin_addr.s_addr = htonl(INADDR_ANY);
    serveraddr.sin_port = htons((unsigned short)portno);


    if (bind(sockfd, (struct sockaddr *) &serveraddr,
       sizeof(serveraddr)) < 0)
      error("ERROR on binding");
      //getsockname(sockfd, (struct sockaddr *)&clientaddr, &clientlen);
      cout<<"server port no"<<serveraddr.sin_port<<endl;
    clientlen = sizeof(clientaddr);

Code to set up client: hostname = name; portno = port;

/* socket: create the socket */
sockfd = socket(AF_INET, SOCK_DGRAM, 0);
if (sockfd < 0)
 error("ERROR opening socket");



/* gethostbyname: get the server's DNS entry */
server = gethostbyname(hostname);
if (server == NULL)
{
  fprintf(stderr,"ERROR, no such host as %s\n", hostname);
  exit(-1);
}

/* build the server's Internet address */
memset((char *) &serveraddr,0, sizeof(serveraddr));
serveraddr.sin_family = AF_INET;
bcopy((char *)server->h_addr,
(char *)&serveraddr.sin_addr.s_addr, server->h_length);
serveraddr.sin_port = htons(portno);
serverlen = sizeof(serveraddr);
struct timeval tv;

tv.tv_sec = 1;                       // TIMEOUT IN SECONDS
tv.tv_usec = 0;                      // DEFAULT
if(setsockopt(sockfd, SOL_SOCKET, SO_RCVTIMEO, &tv, sizeof(tv)) < 0)
  printf("Cannot Set SO_RCVTIMEO for socket\n");

Command that is failing:

  if(sendto (sockfd, ack, strlen(ack), 0,(struct sockaddr*) 
  &clientaddr,sizeof(clientaddr) < 0)
       error("ERROR in sending hello_ACK");

where clientadddr is declared as struct sockaddr_in


Solution

  • EINVAL error is generated because of an invalid argument. So you should either initialize the clientaddr as you have done for serveraddr or if you are receiving a HELLO message earlier/ or any other message from the client you can pass clientaddr as an argument to that recvFrom call.

    recvfrom(socketFileDescriptor, buffer, bufferSize,0,(struct sockaddr *) &clientaddr, &clientlen))
    

    here the clientlen is sizeof(clientaddr) if you are writing in C.