C Программирование на сокет

Question

Я новичок в программировании. Я попробовал базовое программирование сокетов. Он работает хорошо, но моя проблема в том, что я создал связанный список для хранения отправленных и полученных сообщений. Когда я печатаю связанный список, отображаются только отправленные сообщения (ptr3-> запрос), и мое полученное сообщение (ptr3-> response) пуст.

Я думаю, что принятие аргументов - это то, где я совершаю ошибку. Но в обоих случаях я передаю адрес только. Я не могу понять ошибку.

Фрагмент кода:

#include<stdio.h> 
#include<string.h> 
#include<sys/socket.h> 
#include<arpa/inet.h> 
#include<time.h> 
#include<errno.h> 
#include<stdlib.h> 

void create(char*,char*); 
void insert(char*,char*); 
void send_receive(char*,int); 
void display(); 

struct packet_details 
{ 
char *request; 
char *response; 
struct packet_details *next; 
}; 

typedef struct packet_details details; 
details *head = NULL ,*temp = NULL; 

void create(char *send,char *receive) 
{ 
    details *new_node; 
    new_node= (details*)malloc(sizeof(details)); 

    if(NULL == new_node) 
    { 
     printf("Cannot allocate requested memory %d %s %s",__LINE__,__FILE__,__FUNCTION__); 
    } 

    else 
    { 
     new_node->request= send; 
     new_node->response = receive; 
     new_node->next = NULL; 

     head = temp = new_node; 
    } 
} 

void insert(char *send,char *receive) 
{ 
    details *new_node; 
    if(NULL == head) 
    { 
     create(send,receive); 
     return; 
    } 
    new_node= (details*)malloc(sizeof(details)); 

     new_node->request = send; 
     new_node->response = receive; 
     new_node->next = NULL; 

     temp->next = new_node; 
     temp = new_node; 
} 

void display() 
{ 
    details *ptr3; 
    ptr3 = head; 
    while(NULL != ptr3) 
    { 
     printf("\n**********************************\n"); 
     printf("\n**********************************\n"); 

     printf("\n%s",ptr3->request); 
     printf("\n%s",ptr3->response); 

     printf("\n***********************************\n"); 
     printf("\n***********************************\n"); 
     ptr3=ptr3->next; 
    } 
} 

int main() 
{ 
    int socket_desc,a=0; 
    struct sockaddr_in server; 
    char *message; 

    //Create socket 
    socket_desc = socket(AF_INET , SOCK_STREAM , 0); 
    if (socket_desc == -1) 
    { 
     printf("Could not create socket! %s\n",strerror(errno)); 
    } 

    server.sin_addr.s_addr = inet_addr("127.0.0.1"); 
    server.sin_family = AF_INET; 
    server.sin_port = htons(7887); 

    //Connect to server 
    if (connect(socket_desc , (struct sockaddr *)&server , sizeof(server)) < 0) 
    { 
     printf("connect error ! %s\n",strerror(errno)); 
     return 1; 
    } 

    puts("Connected\n"); 

    while(a < 5) 
    { 
    printf("\nCLIENT:"); 
    message = "Hello Server\n"; 
    printf("%s",message); 
    send_receive(message,socket_desc); 
    a++; 

    sleep(10); 

    printf("\nCLIENT:"); 
    message = "How are you?\n"; 
    printf("%s",message); 
    send_receive(message,socket_desc); 
    a++; 

    sleep(10); 

    printf("\nCLIENT:"); 
    message = "Are you busy?\n"; 
    printf("%s",message); 
    send_receive(message,socket_desc); 
    a++; 

    sleep(10); 

    printf("\nCLIENT:"); 
    message = "Process my request\n"; 
    printf("%s",message); 
    send_receive(message,socket_desc); 
    a++; 

    sleep(10); 

    printf("\nCLIENT:"); 
    message = "Bye\n"; 
    printf("%s",message); 
    send_receive(message,socket_desc); 
    a++; 
    } 

    close(socket_desc); 
    return 0; 
} 


void send_receive(char *msg,int socket_desc) 
    { 
    char server_reply[2000]; 
    char *reply; 
    if(write(socket_desc,msg,strlen(msg))<0) 
    { 
    printf("\n data could not be sent"); 
    } 
    read(socket_desc,server_reply,sizeof(server_reply)); 
    printf("server:%s",server_reply); 
    reply = &server_reply[0]; 
    insert(msg,reply); /* passing of two addresses as arguments */ 
} 

Может кто-нибудь мне помочь, пожалуйста?

Answer 1

Каждый раз, когда вы вызываете insert, вы передаете адрес буфера, который был выделен в стеке, то есть server_reply и сохраните его в своем списке. Затем, когда существует send_receive, эта память указывает на мусор.

Вместо того, чтобы сделать это:

new_node->request= send; 
new_node->response = receive; 

ли это:

new_node->request= strdup(send); 
new_node->response = strdup(receive); 

Это делает копию отправленных/полученных данных построена в недавно выделенный буфер. Просто убедитесь, что free() все это память при очистке.