Preparing packet to send

Hi everybody! I need your help for a problem. I have to prepare a packet to send. The packet is a char* and it is divided into two parts:
- the first one is a struct made in this way

Code:

---

  struct data
 {
  char a;
  char b;
  short c;
  short dim;
  char e;
  char f;
  short data_points;
  char* value;
}

---

and its size is 16 bytes (sizeof(data)).

- the second one is a vector (type short int) that contains values and its size depends on data.dim value.

Now I have to concatenate the two strings and I do it in this way:

1) I declare two char* variables
char* buf1 = (char*)data;
char* buf2 = (char*)value;

2) I use char *strcat ( char * dest, const char * src ) function (<string.h>)
char* buffer = strcat(buf1,buf2);

Once that I have transmitted the package, I receive the correct number of bytes but I'm able to extract only the struct values from the buffer. So before sending the package I try to print out the buffer:

Code:

---

data* l;
 l = (data*)buffer;
 cout << "Data received: " << endl;
 cout << "A: " << l->a << endl;
 cout << "B: " << l->b << endl;
 cout << "C: " << l->c << endl;
 cout << "Dim: " << l->dim << endl;
 cout << "E: " << l->e << endl;
 cout << "F: " << l->f << endl;
 cout << "Data_Points: " << l->data_points << endl;

---

and the values (struct data) printed are right.

Now I try to print out the values (short int* values):

Code:

---

buffer+=sizeof(data);  //I move the pointer of sizeof(data) bytes (is it right???)
 short int* f;
 f = (short int*)buffer;
 cout << "Values: " << endl;
 for(int j=0; j<(data.data_points); j++)
 {
    cout << " Ordinate " << f[j];
  }

---

and the values printed are wrong.

Where is the mistake?
Thanks.
-David-
p.s.: Sorry for my poor english...

You cannot use strcat to concatenate the two variables into one char* variable because strcat determines the size of the data to copy by finding 0-chars in both strings. It will propably not work as expected. Use memcpy instead.
Kurt

With memcpy it works fine.
Thanks

memcpy doesn't allocate memory either. Allocate a new string with malloc, and then copy them both into it.


Quzah.