Please see https://stackoverflow.com/questions/605845/do-i-cast-the-res... for some thoughts on this.

In short (as already stated, of course), you don't need to cast void * in C to/from other object pointers. Adding a cast to the return value of malloc() is a bad idea (I'm biased).