Also, I don't think you need to cast malloc(). In fact, it might hide errors.

The type of malloc is void*, I think I remember a compiler telling me that char* is not the same as void*. Consequently I had to cast malloc to char*.