2011-01-30 3 views
1

При попытке создать простой клиент UDP мой код успешно открывает сокет и читает его в небольшом буфере из файла, который будет отправлен на сервер, указанный по адресу хоста и номеру порта по командной строке аргументы.Дескриптор Bad File - простой клиент UDP

Однако sendto и recvfrom оба не работают с «Bad File Descriptor», и я не могу понять, почему.

void main(int argc, char* argv[]){ 
int s, n=0, obytes, inbytes; 
struct sockaddr_in sin; 
char *buffer; 
int address = 0; 

//Checks socket 
if((s = socket(AF_INET, SOCK_DGRAM, 0))<0) { 
    printf("Error creating socket\n"); 
    exit(0); 
} 
//Resets values in socket structure 
memset((char*)&sin, 0, sizeof(sin)); 
sin.sin_port = htons(atoi(argv[2])); 
sin.sin_family = AF_INET; 
sin.sin_addr.s_addr = inet_addr(argv[1]); 
printf("%d\n", sin.sin_addr.s_addr); 
/*Opens file to be sent and reads into buffer*/ 
FILE *readFile; 
readFile = fopen(argv[3], "r"); 
//Checks if file to be read can be opened 
if (readFile==NULL) { 
    perror ("Error opening file"); 
} 
//Reads in all the characters to a buffer 
else{ 
    while (!feof(readFile)) { 
     buffer[n] = fgetc (readFile); 
     n++; 
    } 

buffer[n] = '\0'; 
printf ("Total number of bytes: %d\n", n); 
for(int i = 0; i< n; i++){ 
    printf("%c", buffer[i]); 
} 
} 
printf("File was opened\n"); 
//Sends the buffer to the destination designated by the socket structure and checks to see if bytes were sent 
if((obytes = sendto(s, buffer, strlen(buffer), 0, (struct sockaddr *)&sin, sizeof(sin))) == -1) { 
    perror("Sendto() error!"); 
    exit(1); 
} 
printf("%d bytes were sent\n",obytes); 
//Receives response from the server and checks to see if bytes were actually received 
/* if((inbytes = recvfrom(s, buffer, strlen(buffer)+28, 0, (struct sockaddr *)&sin, sizeof(struct sockaddr*))) == -1) { 
    perror("Recvfrom() error!"); 
    exit(1); 
}*/ 
printf("%d bytes were received.\n", inbytes); 
//Closes file 
fclose (readFile); 

} 
+0

Поставьте вывод своего кода, printf я имею в виду. –

+0

Удивительно, что эта программа получает sendto, потому что она записывает кучу данных в неинициализированный буфер указателя. Чтобы убедиться, что это приводит к сбою вашего файла сокета, измените объявление на char buffer [1000] или что-то еще и посмотрите, работает ли он. В реальной жизни вы, вероятно, захотите использовать какое-то динамическое распределение. Код сокета выглядит нормально, хотя sendto() не удастся, если вы попытаетесь отправить слишком много данных на сокет dgram. – Splat

ответ

1

Без выхода, это своего рода трудно, но первое, что я заметил, что вы никогда не выделяла буфер для записи файла в

4

Я вижу несколько ошибок/проблем:.

  • Кажется, вы никогда не устанавливали n как длину файла
  • Нет выделенного выделения памяти для хранения файла
  • Чтение файла по одному байту за один раз л очень inefficent
  • После загрузки файла вы должны знать длину файла, нет необходимости использовать strlen()
  • Если файл является двоичным, strlen() потерпит неудачу, поскольку он остановится на первые вложенные байтах с значение 0

 Смежные вопросы

  • Нет связанных вопросов^_^