Copy multidimensional array to single array?

Hi!
Been working all day trying to copy this multidimensional array to a single dimensional array. Everytime I try I either get a segmentation fault or it tries to access some wierd memory stuff :p

Here's the code if you'd like to check it. It is using OpenSSL to hash a text so don't mind the OpenSSL code (I know that part is working)

Code:

---

char *EncryptString(const char *encryption, const char *string) {
  EVP_MD_CTX mdctx;
  const EVP_MD *md;
  unsigned char mdv[EVP_MAX_MD_SIZE];
  int md_len;

  OpenSSL_add_all_digests();
  md = EVP_get_digestbyname(encryption);

  EVP_MD_CTX_init(&mdctx);
  EVP_DigestInit_ex(&mdctx, md, NULL);
  EVP_DigestUpdate(&mdctx, string, strlen(string));
  EVP_DigestFinal_ex(&mdctx, mdv, &md_len);
  EVP_MD_CTX_cleanup(&mdctx);

  char mdv_temp[255][5];
  char r_string[512];

  int i=0;

  for(i=0;i<md_len;i++) {
      sprintf(mdv_temp[i],"%02x",mdv[i]);
  }

  for(i=0;i<255;i++) {
      r_string[i] = mdv_temp[i];
  }
}

---

What i want to know is if there is any cool (read: good) way to do this (You are free to tell me how) and/or what I am doing wrong.

What exactly are you trying to do? First off, 'r_string' is too small. It needs to be 255 * 5 characters in length. 255 * 4 + 1, if you strip all of the null characters off of each string, assuming they're all strings. Then just loop through and strcat each line, assuming that they are all actually strings.

Code:

---

char mdv_temp[ 255 ][ 5 ];
char r_string[ 255 * 4 + 1 ] = { 0 };
size_t x;

/* ...do stuff... */

for( x = 0; x < 255; x++ )
{
    strcat( r_string, mdv_temp[ x ] );
}

---

Quzah.

What i want to do is to copy mdv_string to r_string (Where mdv_string is a multidimensional array).

Your example won't copy the null termination, right?

It won't copy the NULs but there will be one at the end when it's finished copying.
And shouldn't the size of r_string be 255*5 - 4?

but i'm keep getting that strange output (like it tries to access memory it shouldn't)

EDIT: I just noticed that the output i want is in the beginning of that weird output :P

Copying a multidementional array into a single dementional array is easy. Simply treat the multidementional array as a single dementional array, using multiArray[0] where you need the name of the array. If your compiler will not let you do this, or issues a warning you could use a pointer to multiArray[0][0] to achieve the same effect.

For concatanating a multidementional array of strings, it is better to use strcat() to get rid of all the null characters.


... If you're printing strange output at the end of the expected output, you're probably missing a null character.

Thanks for the info, but how could i copy the null sign too? I played around with adding it to the end of the array and it didn't give me the weird output anymore (just the output i wanted). Maybe not just copy it, but at least get the position where i should add it.

Quote:

---

Originally Posted by OnionKnight

It won't copy the NULs but there will be one at the end when it's finished copying.
And shouldn't the size of r_string be 255*5 - 4?

---

No. Why would it be that?

There are 255 strings, each 5 characters long, including the null. Subtract one from each one, for the null, multiply by the number of strings, and add one, so that you can null terminate.

size = (255 * (5 - 1)) + 1

As for the rest of you who are lost in confusion, read what I wrote before, and go on your merry way.

Why is it no one listens to anything I say?


Quzah.

I listen to you quzah.

Quote:

---

Originally Posted by seepox

Thanks for the info, but how could i copy the null sign too? I played around with adding it to the end of the array and it didn't give me the weird output anymore (just the output i wanted). Maybe not just copy it, but at least get the position where i should add it.

---

Just copy each value using this:

Code:

---

for(i=0;i<max;i++)
      singleArray[i]=multiArray[0][i];

---