2011-09-12 17 views
2

IveはUDPを使用して次のエコーサーバーを作成しましたが、sendto関数でSegmentation Faultが発生する理由はわかりません。私は数時間の間問題を見つけようとしていましたが、どこにもいませんでした。誰かが間違いや何が間違っているかもしれないかを指摘してください。 おかげセグメンテーションフォールトを与えるUDPサーバー

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


#define true 1 
#define false 0 

int main(int argc,char **args){ 

int BUF_LENGTH=101; 
int port_no=1800; 
struct sockaddr_in serv_addr,rmt_addr; 
//rmt_addr=malloc(sizeof(struct sockaddr_in)); 
char *buffer=malloc(BUF_LENGTH); 
int byte_recv=0; 
int rmt_length=0; 

int sock_id; 
sock_id=socket(AF_INET,SOCK_DGRAM,0); 

if(sock_id<0){ 
    printf("Error creating socket : %d",sock_id); 
    return -1; 
} 


serv_addr.sin_family=AF_INET; 
serv_addr.sin_port=htons(port_no); 
serv_addr.sin_addr.s_addr=inet_addr("127.0.0.1"); 

bind(sock_id,(struct sockaddr*)&serv_addr,sizeof(serv_addr)); 
printf("Created\n"); 
while(true){ 
    printf("Waiting\n"); 
    byte_recv=recvfrom(sock_id,buffer,BUF_LENGTH,0,(struct sockaddr*)&rmt_addr,&rmt_length); 




    printf("%s\n",buffer); 
    if(byte_recv<0){ 
     printf("Error receiving: %d",byte_recv); 
     error("recvfrom"); 
     return -2; 
    } 

    printf("%d:%d %s\n",rmt_length,rmt_addr.sin_port,inet_ntoa(rmt_addr.sin_addr)); 

    byte_recv=sendto(sock_id,buffer,BUF_LENGTH,0,(struct sockaddr*)&rmt_addr,rmt_length); //The segmentation fault comes here 


    printf("Bytes sent: %d \n",byte_recv); 
    if(byte_recv<0) 
     error("sendto"); 



} 
free(buffer); 
return 0; 
} 
+0

+1問題を示す完全なコンパイル可能なプログラムを提供しています。 –

答えて

3

は、これらの行:

error("recvfrom"); 

error("sendto"); 

あなたは彼らが何を考えて行いません。あなたはたぶんperrorと言っていました。

さらに、rmt_lengthは正しく初期化されていません。これを試してください:

int rmt_length=sizeof(rmt_addr); 

最後に、サーバーが受け取るバイト数よりも多くのバイトが返されています。これを試してみてください:

byte_recv=sendto(sock_id,buffer,byte_recv,0,(struct sockaddr*)&rmt_addr,rmt_length); 
+0

あなたは正しかった、私の過失のために申し訳ありません。今私はperrorを使用し、perror( "sendto")で "アドレスファミリはプロトコルでサポートされていません" – Zabi

+0

これは 'rmt_length'を正しく初期化しなかったためです。私の編集された答えを見てください。 –

+0

ありがとうRob、それは働いた。私はそのようなばかげたミスをしたと信じていませんが、私はただ学んでいます。 – Zabi

関連する問題