C++ Winsock2 Recv Müll

Question

Also schreibe ich einen Windows Chat und zu Testzwecken sendet mein Client-Programm eine "Hallo" Nachricht an den Server alle 300 ms.

ersten paar Nachrichten kommen gut, aber dann wie ohne Grund beginnen sie junk-

Offensichtlich werde ich es reparieren wollen und ich suche für Ihre Hilfe :) Hier ist mein Code:

Send-Funktion:

bool Target::Send(char *message) 
{ 
    int length = strlen(message); 
    int result = send(this->ccSock, (char*)&length, sizeof(int), 0); 
    if (result <= 0) 
     return false; 

    Sleep(10); 

    result = send(this->ccSock, message, length, 0); 
    return ((result > 0) ? true : false); 
} 

Empfangsfunktion:

Message Server::Receive(SOCKET socket) 
{ 
    int length = 0; 
    int result = recv(socket, (char*)&length, sizeof(int), 0); 

    Sleep(10); 

    char *rcvData = new char[length]; 
    result = recv(socket, rcvData, length, 0); 

    return { rcvData, result }; 
} 

Nachricht struct:

struct Message { 
    char *msg; 
    int size; 
}; 

Haupt senden Code:

while (true) 
{ 
    if (!target->Send("hello")) 
    { 
     cout << "Connection broken\n"; 
     target->Clean(); 
     break; 
    } 

    Sleep(300); 
} 

Haupt erhalten Code:

while (target.sock) 
{ 
    Message message = server->Receive(target.sock); 
    if (message.size > 0) 
     cout << message.msg << " (" << message.size << ")\n"; 
    else 
    { 
     cout << "Target disconnected\n"; 
     server->Clean(); 
     break; 
    } 

    Sleep(1); 
} 

Ich würde Ihre Hilfe sehr schätzen und erklären, warum dies geschieht!

Answer 1

Ihr Puffer ist nicht null-terminiert. Wenn Sie also versuchen, es mit std::cout Pufferüberlauf zu drucken, tritt auf. Die richtige Version sollte erhalten Code sein:

char *rcvData = new char[length+1]; 
result = recv(socket, rcvData, length, 0); 
rcvData[length] = '\0'; 

Sie auch nie frei zugewiesenen Speicherpuffer, so dass Ihr Code lecken es auf jeden Receive Anruf.