thank you, will look into it

I think that you're being too liberal with the variety of tools that you use. sprintf, strcpy, memset, the list goes on. You're also using manual pointers.

There exist simpler and safer ways of achieving the desired result.

don't we need to free the memory at some point after we use it ?