How to send and receive data socket TCP (C/C++) [duplicate]

Answer 1

Adding to @hmjd's find:

declared at the var decls is your text pointer...

char * text;

then later...

strcpy(text,buf);
while(nbytes_read > 0){
   nbytes_read=recv(sclient(char*)buf,sizeof(buf),0);
   strcat(text,buf);
}

Maybe try setting that 'text' pointer to something beside a random stack value will help as well.

Continuing the barrage, though the following will not necessarily blow up, your date variable as:

char date[8];

on both client and server side The client variable isn't used at all. The server variable, however is:

nbytes_read=recv(sclient,(char *)date,sizeof(date),0);
if(nbytes_read > 0){

Problem is, the date you sent is, in fact, 8 chars wide already: "12/10/12". Therefore, even if you firm-up a null terminator on the end of your string, which you should always do regardless (good practice):

date[ sizeof(date)/sizeof(date[0])-1 ] = 0;

you'll be truncating off the last char of your date.

There are other things wrong with this; we've only pointed out a few. Think about sending length-prefixes with each of these data values in the array, with checks or range to ensure you get what you expected.

Finally, spending some time on the business-end of a debugger would probably do you very well, especially on the server side.

Answer 2

Here's a simple "receive all" function:

int recv_all(int sockfd, void *buf, size_t len, int flags)
{
    size_t toread = len;
    char  *bufptr = (char*) buf;

    while (toread > 0)
    {
        ssize_t rsz = recv(sockfd, bufptr, toread, flags);
        if (rsz <= 0)
            return rsz;  /* Error or other end closed cnnection */

        toread -= rsz;  /* Read less next time */
        bufptr += rsz;  /* Next buffer position to read into */
    }

    return len;
}

Answer 3

One (repeated) mistake is:

nbytes_read=recv(sclient,(char *)date,sizeof(date),0);

recv() does not null terminate. This means date will not have a null terminator if sizeof(date) bytes is read. This is a problem when a non-null terminated string is passed as an argument to printf() with "%s" format specifier. If the string is non-null terminated you may see garbage characters appearing after the actual string data. You need to read one less than the target buffer and null terminate or use the format specifier "%*.s" that does not require null termination:

printf("%.*s", n, s); /* Prints first 'n' bytes from 's'. */

---

Note you can initialise a char[] to all nulls instead of using a for:

char date[8] = "";

or you can use memset().