Плохой файловый дескриптор — простой UDP-клиент

При попытке создать простой 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);

}

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


Ответы (2)


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

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

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

person Eric Fortin    schedule 30.01.2011