2011-09-12 66 views
2

我用UDP写了下面的echo服务器,但我不知道为什么它给了我sendto函数中的分段错误,它收到罚款,但有问题发送数据回客户端。我一直试图找到几个小时的问题,但没有得到任何地方。请有人指出错误或我可能做错了什么。 感谢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