Thread: How to properly free a pointer and its allocated memory

Basically, the rule is "if the pointer is still available, then set it to NULL". If the pointer is local to a function, and you have a free at the very end of the function, then you don't need to set the pointer to NULL.

If you often find yourself writing:

Code:

free(ptr);
ptr = NULL;

you could make a function:

Code:

freeAndNull(void **ptr)
{
   free(*ptr);
   *ptr = NULL;
}

   char *p;
   p = malloc(...);
...
   freeAndNull(&p);
...

Edit: Re-reading the subject of the post: You do not "free" a pointer. You free the memory associate to that pointer. [Of course, you can have a pointer to pointer situation where the first level pointer is also allocated, in which case you'd have to do (at least) two free operations, but it still applies that the original pointer is not freed].


--
Mats