Получение символов мусора после передачи содержимого файла через UDP

Question

Разработка клиентского/серверного приложения с использованием UDP для выполнения программы на удаленном сервере. то есть клиент отправляет исполняемый файл на сервер, сервер выполняет файл, сохраняет результат в файле и отправляет обратно клиенту.

Я закодирован 2 файла

// посылает имя исполняемого файла на сервер; сервер выполняет его // echo program; это код клиента

 #include <sys/types.h> 
     #include <sys/socket.h> 
     #include <stdio.h> 
     #include <stdlib.h> 
     #include <string.h> 
     #include <netinet/in.h> 
    #include <arpa/inet.h> 
    #include<fcntl.h> 
    #include<unistd.h> 
    #include<stdlib.h> 
    void usage(const char *progname) 
     { 
     fprintf(stderr, "Usage: %s <server-IP> <server-port>\n",   
       progname); 
     } 
    int main(int argc, char *argv[]) 
     { 

     if(argc != 3) 
     { 
     usage(argv[0]); 
     exit(1); 
     } 
    int len, result, sockfd,clilen; 
     int n; 
     struct sockaddr_in seraddr; 
     char buf[310], buf1[500]={0}; 
     sockfd = socket(AF_INET, SOCK_DGRAM, 0); 
     if(sockfd<0) 
      { // Check error condition 
       perror("socket failed"); return -1; 
      } 


    bzero(&seraddr, sizeof(seraddr)); 

    seraddr.sin_family = AF_INET; 
    seraddr.sin_addr.s_addr = INADDR_ANY; 
    seraddr.sin_port=htons(atoi(argv[2])); 
    int ap; 

      clilen=sizeof(struct sockaddr_in); 
    //read contents of file into buf; send to server 
    int fd = open("Executable_Client.c", O_RDWR); 
     int n1 = read(fd, buf, sizeof(buf)); 

     n = write(sockfd,buf,n1); 
     ap=sendto(sockfd,buf,n1,0,(const struct sockaddr 
     *)&seraddr, sizeof(struct sockaddr)); 

     int n2 = recvfrom(sockfd, buf1, sizeof(buf1), 0, (struct  
         sockaddr *)&seraddr, &clilen); 
     n = read(sockfd,buf1, n2); 

     printf("The output is: %s\n", buf1); 
     return 0; 
     } 

и

Server.c

// эта программа принимает исполняемый файл от клиента, и выполняет его // эхо-программы; это код сервера

 #include<stdio.h> 
     #include<string.h> 
     #include<sys/types.h> 
     #include<sys/socket.h> 
     #include<netinet/in.h> 
     #include<fcntl.h> 
     #include<unistd.h> 
     #include<fcntl.h> 
     #include<unistd.h> 
     #include<stdlib.h> 

     void usage(const char *progname) 
     { 
     fprintf(stderr, "Usage: %s <server-port>\n", progname); 
     } 


     int main(int argc, char *argv[]) 
     { 

     if(argc != 2) 
     { 
     usage(argv[0]); 
     exit(1); 
     } 

    int sockfd, newsockfd,portno, client, n, ans,clilen; 
    char buf[310], buf1[256] = {0}; 

    struct sockaddr_in seraddr, cliaddr; 
    int i, value; 
    sockfd = socket(AF_INET, SOCK_DGRAM, 0); 

    if(sockfd<0) 
    { 
    perror("socket failed"); return -1; 
    } 

    int size=sizeof(struct sockaddr); 
     bzero(&seraddr,size); 

    seraddr.sin_family = AF_INET; 
    seraddr.sin_addr.s_addr =inet_addr("127.0.0.1"); 
    seraddr.sin_port = htons(atoi(argv[1])); 


    int a=bind(sockfd, (struct sockaddr *)&seraddr, size); 
      if (a<0) 
      { 
      perror("bind"); close(sockfd); return -1; 
      } 

      clilen=sizeof(struct sockaddr_in); 

    int fd1 = open("Executable_Server.c", O_RDWR); 
    //write contents into Executable_Server.c 
    int n1 = recvfrom(sockfd, buf, sizeof(buf), 0, (struct  
         sockaddr *)&seraddr, &clilen); 
      n = write(fd1, buf, n1); 

    system("gcc Executable_Server.c -o execute.out"); 
    system("./execute.out>Output.txt");//>"Output.txt"); 

    //read contents from Output.txt 
    int fd = open("Output.txt", O_RDWR); 
    int n3 = read(fd, buf1, n1); 

    //send contents to client 

    n = write(sockfd, buf1,n3); 
      n = sendto(sockfd, buf1, n3, 0, (const struct 
         sockaddr *)&seraddr, sizeof(struct 
         sockaddr)); 
    } 

также, код Executable_Client.c «S является

// исполнимого этого файла будет отправлен на сервер от клиента; этот файл считывает содержимое файла

 #include<stdio.h> 
     #include<stdlib.h> 
     int main() 
     { 
     int a, b, c; 
     a = 3; 
     b = 4; 

     c = a+b; 

     printf("a+b = %d\n", c); 
       return 0; 
     } 

После выполнения,

в Executable_Server.c, кроме фактического кода Executable_Client.c., много ценного мусора тоже присутствует. В результате в оконечном окне в ubuntu 12.04 показано много ошибок, связанных с блуждающими символами, которые появляются во время выполнения. Может ли кто-нибудь предложить причину этой аномалии?

Answer 1

Код неправильный в разных местах. Просто упомянем наиболее очевидный из короткого взгляда:

Вы смешиваете recvfrom с чтением и пытаетесь прочитать те же данные (то же самое со смешением sendto и писать позже).

int n2 = recvfrom(sockfd, buf1, sizeof(buf1), 0, (struct  
        sockaddr *)&seraddr, &clilen); 
    n = read(sockfd,buf1, n2); 

И вы распечатываете буфер независимо от количества фактически полученных вами данных. Это, конечно же, будет распечатывать любые нежелательные данные, которые были в буфере перед чтением и которые не были перезаписаны чтением.

printf("The output is: %s\n", buf1); 

И, конечно, это плохая идея, чтобы сделать такие вещи, с UDP до тех пор, пока вы не заботитесь о пакетном переназначении, потерях пакетов и дублировании пакетов.