tags:

views:

61

answers:

3

I need to allocate memory for a pointer which needs to be used as a 2d array.I know how to allocate memory for char pointers and int pointers I am confused how memory is allocated of a array of pointers.A pictorial representation of the reason would be very helpful,also is the code below fine?

char *names[5];
for(i=0;i<5;i++)
{
 names[i]=(*char)malloc(sizeof(char));
}
+2  A: 

No, this is not because you are allocating the array assuming a dimension of just 1 element of primitive type char (which is 1 byte).

I'm assuming you want to allocate 5 pointers to strings inside names, but just pointers.

You should allocate it according to the size of the pointer multiplied by the number of elements:

char **names = malloc(sizeof(char*)*5);

You don't need to allocate them one by one with a loop. Note that you need to specify that it is a pointer-of-pointers by using **

Jack
Don't cast the return value from `malloc()`. http://c-faq.com/malloc/mallocnocast.html
Tim
A: 

Also, in short you can do the following too in this case for allocating storage for five characters

names = (char*)malloc(5 * sizeof(char))

kuriouscoder
A: 

What you're doing is allocating space for 5 chars. You could write this and it'll have the same result:

char *names = (char *)malloc(sizeof(char) * 5);

If you want to have an array of pointers, I think this'd be the best code

char **names = (char **)malloc(sizeof(char *) * 5);

I'm not a super-coder, but as what I now, this is the right solution.

Ief2