2011-01-30 9 views
1

単純な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); 

} 
+0

あなたのコードの出力を置くと、私はprintfを意味します。 –

+0

初期化されていないポインタバッファにたくさんのデータを書き込んでいるので、このプログラムはsendtoになるのは驚くべきことです。これがソケットの問題を引き起こしているかどうかを確認するには、宣言をchar buffer [1000]に変更してください。実生活では、何らかの動的割り当てを使用したいと思うかもしれません。ソケットコードは大丈夫ですが、dgramソケットにあまりにも多くのデータを送信しようとすると、sendto()は失敗します。 – Splat

答えて

1

、それはハードなものだが、私は気づいた最初の事はあなたがファイルを書き込むためのバッファを割り当てられないことだった

4

私はいくつかのエラー/問題を参照してください。

  • nをファイルの長さに設定していないようです
  • ファイルを保持するためのメモリ割り当てはありません
  • ファイルを1バイトずつ読むlは、それが持つ最初の組込みのバイトで停止しますので、ファイルをロードした後、あなた、ファイルがバイナリである場合strlen()
  • を使用する必要がファイルの長さを知らない、strlen()は失敗するはずです
  • 非常にinefficentこと値0
関連する問題