Thread: how free unsigned pointer array in best way (calloc)

> buf = cs_calloc(ca->len + 13);
If this is all you did, then the correct thing is just
cs_free(buf);

You don't look at the parameter to calloc, you look at the result.

Each unique result ultimately needs exactly one call to free.

Originally Posted by Salem

> buf = cs_calloc(ca->len + 13);
If this is all you did, then the correct thing is just
cs_free(buf);

then how cs_free() function should be? this is exactly what i asked (my question remains.) i mean do i need to free each of array elements (free (buf[i]) before free buf ? what's wrong about the free procedure i had wrote above that i get that compile warning :

warning: passing argument 1 of ‘free’ makes pointer from integer without a cast

as also wrote in my first post of this thread , free like this is it correct ?

Code:

for (i = 0; i < ca->len + 13; i++) 


{    

 free(buf[i]); 

buf[i] = NULL;

}

 

 free(buf);

Thanks in advance

Is there something wrong with your eyes or reading skill?

It's right there in the very thing you quoted!

i think now i get what u meant , u meant , it's not need to free the elements of arrays , i just need to free (buf) so i should remove that for loop for freeing buf elements . anyway u didn't say this directly to me , so i guess what u meant . lol

If you get the memory in the first place using calloc() / malloc() / realloc(), then the way to free it is using free()

You can put them inside wrappers like cs_calloc() and cs_free() if you want, that's up to you.

Or you can call free() directly.

The point is, you have
ptr = someAllocationFunction();

then later, you do
free( ptr );

It doesn't matter how much memory it is, what structure it contained, or whether you treated it as an array or not.

ONE ALLOC needs ONE FREE

Simples!