2011-03-18 58 views
0

我知道有相关的问题已经回答,但我没有设法解决我的问题。UDP C服务器没有收到数据包

我有一个简单的UDP客户端服务器应用程序。客户端&服务器看起来没问题,但是服务器没有收到来自客户端的数据包,它只是无限接收到网络ID 192(或者客户端可能无法正确发送包)。

我似乎无法找出问题,地址&端口都没问题,我没有任何防火墙,我甚至为端口添加了一个异常,只是为了确保。如果我启动服务器,我可以看到它正在听,确定在正确的端口(netstat -a -s -p udp)。你能给我一个提示什么是错的吗?

这里是我的服务器代码:

/*.. includes */ 
#define PORT 8888 
#define NPACK 10 
#define MAXLEN 100 

void signalError(char* s){ 
    perror(s);  
    exit(1); 
} 

int main() 
{ 
struct sockaddr_in struct_srv, struct_client; 
int s,i,cod, numbytes; 
size_t clientSize; 
int32_t nr; 
int32_t sir1[MAXLEN], sir2[MAXLEN], sirComune[MAXLEN], nrEl1, nrEl2, nrComune; 

//Creating the socket: 
s = socket(PF_INET, SOCK_DGRAM, 0); 
if(s==-1) signalError("Error while creating socket!"); 

memset(&struct_srv, 0, sizeof(struct_srv)); 
struct_srv.sin_family = AF_INET; 
struct_srv.sin_addr.s_addr = htonl(INADDR_ANY); 
struct_srv.sin_port = htons(PORT); 

s = bind(s, (struct sockaddr*) &struct_srv, sizeof(struct_srv)); 
if(s==-1) signalError("Bind error. Port is already in use!"); 

//receive packets:  
nrEl1 = -1; 

char buf[MAXLEN]; 
printf("Accepting packets:\n"); 
//for(i=0;i<NPACK;i++) { 
for(;;) { 
    //Receive packets from client: 
    clientSize = sizeof(struct_client);   
    numbytes = recvfrom(s, buf, MAXLEN - 1, 0, 
       (struct sockaddr*) &struct_client, &clientSize); 
    buf[numbytes] = '\0'; 

    printf("Packet is %d long.\n", numbytes); 
    printf("Packet contains %s:\n", buf);  
    sleep(3); 
} 

close(s); 
return 0; 
} 

而我的客户端代码:

/* includes */ 
#define SRV_IP "127.0.0.1" 
#define NPACK 100 
#define MAXLEN 100 
#define PORT 8888 

void signalError(char* s){ 
    perror(s); 
    exit(1); 
} 

int main(void) 
{ 
struct sockaddr_in struct_client; 
int s, i, result, size_client = sizeof(struct_client); 
int32_t nr, sir1[MAXLEN], sir2[MAXLEN]; 

s = socket(PF_INET, SOCK_DGRAM, IPPROTO_UDP); 
if(s==-1) signalError("Erorr creating socket!"); 

memset((char*) &struct_client, 0, sizeof(struct_client)); 
struct_client.sin_family = AF_INET; 
struct_client.sin_addr.s_addr = htonl(INADDR_ANY); 
struct_client.sin_port = htons(PORT); 
if(inet_aton(SRV_IP, &struct_client.sin_addr)==0) { 
    fprintf(stderr, "inet_aton() failed\n"); 
    exit(1); 
} 

char buf[MAXLEN]; 
int len; 
for(i=0;i<NPACK;i++) {  
    printf("Give packet %d:\n", i+1);  
    fgets(buf, sizeof(buf), stdin); 
    buf[strlen(buf)-1] = '\0'; 
    printf("I've read %s\n", buf); 
    printf("Sending packet %d\n", i+1); 
    result = sendto(s, buf, sizeof(buf)+1, 0, 
      (struct sockaddr*) &struct_client, size_client); 
    if(result==-1) signalError("Error sending packets!");   
} 

close(s); 
return 0; 
} 
+0

你有没有试过嗅探应用程序之间的数据包?简单的netcat足以实现它。 – p4553d 2011-03-18 13:47:02

+0

nr和buf是我必须发送2个整数数组到服务器。我第一次尝试发送数字(没有运气),然后我想看看它是否适用于字符串。但这不是问题。我还添加了乔纳森的第一个建议,但仍然是相同的结果。服务器只是收到垃圾。 – joanna 2011-03-18 14:23:30

+0

根据上面的代码,你仍然发送'&nr',这是从来没有初始化。你需要设置'nr'使它发送除了已经存在的东西以外的东西,听起来就像是ASCII字符“192”。 – Jonathan 2011-03-18 14:40:27

回答

1

这里有几个小问题:

  1. 在你的服务器上,每次调用recvfrom之前设置size_client = sizeof(struct_client)。它既是输入参数又是输出参数,所以你要确保一次调用的输出不会干扰到下一次的输入。
  2. 确保将buf的最后一个字节设置为,以防止printf超出边界读取内存。
  3. 什么是cod?它从哪里来的?我想你的意思是测试numbytes
  4. 设置buf[strlen(buf) - 1] = '\0'截断缓冲区的最后一个字符;你想要做什么?

而主要的原因这是行不通的:

你永远不会初始化nr,所以没有告诉什么SENDTO被发送。这当然不发送buf,我认为这是你想要的。你可能希望这样的:

sendto(s, buf, sizeof(buf), ...) 
+0

我修改了你的建议,但仍然一样。所有这些都是我关心的事情(比如'\ 0'),但现在它只是令人讨厌,因为我无法弄清楚问题所在,并在经过一些草率修改后发布了代码。 – joanna 2011-03-18 14:35:17

0

在您的客户端代码,即过网发送没有初始化,可以指向nr变量无论它想要什么。您可能想发送您的buf阵列。

1

除非我错过了一些激烈的东西,你只是在这里发送垃圾。

nr永远不会在发件人上初始化,除非在评论中。 buf正在初始化,但没有在任何地方使用。

你可以尝试改变的sendto部分是这样的:

result = sendto(s, buf, strlen(buf)+1, 0, 
     (struct sockaddr*) &struct_client, size_client); 
相关问题