как инициализировать буфер

Question

Это мой код сокетов сервера / клиента. У меня проблема с буфером, когда клиент получает сообщение от сервера, который является буфером (я инициализировал его как «12»), вывод выводится так:

[+]Client Socket is created.
[+]Connected to Server.
Temp : 
  12121212121212121212121212121212121212121212121212121212121212121212121212121212 
  12121212121212�L��Q� 

сервер:

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

#define PORT 4444

int main(){

int sockfd, ret;
 struct sockaddr_in serverAddr;

int clientSocket;
struct sockaddr_in newAddr;

socklen_t addr_size;

char buffer[1024];
    char Temp[4]= "12";
pid_t childpid;

sockfd = socket(AF_INET, SOCK_STREAM, 0);
if(sockfd < 0){
    printf("[-]Error in connection.\n");
    exit(1);
}
printf("[+]Server Socket is created.\n");

memset(&serverAddr, '\0', sizeof(serverAddr));
serverAddr.sin_family = AF_INET;
serverAddr.sin_port = htons(PORT);
serverAddr.sin_addr.s_addr = inet_addr("127.0.0.1");

ret = bind(sockfd, (struct sockaddr*)&serverAddr, sizeof(serverAddr));
if(ret < 0){
    printf("[-]Error in binding.\n");
    exit(1);
}
printf("[+]Bind to port %d\n", 4444);

if(listen(sockfd, 10) == 0){
    printf("[+]Listening....\n");
}else{
    printf("[-]Error in binding.\n");
}


while(1){
    clientSocket = accept(sockfd, (struct sockaddr*)&newAddr, &addr_size);
    if(clientSocket < 0){
        exit(1);
    }
    printf("Connection accepted from %s:%d\n", inet_ntoa(newAddr.sin_addr), 
   ntohs(newAddr.sin_port));

    if((childpid = fork()) == 0){
        close(sockfd);

        while(1){



        send(clientSocket, Temp, strlen(Temp), 0);
                bzero(buffer, sizeof(buffer));
            if(strcmp(Temp, ":exit") == 0){
                printf("Disconnected from %s:%d\n", 
 inet_ntoa(newAddr.sin_addr), ntohs(newAddr.sin_port));
                break;
        }
    }

 }



}
return 0;
}

клиент:

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

#define PORT 4444

int main(){

int clientSocket, ret;
struct sockaddr_in serverAddr;
char buffer[1024];
    char Temp[4];

clientSocket = socket(AF_INET, SOCK_STREAM, 0);
if(clientSocket < 0){
    printf("[-]Error in connection.\n");
    exit(1);
}
printf("[+]Client Socket is created.\n");

memset(&serverAddr, '\0', sizeof(serverAddr));
serverAddr.sin_family = AF_INET;
serverAddr.sin_port = htons(PORT);
serverAddr.sin_addr.s_addr = inet_addr("127.0.0.1");

ret = connect(clientSocket, (struct sockaddr*)&serverAddr, 
sizeof(serverAddr));
if(ret < 0){
    printf("[-]Error in connection.\n");
    exit(1);
}
printf("[+]Connected to Server.\n");
   bzero(buffer,512);

   int words = 0;
    char c;

while(1){


             recv(clientSocket, Temp, 1024, 0);
    if(recv(clientSocket, Temp, 1024, 0) < 0){
        printf("[-]Error in receiving data.\n");
    }
           printf("Temp : %s \n",Temp);
       if(strcmp(Temp, ":exit") == 0){
        close(clientSocket);
        printf("[-]Disconnected from server.\n");

    } exit(1);
}



return 0;
}

Пожалуйста, кто-нибудь может сказать мне, как я могу решить эту проблему, любая помощь будет оценена.

Answer 1

Я вижу несколько проблем с вашим клиентским кодом:

char Temp[4];
...
recv(clientSocket, Temp, 1024, 0);

Здесь вы читаете до 1024 байтов в Temp, хотя для Temp выделено только 4 байта. Если вы читаете более 4 байтов, другие структуры данных будут заполнены прочитанными данными, то есть классическим переполнением буфера.

printf("Temp : %s \n",Temp);

Здесь вы предполагаете, что Temp - это строка с окончанием \0, а это не так. Поэтому он будет распечатывать данные, начиная с позиции &Temp до \0 байта, который он найдет где-то (выход за пределы Temp на основе вашего вывода).

if(strcmp(Temp, ":exit") == 0){

Несмотря на то, что Temp составляет всего 4 байта, этот оператор предполагает не менее 6 байтов (5-байтовая строка и \0 для маркера конца строки).

Аналогичные проблемы есть и в коде сервера. Также странным является то, что ваша переменная buffer инициализируется, но никогда не используется. Я предполагаю, что вы изначально хотели использовать это вместо Temp для отправки и получения.