C++/Sockets.h ♣ - glitch or error?

[Adam01]

Hi, I got a working server, and a working client.
the client and sever send a message to each other, in order.
But after user input (for their name)
The server goes into a for loop for the client messages, and then server automaticly recieves the first message as this symbol: ♣, is this just an error or a glitch?

[gpraghuram]

This should be an error.
How are you accepting the connection on server and how the client sends message sends the message back to server?

Raghu

[Adam01]

Ok,

The server accepts multiple clients, and each client has its own thread.

This is the accept loop in main():

for(;;){
    sin_size = sizeof their_addr;
    if((clientsocket = accept(serversocket, (struct sockaddr *)&their_addr, &sin_size)) == -1){
message.write("Client Accept Error.");
std::cout << WSAGetLastError(); message.write("\n");
    }
else {message.write("Client connected: ");
message.write(inet_ntoa(their_addr.sin_addr));
message.write("\n");
_beginthread(clientloop, 0, (void*)clientsocket);
}

After the second call to for() is when the problem is caught.
The previous calls to recv are fine, retrieve valid responces.


I added // <---- comments to the main points of the client thread.
and this is the client thread main stuff: (not a loop)

for(;;){
            std::cout << "::Sending welcome message: ";
            if (send(client[id].clientsocket, "Welcome", 7, 0) == -1)
            {std::cout << "\n!:Could not send message to client\n";
            std::cout << WSAGetLastError(); message.write("\n");}
            else message.write("Sent.\n");
std::cout << "Server: Ping\n";
send(client[id].clientsocket, "-Ping", 5, 0);
if((recv(client[id].clientsocket, client[id].buf, client[id].bufsize, 0)) == -1)
{std::cout << "!:Error recieving pong\n"; break;}
else if(strstr(client[id].buf,"-Pong") == NULL)
{ std::cout << "!:Error recieving pong\n";break;}
else
message.write("Client["); message.write(id); message.write("]"); message.write(": Pong.\n");
if((recv(client[id].clientsocket, client[id].clientname, client[id].namesize, 0)) == -1)
{std::cout << "!:Error recieving client name\n"; std::cout << WSAGetLastError(); message.write("\n"); break;}
else { std::cout << "Server: Recieved client name as '" << client[id].clientname << "'.\n"; }
recv(client[id].clientsocket, client[id].buf, client[id].bufsize, 0); // <--- so the server doesnt display the ♣
for(;;){  // <----- message handling loop
if((recv(client[id].clientsocket, client[id].buf, client[id].bufsize, 0)) == -1)
{ std::cout << "!:Error recieving client message\n";break;}
else {
if(strstr(client[id].buf,"\\#disconnect")) break;
message.write("["); message.write(id); message.write("]"); message.write(client[id].clientname); message.write(": "); message.write(client[id].buf); message.write("\n");
}
}
break;
}