UDP-сервер дает ошибку сегментации - PullRequest
Новая
       10

UDP-сервер дает ошибку сегментации

2 голосов
/ 13 сентября 2011

Я написал следующий эхо-сервер, используя UDP, но я понятия не имею, почему он дает мне ошибку сегментации в функции sendto, он получает нормально, но имеет проблемы с отправкой данных обратно клиенту. Я пытался найти проблему в течение нескольких часов, но не получил, где. Может кто-нибудь, пожалуйста, укажите на ошибку или что я могу делать неправильно. Спасибо 

#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;
}

1 Ответ

3 голосов
/ 13 сентября 2011

Эти строки: 

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);
...